1160469 - Cloud tabs header should be single element and heading for accessibility

Status: RESOLVED FIXED

(Firefox for iOS :: Home screen, defect)

Description

[Boris Dušek]

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_10_3) AppleWebKit/600.5.17 (KHTML, like Gecko) Version/8.0.5 Safari/600.5.17

Steps to reproduce:

1. Tap location label
2. Tap "Cloud tabs" button
3. Turn on VoiceOver
4. Put VoiceOver cursor on a header item (e.g. "Boris' Firefox on <device>")


Actual results:

VoiceOver read just the main title of the cell.
When I moved VoiceOver cursor to the next element, the next element was "last synced : ..."
VoiceOver did not announce that the cell is a heading, so quick navigation to next heading by VoiceOver rotor was not possible


Expected results:

VoiceOver would read the whole contents of the cell at once.
VoiceOver would announce the cell is a heading and allow quick navigation directly between these headings using VoiceOver rotor.

Comment 1

[Boris Dušek]

Attachment: Screenshot: Unable to assign myself to bug

Please how do I assign myself to this bug? (see the screenshot, situation after clicking the value right to the "Assigned To:" field)

Comment 2

[atrain]

It's all yours. No need to ask :)

Comment 3

[Boris Dušek]

To not have to file a separate bug: Computer vs. Mobile device should also be announced (cell image)

Comment 4

[Boris Dušek]

Attachment: Pull request

Comment 5

[Boris Dušek]

(In reply to atrain from comment #2)
> It's all yours. No need to ask :)

I was wondering how one should follow the "Assign yourself" instruction in the Contributor guidelines (README.md on GitHub) ;-)

Comment 6

[Stefan Arentz | :st3fan | ⏰ EST | he/him]

I'm not sure why you can't assign bugs to yourself. Let me investigate.

The patch looks great. I'll merge it soon.

Comment 7

[Boris Dušek]

Thanks Stefan but no need to investigate that, I applied for the "editbugs" privilege as suggested by Brian and Nick, approval is pending, until then I can't assign bugs to myself.

Comment 8

[Stefan Arentz | :st3fan | ⏰ EST | he/him]

Added a quick comment to the patch about a println() that was left in. Other than that it should be good to go.

Comment 9

[Stefan Arentz | :st3fan | ⏰ EST | he/him]

Merged. Thank you for the contribution.