search

LIT-EIA / adapt-esdc-accessibilityfixes

Extension to load and run a .js file when a course loads
GNU General Public License v3.0
5 stars 2 forks source link

Menu screen - button issues #66

Closed LIT-EIA-OWNER closed 1 year ago

LIT-EIA-OWNER commented 3 years ago

From Melissa:

Headers are not related to the buttons.

If you menu has:

The intro page [ View ]

Screen reader should read:

The intro page [ View The intro page ]

Could we code a way to say that for screen readers without needing to show on the buttons?

GGD4V commented 3 years ago

Should fetch menu title and populate button aria label with it to make the association that this is what is interacted with.

LIT-EIA-OWNER commented 3 years ago

Added the code for it but the only problem is that the code adds "visited" to the button once you've seen it.

Need to check the content and adjust the aria label if needed. DT is able to change the word "visited".

Commented out the code for now, it's under menufixes();

ftourangeau commented 1 year ago

This has been added to Adapt Best Practices Guide and has been adopted by most DTs. Closing for now