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

Component - Quick navigation - Best practices? #25

Open LIT-EIA-OWNER opened 3 years ago

LIT-EIA-OWNER commented 3 years ago

• Uses Tooltips that when hover displays text that show information that a keyboard user wouldn’t get. • Has a next page button even though there isn’t a next page. It’s locked, I would assume some users would think there’s another page and wonder why they can’t unlock it.

LIT-EIA-OWNER commented 3 years ago

Code was added to the JS to remove the tooltips but it doesn't always work - tooltips not removed if using trickle since the navigation is hidden in the dom?

Technically JS fixes are not needed for these but nice to have

GGD4V commented 3 years ago

Tooltips are now fixed. We could target locked button inside quicknav onload and make them display none. Most assistive technologies would ignore.

GGD4V commented 3 years ago

Should be all fixed now.

LIT-EIA-OWNER commented 3 years ago

Disabled the code that was removing disabled buttons from the dom as it was also removing buttons that were disabled for other reasons!

I think we should just show DTs how to remove it themselves?