- Issue created by @rkoller
- First commit to issue fork.
- 🇺🇸United States smustgrave
The block navigation twig believe was incorrect so fixed that.
- 🇩🇪Germany rkoller Nürnberg, Germany
yeah with your changes on the navigation twig if i reapply my changes locally that i've suggested on gitlab now also the navigation button is correctly announced with dimmed aka disabled.
- 🇩🇪Germany rkoller Nürnberg, Germany
at the moment the merge request still uses
aria-'disabled' => $noTips,
. i've just manually reapplied the changes i'Ve suggested on the MR in #7 and retested with your changes in9ccd9d09 - Fix blcok
by readding'aria-disabled' => 'true',
. with that it looks good. aria-disabled needs the true and isnt standalone like disabled. - 🇩🇪Germany rkoller Nürnberg, Germany
from my perspective it looks good now. for both navigation and toolbar the button is announced as a dimmed dialog popup button and the cursor changes to the disabled type on hover.
-
smustgrave →
committed 8a57fa70 on 2.0.x
Issue #3475981: Improve the "No tour" button
-
smustgrave →
committed 8a57fa70 on 2.0.x
Automatically closed - issue fixed for 2 weeks with no activity.