Support for ARIA 1.1 combo boxes in Firefox and Chrome.#10046
Support for ARIA 1.1 combo boxes in Firefox and Chrome.#10046michaelDCurran merged 1 commit intonvaccess:masterfrom
Conversation
|
PR introduces Flake8 errors 😲 See test results for Failed build of commit 52fcc10938 |
94df760 to
d872d91
Compare
|
PR introduces Flake8 errors 😲 See test results for Failed build of commit 939e177bc6 |
1. If the first child of a combo box is a text box, render that text box in browse mode. 2. Previously, we treated anything inside a combo box as being outside a browse mode document (nvaccess#8687). Don't do this for text boxes. Otherwise, you can't switch back to browse mode when the text box has focus.
|
Some things I observed.
|
Do you mean pressing alt+downArrow from browse mode or focus mode? Those combo boxes seem to have weird behaviour with alt+downArrow. That said, downArrow is usually enough to open an autocomplete if appropriate.
When you say "opening it", do you mean alt+downArrow? That behaves strangely for me with these examples; it seems to open, but when you release alt, it collapses.
The implementation has the label on both the text box and the list, so yes, this is related to the implementation. If that's common, we could potentially filter this out, but I think that's out of scope for this PR. |
In both browse and focus mode.
Exact sstr are:
|
The fact that this doesn't work in focus mode is because the test cases don't support it. I don't know of any ARIA 1.1 combo boxes that do. They tend to expand when you start typing or when you press down arrow. My feeling is that we shouldn't do anything for browse mode here unless we can find a combo box that actually supports this. That can be done as a follow-up. Does that work for you?
For me, the list gets focus when I press down arrow here. However, I'm using Firefox nightly. I suspect you're being bitten by https://bugzilla.mozilla.org/show_bug.cgi?id=1571327 , which I fixed a month ago (targeting Firefox 70). |
Sure!
Ah, in that case, let's assume it will be fixed in FF70 for me as well. |
Link to issue number:
Fixes #9616 .
Summary of the issue:
ARIA 1.1 combo boxes (where the combo box contains a text box and a pop-up such as a list box) often weren't rendered in browse mode at all, making them effectively non-existent. Even if they were, you couldn't read the value or press enter to interact with the text box.
Description of how this pull request fixes the issue:
Testing performed:
Known issues with pull request:
This assumes that a text box will always be the first child of the combo box. All the examples (including the official spec examples) do this and this is what I've seen in the wild. We could change this to scan all children, but I'm reluctant to do this unless we have to.
Change log entry:
Bug Fixes:
- ARIA 1.1 combo boxes are now supported in Mozilla Firefox and Google Chrome. (#9616)