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)

x86
All
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla38

People

(Reporter: yzen, Assigned: yzen)

Details

Attachments

(1 file, 2 obsolete files)

Some of the statusbar icons, elements in Gaia need to be accessible. I was thinking of using role="status" for that.
Attached patch 1000295 patch. (obsolete) — Splinter Review
Attachment #8411130 - Flags: review?(eitan)
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..
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.
Attachment #8411130 - Flags: review?(eitan)
Assignee: yzenevich → nobody
Attached patch 1000295 patch v2 (obsolete) — Splinter Review
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 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)
Attached patch 1000295 patch v3Splinter Review
Added a check for flat subtree and single liniage for STATUSBAR traversal.
Attachment #8553930 - Attachment is obsolete: true
Attachment #8553976 - Flags: review?(eitan)
Comment on attachment 8553976 [details] [diff] [review]
1000295 patch v3

Love it!
Attachment #8553976 - Flags: review?(eitan) → review+
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.

Attachment

General

Created:
Updated:
Size: