The user of the screen reader gets access to almost all implemented operating possibilities, but they are not understandable.
The "Menu selection" label is not announced by the screen reader.
A widget to select / expand menu tree is not accessible because
a. Does not have an accessible name (headings or label)
b. “Select” & “Expand” labels are not announced by the screen reader
c. “All”, “None” options are links instead of buttons.
d. “All link” & “None link” messages announced by the screen reader are incomprehensible to the blind person.
The names of the menus in which the user is to select are not announced by the screen reader.
Aliases next to menu item names are unnecessary. They do not provide new information, but they cause information noise, confusion, etc.
Accessibility errors in the context menu:
a. Nonconsecutive level heading used. The menu contains an <h5>
tag directly following an <h3>
(“Sub-items”). In order to maintain a consistent outline of the page for assistive technologies, reduce the gap in the heading level by upgrading this tag to an <h3>
or <h2>
or see paragraph b. below.
b. The names of items in the context menu are unclear. They should rather be: “Select sub-items”, “Deselect sub-items”, “Expand sub-items”, “Collapse subitem”. In this case the heading H5 (Sub-items) is not needed.
c. When you select or deselect from the context menu, all items in the level are always selected, not just the selected item. Blind screen reader user does not see or hear it.
d. Icons next to the names of items in the context menu are unnecessary.
tabindex="-1"
attribute.Labels |
Added:
?
|
Title |
|
@dgrammatiko : Do you want to improve only the switch?
Your code seems to be correct, but I think you first need to rebuild the code responsible for selecting a menu item to define the switch's operations well.
@zwiastunsw ok define the HTML here and I'll do the coding part
Status | New | ⇒ | Discussion |
Category | ⇒ | Accessibility com_menus |
Labels |
Added:
J4 Issue
|
Title |
|
Labels |
Added:
?
|
Category | Accessibility com_menus | ⇒ | com_menus |
Title |
|
Status | Discussion | ⇒ | Confirmed |
Build | master | ⇒ | 4.0-dev |
Labels |
Removed:
?
|
@chmst is this something you can look into for Accessibility?
Category | com_menus | ⇒ | Accessibility com_menus |
simply not true but keep repeating it and magically someone will read your mind to decide what the
problem is and how it can be fixed to satisfy you. until then i guess this will be left as it is because no one will waste their time - I know I wont be making the same mistake again. Writing code and just getting a fail without any explanation or even suggestion taught me my lesson.
If you have nothing to say, you better not say anything. You could finally learn that.
Words are cheap. Actions have real value.
Alright, enough. This can be worked we just need someone with the knowledge/time to do so.
I'll re-activate the PRs tomorrow and see what happens. I am highly interested in a accessible backend.
issue 2 was not solved #28726 (comment)
Labels |
Added:
No Code Attached Yet
bug
Removed: ? |
@zwiastunsw if you can come up with an HTML markup I'll do the js part...
PS nevermind here is the correct HTML markup:
Goes to
administrator/components/com_modules/tmpl/module/edit_assignment.php
lines 40-52