ASU / asu-unity-stack

ASU Unity Design System
https://unity.web.asu.edu/
Other
11 stars 8 forks source link

UDS-1467: fix(header): fix accessibility bug in header #1152

Closed daemon-node-byte closed 1 year ago

daemon-node-byte commented 1 year ago

Description

Problem Detected a bug during the accessibility page scan on the Undergrad Admissions App

Extra: large empty space in mobile nav link dropdown

Container element is empty Some roles are designed to contain other roles. This element has this type of role, but it doesn't contain any required-owned elements.

Solution Prevent the empty element from writing to the DOM.

Extra: removed min-height property from element

Links

Checklist

Browsers

Images

Before removing min-height

Screenshot 2023-08-23 at 3 54 08 PM

After removing min-height

Screenshot 2023-08-23 at 3 54 52 PM

With nav links and removed min-height

Screenshot 2023-08-23 at 3 58 46 PM

scott-williams-az commented 1 year ago

Dave is correct, I was looking at the specs and the docs assume the open menu take over the screen. https://xd.adobe.com/view/56f6cb78-9af5-4b12-b4ce-ef319f71113f-03a5/screen/02335e90-65c1-4245-8fc7-71b70322c4ff/

Edited: Oops I missed the no Nav example, I think this should be OK (Sorry for the back and forth)

daemon-node-byte commented 1 year ago

Thank you @scott-williams-az and @davidornelas11 for review and approvals.