-
Notifications
You must be signed in to change notification settings - Fork 0
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
Accessibility updates #555
Conversation
poojagunturu96
commented
Oct 17, 2023
•
edited
Loading
edited
- ALL: Check the accessibility of link elements used in the gallery component #447
- Table header text in table of links on profile detail appears twice in markup to make design work #542
- ALL: H2 in Main Navigation appears in document before H1 #539
- ALL: aria-labelledby is unsupported when used on the header element #540
- ALL: ARIA attribute aria-selected unsupported on td element #562
…accessbility. For #542.
… causes accessbility issues when an h1 is present after the h2, replace at with aria-label instead. For #539.
<a href="/gallery-images#modal-{{ id }}" class="sr-only" target="_blank">View the gallery in a new window.</a> | ||
<a class="gallery__link" data-lightbox-open="midd-lightbox-1" aria-label="Open gallery" aria-describedby="midd-gallery-1-label midd-gallery-1-count" role="button"> | ||
<a | ||
href="/" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
On this, can we retain the unique URLs for viewing the page of the gallery images at /gallery-images#modal-{{ id }}
? I worry that linking to the homepage isn't contextually relevant.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, I had reverted back to the old code which had href="/"
but I meant for it to be the link to the gallery images page.