-
Notifications
You must be signed in to change notification settings - Fork 360
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
Disclosure examples for FAQ and image description: Update accessibility features documentation and implement coding practices in latest code guide #1814
Conversation
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.
Functions appropriately with Firefox/Chrome and NVDA/JAWS latest. HCM Black and White look appropriate.
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.
Code and test review are good.
…her APG examples, and added forced-color-adjust property to the SVG images
I see @jesdaigle provided approving code review. But, I see an unresolved comment by Sarah and Simon about a switch statement with an unnecessary case for the enter key. That case is still in the Javascript. Can we please have a ffresh look at the latest code? I added this to November 2, 2021 Agenda · w3c/aria-practices Wiki so we can check visuals. If anyone has the chance to look at visuals, including in high contrast, before the meeting, that would certainly help. |
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.
Editorial review complete. Approve.
@mcking65 |
High contrast check is complete and two examples look good. |
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.
I see the room for visual design improvement but this is not the deal breaker.
I checked with High contrast mode for two examples both PC and Mac. They looked good.
The javascript for the discsloure button examples had some poor documentation, so I update the two examples to:
Preview Link: FAQ
Preview Link: Image Description
Review checklist
Preview | Diff