Missing labelled-by, label-for relationship with figures and their captions
Categories
(Core :: Disability Access APIs, defect)
Tracking
()
People
(Reporter: jdiggs, Assigned: Jamie)
References
(Blocks 2 open bugs, Regression)
Details
(Keywords: regression, Whiteboard: [ctw-23h2])
Attachments
(2 files)
(deleted),
text/html
|
Details | |
(deleted),
text/x-phabricator-request
|
diannaS
:
approval-mozilla-beta+
diannaS
:
approval-mozilla-esr115+
|
Details |
Steps to reproduce:
- Load the attached test case.
- Use an accessibility inspector (e.g. Accerciser) to view the accessible relations of figure and the figcaption elements.
Expected results: There would be a label-for relation on the figcaption pointing to the figure and a labelled-by relation on the figure pointing to the figcaption. See https://w3c.github.io/html-aam/#el-figcaption and https://w3c.github.io/html-aam/#el-figure.
Actual results: These relationships are missing.
Impact: Orca double-speaks the caption text.
Explanation: When caret navigation is used to move into a figure with a figcaption, Orca updates its location to the start of the figcaption's text and presents that text. However, because the user had moved from outside of the figure into it, Orca first presents the new ancestry which includes the figure, whose name is the same as the caption text. In order to not double-speak the text, Orca first checks to see if the name of the figure came from the caption text (i.e. as opposed to aria-label). And the way it performs this check is by looking for the labelled-by/label-for relation.
Assignee | ||
Comment 1•1 year ago
|
||
Ah. That totally makes sense. Sorry; I didn't think of relations when this was reported to us on Matrix, but I should have.
We do actually support these relations in LocalAccessible, but we aren't including them in the cache for RemoteAccessible. That means this regressed in Firefox 113.
Assignee | ||
Updated•1 year ago
|
Assignee | ||
Updated•1 year ago
|
Assignee | ||
Comment 2•1 year ago
|
||
Comment 4•1 year ago
|
||
bugherder |
Comment 5•1 year ago
|
||
The patch landed in nightly and beta is affected.
:Jamie, is this bug important enough to require an uplift?
- If yes, please nominate the patch for beta approval.
- If no, please set
status-firefox116
towontfix
.
For more information, please visit BugBot documentation.
Assignee | ||
Comment 6•1 year ago
|
||
Comment on attachment 9343391 [details]
Bug 1842160: Support the LABELLED_BY/LABEL_FOR relations on HTML figure/figcaption for RemoteAccessible.
Beta/Release Uplift Approval Request
- User impact if declined: Figure captions will be spoken twice by some screen readers, which is annoying for users.
- Is this code covered by automated tests?: Yes
- Has the fix been verified in Nightly?: Yes
- Needs manual test from QE?: No
- If yes, steps to reproduce:
- List of other uplifts needed: None
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): Isolated change which only affects a11y label relations for figures/figcaptions. Covered by automated tests.
- String changes made/needed:
- Is Android affected?: No
Updated•1 year ago
|
Comment 7•1 year ago
|
||
Comment on attachment 9343391 [details]
Bug 1842160: Support the LABELLED_BY/LABEL_FOR relations on HTML figure/figcaption for RemoteAccessible.
Approved for 116.0b7
Updated•1 year ago
|
Comment 9•1 year ago
|
||
Comment on attachment 9343391 [details]
Bug 1842160: Support the LABELLED_BY/LABEL_FOR relations on HTML figure/figcaption for RemoteAccessible.
Approved for 115.1esr
Comment 10•1 year ago
|
||
uplift |
Updated•1 year ago
|
Updated•1 year ago
|
Comment 11•1 year ago
|
||
Reproduced the issue on Firefox 117.0a1 (2023-07-07) on Ubuntu 22.04 by following the info provided in Comment 0 where Orca was reading twice the caption text from the testcase provided.
The issue is no longer reproducible on Firefox 115.1.0esr, Firefox 116.0 and Firefox 117.0a1 (2023-07-24) on the same system.
Description
•