Closed
Bug 1000295
Opened 10 years ago
Closed 9 years ago
[AccessFu] Enable traversing to elements with role="status".
Categories
(Core :: Disability Access APIs, defect)
Tracking
()
RESOLVED
FIXED
mozilla38
People
(Reporter: yzen, Assigned: yzen)
Details
Attachments
(1 file, 2 obsolete files)
6.06 KB,
patch
|
eeejay
:
review+
|
Details | Diff | Splinter Review |
Some of the statusbar icons, elements in Gaia need to be accessible. I was thinking of using role="status" for that.
Assignee | ||
Comment 1•10 years ago
|
||
Attachment #8411130 -
Flags: review?(eitan)
Comment 2•10 years ago
|
||
How are you planing to use the status role? For each element on the bar? for the entire bar? I ask because I think the status role is more accurately a region or container, so I am not sure if a MATCH | IGNORE_SUBTREE result is correct during traversal. I could be convinced otherwise..
Comment 3•10 years ago
|
||
Role "status" is problematic for several reasons: http://www.w3.org/TR/wai-aria/roles#status 1. It is indeed a container. 2. It has an implicit live region association, meaning anything contained within will speak automatically as it updates. The clock every minute, for example. Reading articles or web pages would then always be interrupted, or information appended, as soon as a chance arises. The status bar itself should instead be a container with role "group": http://www.w3.org/TR/wai-aria/roles#group, and an aria-label of "Status bar". Elements within should be swipable of course, the status bar container itself not. it will be announced within the hierarchy logic.
Updated•10 years ago
|
Attachment #8411130 -
Flags: review?(eitan)
Assignee | ||
Updated•10 years ago
|
Assignee: yzenevich → nobody
Assignee | ||
Comment 4•9 years ago
|
||
Updated the patch with better test example. Need it for an email app in Gaia, where we have a status in the footer that updates when we sync the folder with the time of last sync.
Attachment #8411130 -
Attachment is obsolete: true
Attachment #8553930 -
Flags: review?(eitan)
Comment 5•9 years ago
|
||
Comment on attachment 8553930 [details] [diff] [review] 1000295 patch v2 Review of attachment 8553930 [details] [diff] [review]: ----------------------------------------------------------------- eeejay> yzen: wouldn't it be a single traversal node anyway because isFlatSubtree should be true? 13:10 <•yzen> eeejay we only check that for specific roles 13:10 <eeejay> ah 13:11 <eeejay> yzen: so maybe that could be added for status role 13:11 <•yzen> eeejay ya that's a good point
Attachment #8553930 -
Flags: review?(eitan)
Assignee | ||
Comment 6•9 years ago
|
||
Added a check for flat subtree and single liniage for STATUSBAR traversal.
Attachment #8553930 -
Attachment is obsolete: true
Attachment #8553976 -
Flags: review?(eitan)
Comment 7•9 years ago
|
||
Comment on attachment 8553976 [details] [diff] [review] 1000295 patch v3 Love it!
Attachment #8553976 -
Flags: review?(eitan) → review+
Assignee | ||
Comment 8•9 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/0a6c28d7c304
Comment 9•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/0a6c28d7c304
Assignee: nobody → yzenevich
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla38
You need to log in
before you can comment on or make changes to this bug.
Description
•