Settings Editor: Add aria labels for input elements Fixes: #54836#55543
Settings Editor: Add aria labels for input elements Fixes: #54836#55543cleidigh merged 1 commit intomicrosoft:masterfrom
Conversation
|
Thanks! Yeah I'm still asking around, I think we are on our own to figure this out though... I think using treegrid or grid would be fine if you know how to do that. |
|
the ambiguities are really frustrating we will have to change the widgets or implement something separately to use any of the grid what's more important is to get the navigation in good order, I think I have seen some other errors |
|
I just got another tip about someone who can possibly help... I'll keep you in the loop. |
|
To give you a quick update on this - I am thinking about #54039 and prototyping an alternative. Since this impacts the accessibility story quite a bit, we can wait on solving this. I do have an expert to ask and I will talk to them this week. |
|
sounds good, when you know how you want to do the structure , I can help the aria stuff, I know there's a lot of issues to do in August, I'll try to help out where I am useful |
|
I've checked in the change for #54039, so now there is no list selection. I think we want to ideally read the title, description, and value when the control is focused. I think getting rid of "role=tree" will help and I'm ok with that. Do you know how to do that, or should we put together a simple example that I can take to someone to ask for help? It sounds like |
|
@roblourens Are you using a table for the layout? That would give us a couple options but it's not an absolute necessity. Are we past endgame when you commits are okay? |
|
I am still using the tree control, that hasn't changed. But I disable actual row selection and focus. Only controls and links are focusable. And yes, commits to master are ok. |
|
@roblourens |
|
Yes, but we can remove that aria role. If we have to change the tree widget to avoid adding it in this case, we can do that. |
|
one step ahead of you, I tried that when working on the labels earlier, we have to change the other management attributes as well. One thought I had is to use and modify the list widget to use |
Not sure what you mean by that exactly, but I'd rather just change whatever attributes are needed in the tree widget to get the result we need. |
|
Looking really good ! treegrid and grid are aria roles that both cleanly support interactive elements within them. |
|
I see what you're saying. Since the tree and row elements themselves are no longer selectable at all, I wonder whether those attributes really come into play here? Since the screenreader will have nothing to focus that reads "row 5 of 10" or whatever. I have no clue. |
|
there are actually issues at multiple points. I did quite a bit of research and it became clear or unclear about how or if interactive form elements ARIA treegrid and grid unambiguously supports form elements, although I have seen some labeling issues with these as well. we can just some experiments to see what the state of things is. |
|
I understand. But I wonder whether we can just remove the roles, instead of switching to a different type of role like treegrid or grid. If the screenreader just sees some form input elements, and doesn't see tree rows at all, that might be a better experience. |
|
I'm hoping we can do just that. I got some experiments done yesterday |
|
We plan to upgrade to 2.0.7 this month which is just a minor upgrade from the current version. So there won't be a big behavior change in chrome/screenreaders. |
|
YeeHah ! I think I finally found the magic combo ! I think I can post to PR later so you can look at it. I assume you utilize NVDA Do you know about the screen viewer under tools? this allows you to see exactly what is being read |
|
Great! Can't wait to see it. |
|
okay don't forget it's wip this appears to address the problem of labels and input elements if they fall under a tree (and no one is there to hear it) I changed the top level role=tree to form AND ALSO removed the role attribute from the individual element Current general form: Category Item [Modified] edit Some notes and decisions:
Separate suggested: I thought it reset to default per item would be a good addition PR |
@roblourens
This addresses labels for input elements as best as possible for elements within a tree.
My research suggests interactive elements within a tree are not formally supported
but at least NVDA does its best.
I believe will have to revisit this with more information, perhaps we need to use treegrid or
grid both of which explicitly support interactive elements. These might need to be added as options
for list and tree widgets
I will also work at adding support for the links within descriptions but I will wait till after this milestone
No response yet on my question on this for NVDA