Skip to content

More Globus screenshots#1315

Open
jen-reeve wants to merge 6 commits into
mainfrom
globus-screenshots
Open

More Globus screenshots#1315
jen-reeve wants to merge 6 commits into
mainfrom
globus-screenshots

Conversation

@jen-reeve

Copy link
Copy Markdown
Contributor

No description provided.

@jen-reeve jen-reeve requested a review from janamakar June 22, 2026 20:42
@jen-reeve jen-reeve self-assigned this Jun 22, 2026
@github-actions

github-actions Bot commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

Test deployment successful!! (2026-06-22T20:55:49Z)
Preview available at https://callumwalley.github.io/mkdocs-demo-deploy/nesi/support-docs/globus-screenshots

Seems the following pages differ;



See all deployed demo sites

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

Ahoy, ye lazy landlubbers! Ye've gone and added some fancy pictures to the Globus Bookmarks documentation, but ye couldn't be bothered to write proper alt text for 'em! Just slappin' the filenames in there is a right insult to any blind sailor tryin' to navigate these waters. All three of the reviewer's comments are spot on—fix them alt texts to actually describe what's in the images, or prepare to walk the plank!

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.


1. In the `File Manager` tab, open the collection you are interested in and find your way to the
`Path` you are interested in bookmarking. Then click the bookmark icon.
![Globus_Bookmarking_1.png](../../assets/images/Globus_Bookmarking_1.png)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

Ahoy! I see ye've graced us with screenshots, but did ye lose yer tongue? Usin' the filename as the alt text is a lazy pirate's way. A blind sailor usin' a screen reader won't have a clue what they're lookin' at. How about we write some actual descriptive text instead of just regurgitatin' the filename, eh? Aye, the Accessibility Standards (WCAG) we claim to follow demand better!

Suggested change
![Globus_Bookmarking_1.png](../../assets/images/Globus_Bookmarking_1.png)
![Clicking the bookmark icon in the Globus File Manager](../../assets/images/Globus_Bookmarking_1.png)
References
  1. Accessibility Standards (WCAG spec) are referenced, and image alt text should be descriptive rather than just the filename. (link)

`Path` you are interested in bookmarking. Then click the bookmark icon.
![Globus_Bookmarking_1.png](../../assets/images/Globus_Bookmarking_1.png)
2. Give a name to your bookmark and click `Create Bookmark`.
![Globus_Bookmarking_2.png](../../assets/images/Globus_Bookmarking_2.png)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

Avast! Another filename masqueradin' as alt text! Do ye expect the screen readers to speak in code? Give it a proper description of the bookmark creation dialog, unless ye want the crew to mutiny over poor accessibility.

Suggested change
![Globus_Bookmarking_2.png](../../assets/images/Globus_Bookmarking_2.png)
![Entering a name for the bookmark and clicking Create Bookmark](../../assets/images/Globus_Bookmarking_2.png)
References
  1. Accessibility Standards (WCAG spec) are referenced, and image alt text should be descriptive rather than just the filename. (link)

2. Give a name to your bookmark and click `Create Bookmark`.
![Globus_Bookmarking_2.png](../../assets/images/Globus_Bookmarking_2.png)
3. This will create a bookmark, and make the bookmark icon go gold.
![Globus_Bookmarking_3.png](../../assets/images/Globus_Bookmarking_3.png)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

Shiver me timbers, a third one! 'Globus_Bookmarking_3.png' tells a poor soul absolutely nothin'. Describe the gold bookmark icon so even a one-eyed pirate can understand what's happened here!

Suggested change
![Globus_Bookmarking_3.png](../../assets/images/Globus_Bookmarking_3.png)
![The bookmark icon turning gold after successful creation](../../assets/images/Globus_Bookmarking_3.png)
References
  1. Accessibility Standards (WCAG spec) are referenced, and image alt text should be descriptive rather than just the filename. (link)

@jen-reeve jen-reeve marked this pull request as ready for review June 22, 2026 20:58
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.

1 participant