Change about window to be an accessible dialog and put all relevant static info into its acc description

VERIFIED FIXED in Firefox 10

Status

()

Firefox
Disability Access
VERIFIED FIXED
6 years ago
6 years ago

People

(Reporter: Jamie, Assigned: MarcoZ)

Tracking

({regression})

Trunk
Firefox 10
x86_64
Windows 7
regression
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

6 years ago
The details box containing most of the text in the About window (e.g. "Nightly is designed by Mozilla, a global community working together to make the Internet better. ...") has an accessible role of unknown. It used to (and should) have a role of property page. This is a vbox with an id of "detailsBox".

Unfortunately, I'm not sure when this broke. It is correct in an old Firefox 5pre build I have here (Mozilla/5.0 (Windows NT 6.1; WOW64; rv:5.0a2) Gecko/20110517 Firefox/5.0a2), but not in 10pre.
(Assignee)

Comment 1

6 years ago
Taking this.
Assignee: nobody → marco.zehe
Status: NEW → ASSIGNED
Summary: Details box in About window has accessible role of unknown instead of property page → Change about window to be an accessible dialog and put all relevant static info into its acc description
(Assignee)

Comment 2

6 years ago
Created attachment 566810 [details] [diff] [review]
Patch

This patch tricks accessibility APIs into thinking that this window is actually a dialog. Aside from the update stuff, it really is anyway. The relevant information the user wants to hear when initially entering the window is all shoved into the acc description of the dialog accessible itself, so different chunks no longer have to be discovered by tabbing around the dialog if the user just wants to hear the information.
Attachment #566810 - Flags: review?(gavin.sharp)
(Assignee)

Updated

6 years ago
Assignee: marco.zehe → nobody
Component: Disability Access APIs → Disability Access
Product: Core → Firefox
QA Contact: accessibility-apis → disability.access

Comment 3

6 years ago
(In reply to Marco Zehe (:MarcoZ) from comment #2)
> Created attachment 566810 [details] [diff] [review] [diff] [details] [review]
> Patch
> 
> so different chunks no longer have to be discovered by tabbing
> around the dialog if the user just wants to hear the information.

Marco, about dialog contains some information like up-to-date status that is changed after you open the dialog or it contains different links. Is it still navigable content?
(Assignee)

Comment 4

6 years ago
(In reply to alexander surkov from comment #3)
> Marco, about dialog contains some information like up-to-date status that is
> changed after you open the dialog or it contains different links. Is it
> still navigable content?

Yes, this patch just changes some ARIA magic around that has no impact on keyboard navigation.
Attachment #566810 - Flags: review?(gavin.sharp) → review+
Inbound: https://hg.mozilla.org/integration/mozilla-inbound/rev/159a1ce7cabc
Assignee: nobody → marco.zehe
Target Milestone: --- → Firefox 10
https://hg.mozilla.org/mozilla-central/rev/159a1ce7cabc
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
(Assignee)

Comment 7

6 years ago
Verified fixed in Mozilla/5.0 (Windows NT 6.1; WOW64; rv:10.0a1) Gecko/20111023 Firefox/10.0a1
Status: RESOLVED → VERIFIED
(Reporter)

Comment 8

6 years ago
I assume this change was Firefox specific, as it still exists in Thunderbird (tested with Mozilla/5.0 (Windows NT 6.1; WOW64; rv:10.0a1) Gecko/20111024 Thunderbird/10.0a1). Is this assumption correct (in which case I'll file another bug against TB) or is something odd happening here?
(Reporter)

Comment 9

6 years ago
I mean the bug still exists in Thunderbird; i.e. it isn't fixed. :)
(Assignee)

Comment 10

6 years ago
Yes, Thunderbird uses its own version of the dialog, so this is product specific.
(Assignee)

Comment 11

6 years ago
You can just say "port the accessibility changes from bug 694266" to Thunderbird in that new bug.
You need to log in before you can comment on or make changes to this bug.