User tests: Successful: Unsuccessful:
Pull Request resolves #48408.
Improves Cassiopeia navigation accessibility by making current-menu and hover underlines solid white and replacing the submenu keyboard-focus outline with a clearly visible solid outline, improving non-text contrast under WCAG 2.2 SC 1.4.11.
opacity: .2, giving about 1.82:1 against the dark header. It now uses opacity: 1 (blocks/_header.scss).1px dotted $gray-400 is replaced with 2px solid $gray-900 (vendor/metismenu/_metismenu.scss).npm run build:dev -- -n templates/site/cassiopeia or npm run build if applicable.Tab to focus a first-level submenu link, then a second-level submenu link.
Please select:
Documentation link for guide.joomla.org:
No documentation changes for guide.joomla.org needed
Pull Request link for manual.joomla.org
No documentation changes for manual.joomla.org needed
| Status | New | ⇒ | Pending |
| Category | ⇒ | NPM Change |
| Labels |
Added:
NPM Resource Changed
PR-6.2-dev
|
||
If that's the position used in the sample data then it should be fixed here as well
Good job we don't vote to decide anything then
@mbeganyi-a11y I'll look into fixing the topbar contrast issue and update the PR if I find a suitable solution. Thanks for pointing it out!
| Labels |
Added:
a11y
|
||
@Nikhil-sinhaa Nice work to isolate the focus indicators; we're close. The only issues remain in the footer position where the drop-down arrows and their focus indicators are now a very low contrast against the background. But we're making progress.
The screenshots below show the two focus indicators in the footer. The first depicts the troublesome focus and default state of the drop-down arrows. The second depicts the top-level focus which looks good.
Change (_footer.scss only)
I have tested this item ✅ successfully on 6c3a142
The latest changes fix the footer focus, although the arrows I get are solid triangles and not just chevrons, as your most recent screenshot shows. Great work here, @Nikhil-sinhaa; thanks so much for your contribution to accessibility.
I have tested this item ✅ successfully on 6c3a142
The latest changes fix the footer focus, although the arrows I get are solid triangles and not just chevrons, as your most recent screenshot shows. Great work here, @Nikhil-sinhaa; thanks so much for your contribution to accessibility.
Thanks for testing and for all the feedback, @mbeganyi-a11y! Glad the footer focus is working now.
This is a really good effort, @Nikhil-sinhaa and it solves for most use cases, however, unfortunately, not all. (But this isn't a fault of yours, necessarily!)
If the menu is placed in the "topbar" region, the darker focus indicator becomes nearly invisible. See the screenshot below showing the indicator in that place:
If there is some way to ensure that the focus indicator is visible in all circumstances, it would solve this problem. However, I'm of the belief that this is a design system requirement and will hopefully be addressed in the coming months. Unless there's some way to make a custom focus indicator for the "topbar" nav placement.
Irrespective of this, I think this PR does a good job of improving the focus indicator in many circumstances, so I'd say it's better than the current implementation and should be merged as an intermediary solution between the current implementation and an updated design system. What do others think?