Closed Bug 335645 Opened 19 years ago Closed 18 years ago

[meta] Add-ons window accessibility

Categories

(Toolkit :: Add-ons Manager, defect)

1.8 Branch
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla1.8.1beta2

People

(Reporter: robert.strong.bugs, Unassigned)

References

Details

(Keywords: access, meta, Whiteboard: [mustfix])

Bug 329045 made several changes to the Extension Manager that should be reviewed in order to be sure that it is doing the right things in regards to accessibility. The obvious changes that need to be reviewed are extensions.xul (the radiogroup that is based on the pref panel buttons) and extensions.xml (the addonsmessage which is based on browsermessage). cc'ing aaronlev and Mark Pilgrim
access key word?
Keywords: access
While trying to reproduce bug 335747 I've noted that Console²'s FAYT implementation (which might eventually end up in Toolkit through bug 298371) for <richlistbox> doesn't work any longer. That implementation depends on the label property of <richlistitem> which for add-ons now always returns an empty string because of the heavy use of inner bindings. To resolve this, #addon-base or any binding inheriting from it should reimplement the label property, returning at least name and version (and possibly description and installation state) of the add-on. I'm not sure though, whether the label property is used elsewhere...
It would probably be best to utilize the fayt_label for the EM as in your patch or what ever method gets implemented as the next rev of richlistbox.
Assignee: nobody → pilgrim
Depends on: 338048
The add-ons window also uses <prefpane> and therefore triggers bug 334370.
Depends on: 334370
Summary: Review of accessibility for the Extension Manager → [meta] Add-ons window accessibility
(In reply to comment #2) > To resolve this, #addon-base or any binding inheriting from it should > reimplement the label property, returning at least name and version (and > possibly description and installation state) of the add-on. I believe this is the same bug that I independently found and logged as bug 338048.
Keywords: meta
Assignee: pilgrim → nobody
Blocks: fox2access
Flags: blocking-firefox2?
Keywords: sec508
Depends on: 339646
--> blocking beta2
Flags: blocking-firefox2? → blocking-firefox2+
Target Milestone: --- → Firefox 2 beta2
Depends on: 340261
Mark, I'm putting this on your list for now, since the active dependencies are both assigned to you.
Whiteboard: [mustfix]
Dependencies are on the blocker list, pulling this off (no meta bugs!)
Flags: blocking-firefox2+
No longer depends on: 334370
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Product: Firefox → Toolkit
You need to log in before you can comment on or make changes to this bug.