Closed Bug 1473605 Opened Last year Closed Last year

[AccessFu] Don't group together labels and controls in traversal


(Core :: Disability Access APIs, enhancement)

Not set



Tracking Status
firefox63 --- fixed


(Reporter: eeejay, Assigned: eeejay)



(1 file, 1 obsolete file)

We did this originally for B2G Settings app. They had an interesting hack for restyling checkboxes. At the expense of adding more navigation items to the screen we should seperate the two. Because (a) This is what users expect and (b) because this grouping makes it hard to follow focus correctly. For example, if a labeled entry is focused, the label gets the virtual cursor, that is confusing.
Fixed some eslint errors.
Attachment #8990117 - Attachment is obsolete: true
Attachment #8990139 - Flags: review?(yzenevich)
Attachment #8990117 - Flags: review?(yzenevich)
Does this mean that for a check box, there'll be two "flick" stops: one for the label and one for the check box? I think this is going to cause efficiency/productivity problems. Certainly, VoiceOver seems to filter out most label elements altogether for this reason. While there are problems with that approach, I think the extra "flick" stops are worse.

This might be something to cover in a different bug; the way it's currently being done is obviously causing focus problems.
Comment on attachment 8990139 [details] [diff] [review]
Don't clump labels and controls together. It messes with focus events. r?yzen

Review of attachment 8990139 [details] [diff] [review]:

Thanks, I also do like the idea of possibly pre-fitlering labels for form controls, if I interpreted Jamies comment correctly.
Attachment #8990139 - Flags: review?(yzenevich) → review+
The problem with prefiltering labels is that there will be less explore by touch targets on the page and the user won't get a good idea of what is available. I guess for that case the ebt rule could include labels.. That would also be consistent with how mouse users can click on labels to toggle checkboxes but they are not in the tab order..

Anyway, it is worth revisiting this issue further down.
Pushed by
Don't clump labels and controls together. It messes with focus events. r=yzen
Keywords: checkin-needed
Closed: Last year
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
Assignee: nobody → eitan
You need to log in before you can comment on or make changes to this bug.