-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
[4.0][a11y] Mark the link group in the Quickicon module as a list #20995
Comments
Sorry but I haven't got a clue what you mean. You really are going to have to provide more information |
Currently, the module contains a group of 6 links. But this is not a list of links. Each is covered by the div tag. |
and why should it be a list of links? If you want to say something is wrong and expect someone to fix it then you really need to explain what is wrong and why |
We have here a set of navigation links. It doesn't matter that they look like buttons. This is a disordered list. And so it should be marked. And since this is a navigation shortcut set, you should assign a role=navigation to this set. |
so how are these links any different to the links in the latest articles module. what is it that makes these links to be navigation links and the others are not. |
Of course, not every set of links should be marked as a navigation element, but every important navigation element - yes. Bruce Lawson wrote about that in his book "Introduction HTML.5". In one of his presentations, however, he stressed that these must be only important sets of links. Because if everything is highlighted, nothing is highlighted. |
the last two (update) and are more than just navigation |
I agree, but they are also navigation. |
closed - please test #22614 |
Quick icon module include a set of navigation links.
A group of links in the Quick icon module is not marked as a list.
Links are not part of the list.
In addition, it shall be marked as a navigation element (nav).
See: Info and Relationships (Level A)
The text was updated successfully, but these errors were encountered: