-
Notifications
You must be signed in to change notification settings - Fork 37
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
SideNavigation a11y fix #1496
SideNavigation a11y fix #1496
Conversation
…Twin/iTwinUI into xander/sidenavigation-a11y-fix
I'm still conflicted on the label for the "Toggle" was another idea I had, but it's kind of vague as to what exactly it's toggling. I could have it as "Toggle side menu," but then the question is... toggle what about the side menu, exactly? I could just leave it as "Expand side menu," and the user would likely assume that the button would also collapse the side menu if it's already expanded (a common design pattern), but again, I don't want to run the risk of being too vague. I have the label as "Expand or collapse side menu" for now because I'd rather run the risk of giving too much information at first than not enough. It's less work to have the full intent established before we whittle it down into something better, than to build something new from the ground up imo. |
On another note: even when the |
packages/itwinui-react/src/core/SideNavigation/SideNavigation.tsx
Outdated
Show resolved
Hide resolved
Co-authored-by: Mayank <9084735+mayank99@users.noreply.github.com>
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.
LGTM. i believe your concerns from above have been addressed now
Changes
#1148 - All SideNavigation examples are failing due to
button-name
Axe violations. In all examples it occurs in theexpand
button, and inSubmenuExample
it also occurs in thesettings
button within thesubmenu
. The examples and / or component will be modified to resolve these rule violations.Expand or collapse side menu
in the expand button inSideNavigation
.SubMenuExample
SubMenu Settings Button
through labelSubmenu settings
.Testing
Testing will be conducted through the Cypress web a11y script. Any breaking changes will be evaluated through unit testing & visual tests.
I am also reading the examples with NVDA to assess their screen reader compatibility.
Docs
N/A