Add-ons window additional screen reader issues

RESOLVED FIXED in Firefox 2

Status

()

Firefox
Disability Access
RESOLVED FIXED
12 years ago
12 years ago

People

(Reporter: Wayne DeAngelo, Assigned: Mark Pilgrim (inactive))

Tracking

({access, fixed1.8.1, sec508})

2.0 Branch
Firefox 2
x86
Windows XP
access, fixed1.8.1, sec508
Points:
---
Bug Flags:
blocking-firefox2 +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

12 years ago
Steps to reproduce:

- Run Firefox 2.0a2 nightly build 20060810 and Microsoft Inspect32 (or WindowEyes 5.5).
- Select Tools > Add-ons.  Extensions pane appears with first extension
selected.
- WindowEyes speaks "Add-ons" (window title), 
- Press down arrow.  Second extension is selected.
- WindowEyes speaks "Custom control". tab over to either disable and press enter.
- You will notice that WE does not speak the statement with the window "This add-on will be disabled when Bon Echo is restarted.
(Reporter)

Updated

12 years ago
Flags: blocking-firefox2?
(Reporter)

Comment 1

12 years ago
NOTE: This reported problem is seen after installing the fix for bug 338048

Comment 2

12 years ago
Sounds like 2 problems. Window-Eyes should say "custom control" and it should speak the entire list item's text.

What's the accessible name/role we're exposing?
Duplicate of bug 335645?

Comment 4

12 years ago
Note a dupe, and not fixed. This is the 2nd set of problems seen after bug 338048 was checked in.
Keywords: fixed1.8.1
Summary: Add-ons window not accessible to screenreaders → Add-ons window additional screen reader issues

Updated

12 years ago
Assignee: nobody → pilgrim
Flags: blocking-firefox2? → blocking-firefox2+
Target Milestone: --- → Firefox 2
(Assignee)

Comment 5

12 years ago
Created attachment 233904 [details] [diff] [review]
get operation text out of anonymous content nestled within hbox

Everything that needs to be said is in the comments in the patch.
Attachment #233904 - Flags: review?(bugs.mano)
Comment on attachment 233904 [details] [diff] [review]
get operation text out of anonymous content nestled within hbox

>Index: toolkit/mozapps/extensions/content/extensions.xml
>===================================================================
>RCS file: /cvsroot/mozilla/toolkit/mozapps/extensions/content/extensions.xml,v
>retrieving revision 1.17.2.14
>diff -u -8 -p -r1.17.2.14 extensions.xml
>--- toolkit/mozapps/extensions/content/extensions.xml	14 Aug 2006 17:49:21 -0000	1.17.2.14
>+++ toolkit/mozapps/extensions/content/extensions.xml	16 Aug 2006 01:58:06 -0000
>@@ -173,17 +173,17 @@
>   <binding id="addon-selected" extends="chrome://mozapps/content/extensions/extensions.xml#addon-base">
>     <content>
>       <xul:hbox flex="1">
>         <xul:vbox class="addon-icon" xbl:inherits="iconURL"/>
>         <xul:vbox flex="1" class="addonTextBox">
>           <xul:hbox anonid="addonNameVersion" class="addon-name-version" xbl:inherits="name, version"/>
>           <xul:label anonid="addonDescriptionWrap" class="descriptionWrap" xbl:inherits="xbl:text=description"/>
>           <xul:vbox anonid="addonSelectedStatusMsgs" class="selectedStatusMsgs">
>-            <xul:hbox class="addon-optype attention" align="center"/>
>+            <xul:hbox anonid="addonOpType" class="addon-optype attention" align="center"/>
>             <xul:hbox flex="1" class="updateAvailableBox attention">
>               <xul:label xbl:inherits="value=updateAvailableMsg" crop="end"/>
>             </xul:hbox>
>             <xul:hbox flex="1" class="incompatibleBox attention">
>               <xul:label anonid="incompatibleLabel" crop="end"/>
>             </xul:hbox>
>             <xul:hbox flex="1" class="needsDependenciesBox attention">
>               <xul:label value="&needsDependencies.label;" crop="end"/>
>@@ -261,28 +261,53 @@
>       <field name="_descriptionWrap">
>         document.getAnonymousElementByAttribute(this, "anonid", "addonDescriptionWrap");
>       </field>
> 
>       <field name="_selectedStatusMsgs">
>         document.getAnonymousElementByAttribute(this, "anonid", "addonSelectedStatusMsgs");
>       </field>
> 
>+      <field name="_opType">
>+        document.getAnonymousElementByAttribute(this, "anonid", "addonOpType");
>+      </field>
>+
>       <property name="label" readonly="true">
>         <getter>
>           <![CDATA[
>             var labelPieces = [];
> 
>             // Add name and version
>             labelPieces.push(this._nameVersion.getAttribute("name"));
>             labelPieces.push(this._nameVersion.getAttribute("version"));
> 
>             // Add description
>             labelPieces.push(this._descriptionWrap.textContent);
> 
>+            // Add operation message (if any)
>+            // Note 1: must handle this separately from other the labels
>+            // because this message is in an anonymous content node
>+            // within an hbox which is dynamically bound to a different
>+            // "addon-needs-*" binding based on the operation type.
>+            // Note 2: must check hbox's CSS "display" property with
>+            // getComputedStyle, because XUL "hidden" and "collapsed"
>+            // attributes never change.

Just |hidden|, not collapsed (the collapsed attribute is mapped to a |visibility: hidden;| style rule, which you don't, and you shouldn't check).

>+            // Note 3: ....
>+            // &toBeDisabled.label text), but the hbox is no longer visible
>+            // because its CSS "display" property is reset to "none".

"is no longer visibile." is enough :)

>+            if (document.defaultView.getComputedStyle(this._opType, '').display != 'none') {
>+              labelPieces.push(document.getAnonymousNodes(this._opType)[0].value);
>+            }
>+

nit: ger rid of single-line braces.

r=mano otherwise.
Attachment #233904 - Flags: review?(bugs.mano) → review+
(Assignee)

Comment 7

12 years ago
Created attachment 234029 [details] [diff] [review]
fixed nits
Attachment #233904 - Attachment is obsolete: true
(Assignee)

Updated

12 years ago
Whiteboard: [checkin needed]
mozilla/toolkit/mozapps/extensions/content/extensions.xml 	1.37
Status: NEW → RESOLVED
Last Resolved: 12 years ago
Resolution: --- → FIXED
Whiteboard: [checkin needed]
Comment on attachment 234029 [details] [diff] [review]
fixed nits

a=beltzner on behalf of 181drivers
Attachment #234029 - Flags: approval1.8.1+

Updated

12 years ago
Whiteboard: [checkin needed (1.8 branch)]
mozilla/toolkit/mozapps/extensions/content/extensions.xml 	1.17.2.17
Keywords: fixed1.8.1
Whiteboard: [checkin needed (1.8 branch)]
Version: Trunk → 2.0 Branch
You need to log in before you can comment on or make changes to this bug.