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.

Sign up for GitHub

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

Jump to bottom

[4.0] A11y - Default-Icon main menu without description #26656

Closed
chmst opened this issue Oct 19, 2019 · 1 comment
Closed

[4.0] A11y - Default-Icon main menu without description #26656

chmst opened this issue Oct 19, 2019 · 1 comment
Labels
No Code Attached Yet

Comments

@chmst
Copy link
Contributor

chmst commented Oct 19, 2019

Steps to reproduce the issue

The Main Menu (with home) has the default icon. This icon is not visible for screenreaders.
brianteeman added a commit to brianteeman/joomla-cms that referenced this issue Oct 19, 2019
@brianteeman
[4.0] Default Menu icon
40c9fc7 
Add aria hidden class to the icon and sr-only text to indicate that it is the default menu

pr for joomla#26656
@brianteeman brianteeman mentioned this issue Oct 19, 2019
Merged
@Quy Quy closed this as completed Oct 19, 2019
@Quy
Copy link
Contributor

Quy commented Oct 19, 2019

Please test PR #26730

This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/26656.

wilsonge pushed a commit that referenced this issue Oct 19, 2019
@brianteeman @wilsonge
[4.0] Default Menu icon (#26730)
abcb0f2 
Add aria hidden class to the icon and sr-only text to indicate that it is the default menu

pr for #26656
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
No Code Attached Yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@Quy @chmst @joomla-cms-bot