Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Main menu site admin] Org admin menu is hidden but still part of the DOM #6411

Closed
johanna-skylight opened this issue Aug 24, 2023 · 0 comments · Fixed by #6440
Closed

[Main menu site admin] Org admin menu is hidden but still part of the DOM #6411

johanna-skylight opened this issue Aug 24, 2023 · 0 comments · Fixed by #6440
Assignees
Labels
Accessibility Issues that affect SR's accessibility Growth & Engagement

Comments

@johanna-skylight
Copy link
Contributor

Area or Component

Main menu as site admin

Quick Summary

The menu list for the organization admin is still part of the DOM even though it is "hidden" for the sighted users.

How to Replicate

Check the DOM.

Accessibility Violation

Content is not a list but it is marked as such.

Action Requested

Add logic to not include the ul menu element in the DOM for site admins.

Screenshots (optional)

See how the DOM is still part of the page but is not visible because the menu items don't have text.

Screenshot 2023-08-24 at 10 41 49 AM
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Accessibility Issues that affect SR's accessibility Growth & Engagement
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants