Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Screenshot updates to tutorials #2288

Merged
merged 5 commits into from
Mar 1, 2024
Merged

Conversation

rosiel
Copy link
Member

@rosiel rosiel commented Mar 1, 2024

Purpose / why

In a recent commit to the Starter site, we changed the node form, so that Model (required field) is now at the top. I changed the two tutorials and their embedded screenshots to reflect the new form. While I was in there, I made minor edits to wording.

What changes were made?

  • Screenshots updated
  • Screenshots that were jpg are now png (and what a difference in quality!)
  • Changes to "Create a resource node" and "How to create a collection" to reflect this, and just clean up the wording and alt text a little. No major content changes were done.

Verification

Check for grammar and verify the images work.

Interested Parties

  • @Islandora/documentation
  • @Islandora/committers

Checklist

Pull-request Reviewer

Pull-request reviewer should ensure the following:

Person Merging

The person merging should ensure the following:

  • Does mkdocs still build successfully? (This is indicated by TravisCI passing. To test locally, and see warnings, see How To Build Documentation.)
  • If pages are renamed or removed, have all internal links to those pages been fixed?
  • If pages are added, have they been linked to or placed in the menu?
  • Did the PR receive at least one approval from a committer, and all issues raised have been addressed?

@ajstanley ajstanley merged commit e472d32 into Islandora:main Mar 1, 2024
1 check passed
@ajstanley
Copy link
Contributor

Tested locally. It all does what it's meant to.

@rosiel rosiel deleted the screenshot-updates branch March 1, 2024 21:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants