As pointed out by @Volker_E, the ARIA tags we have previously added are not complete enough, for reference:
https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/AdvancedSearch/+/444630/
Specifically the remark about aria-expanded:
This will only work in connection with aria-controls as the button.$elment isn't a parent.
As part of this ticket, aria-controls should be added so that screen readers know what to do with the newly added aria-expanded tag.
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| Add aria-controls attribute | mediawiki/extensions/AdvancedSearch | master | +5 -1 |
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | None | T198599 Content checklist for making AdvancedSearch a default feature | |||
| Resolved | None | T181488 Improve screenreader compatibility in AdvancedSearch | |||
| Resolved | Lea_WMDE | T199804 Further improve screenreader compatibility in AdvancedSearch by adding aria-controls tags |
Change 447619 had a related patch set uploaded (by Tonina Zhelyazkova; owner: Tonina Zhelyazkova):
[mediawiki/extensions/AdvancedSearch@master] Add aria-controls attribute
Change 447619 merged by jenkins-bot:
[mediawiki/extensions/AdvancedSearch@master] Add aria-controls attribute
Thanks @Tonina_Zhelyazkova_WMDE – yes, this is a weakly supported attribute. But it follows the standard docs and in such case we should make use of the standard in cases where other (currently probably wider supported) solutions are too time-costly. Or as Léonie Watson puts it at the end of the article mentioned by you:
Despite the fact support amongst browsers and screen readers could be better, the aria-controls attribute is still worth using. Creating relationships within the DOM is certainly more robust, but it isn’t always practical or achievable. The presence of aria-controls won’t damage the User Experience (UX) for people using UA that don’t support it, but it will enormously improve the UX for those people whose browser/screen reader combinations do.
And I don't see a time-sensitive solution right now, as it would probably result in a major refactoring and some parent child construct, that leads to other issues. See also Heydon's –as second critical author on aria-controls support above– example on progressive collapsibles using exactly the structure we now have.