Closed
Bug 1660400
Opened 4 years ago
Closed 4 years ago
Sheet count should be included in modal a11y description
Categories
(Toolkit :: Printing, enhancement, P2)
Toolkit
Printing
Tracking
()
People
(Reporter: Jamie, Assigned: Jamie)
References
Details
(Keywords: access, Whiteboard: [print2020_v81])
Attachments
(1 file)
47 bytes,
text/x-phabricator-request
|
RyanVM
:
approval-mozilla-beta+
|
Details | Review |
STR (with the NVDA screen reader):
- Open this URL:
data:text/html,test
- Press control+p to open the Print modal.
- Expected: Before speaking the destination, NVDA should speak the sheet count.
- Actual: The sheet count isn't reported.
This can be fixed by adding aria-describedby="sheet-count" to the body element for the print page.
Updated•4 years ago
|
Severity: -- → S4
Priority: -- → P2
Whiteboard: [access-s4] → [access-s4][print2020_v81]
Assignee | ||
Comment 1•4 years ago
|
||
Updated•4 years ago
|
Assignee: nobody → jteh
Status: NEW → ASSIGNED
Updated•4 years ago
|
Attachment #9171600 -
Attachment description: Bug 1660400: Include sheet count in Print modal a11y description. → Bug 1660400: Include sheet count in Print modal a11y description and make it a live region.
Pushed by jteh@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/ebfa542ac69a Include sheet count in Print modal a11y description and make it a live region. r=mstriemer
Comment 3•4 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
status-firefox82:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 82 Branch
Assignee | ||
Comment 5•4 years ago
|
||
Comment on attachment 9171600 [details]
Bug 1660400: Include sheet count in Print modal a11y description and make it a live region.
Beta/Release Uplift Approval Request
- User impact if declined: Screen readers will find it difficult or impossible to access the sheet count in the new print modal.
- Is this code covered by automated tests?: No
- 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): Simply adds (well understood) attributes that are picked up by the accessibility engine.
- String changes made/needed: None.
Flags: needinfo?(jteh)
Attachment #9171600 -
Flags: approval-mozilla-beta?
Comment 6•4 years ago
|
||
Comment on attachment 9171600 [details]
Bug 1660400: Include sheet count in Print modal a11y description and make it a live region.
Approved for 81.0b5.
Attachment #9171600 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Comment 7•4 years ago
|
||
bugherder uplift |
status-firefox81:
--- → fixed
Updated•11 months ago
|
Accessibility Severity: --- → s4
Whiteboard: [access-s4][print2020_v81] → [print2020_v81]
You need to log in
before you can comment on or make changes to this bug.
Description
•