CU-CommunityApps / cwd_base

A lightweight Drupal 8+ theme, implementing Cornell Branding and standard CWD components.
1 stars 2 forks source link

Javascript fixes for dropdown menus hover + escape key functionality #62

Closed judaw closed 1 year ago

judaw commented 1 year ago

Recs from Alison:

Question from Alison:

alisonjo315 commented 1 year ago

Maybe someone could put this branch (or a combination of this branch and #61) on a cddemo multidev, so it can be reviewed IRL?

(Cross-posting on #61)

alisonjo315 commented 1 year ago

TL;DR: Is there a chance the HR drop-down menus were built / evaluated only with Voiceover, not NVDA...?

UPDATE 2: Annie figured out that you have to switch to "forms mode" in NVDA, in order to use the drop-down main menu. Not that that's okay, I'm just sharing our findings.

(superseded by "update 2") UPDATE 1: I ran into the same issues on LASSP (the issues I describe below, not being able to get to the drop-down children), so I suppose it's probably a preexisting condition, so to speak?? https://www.lassp.cornell.edu/


I apologize if I'm off-base here or out of scope or whatever, I'm mediocre-at-best at using NVDA and my struggle could absolutely be me not the HR site -- but anyway:

While reviewing Engineering, I was trying to figure out how a main menu with drop-downs should behave, and I remembered there was a lot of accessibility work done on the HR menu, so I tried out the HR main menu, navigating with a screen reader and keyboard -- specifically with NVDA, because I'm on Windows.

FWIW, when I'm not using NVDA, when I'm just navigating the HR site via keyboard, I can indeed use "down arrow" to enter the drop-down sub-menus of the main menu.


One other thing: The "current page" is visually highlighted, but there's no "aria-current" attribute. I know this is a separate issue, but I didn't want to bother creating a new issue until someone says "yeah that's a problem."

ama39 commented 1 year ago

This is a duplicate of the same request in the cwd_framework repo. Fixes for these issues were covered as part of the major main nav work in the 3.0.1 update (megamenu, hover changes, etc...), so we can close this request. Thank you!