251092 - Alt+U doesn't work in Theme Manager

Status: VERIFIED FIXED

(Toolkit :: Add-ons Manager, defect, P2)

Description

[Jesse Ruderman]

Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.7) Gecko/20040712
Firefox/0.9.1+

Alt+U doesn't work in Theme Manager.  It does work in the Extension Manager.

Comment 1

[Blake Ross]

As far as I can tell, keyboard navigation (e.g. tabbing) doesn't work in this
window at all.

Comment 2

[Ryan Polk (Quark)]

from extensions.dtd:
<!ENTITY cmd.update.label                 "Update">
<!ENTITY cmd.update.accesskey             "U">
<!ENTITY cmd.update.tooltip               "Checks for Updates to your Extensions">
<!ENTITY cmd.useTheme.label               "Use Theme">
<!ENTITY cmd.useTheme.accesskey           "U">

Alt+U (for Update) isn't working because it's being overwritten by the Use Theme
accesskey.

Comment 3

[Ryan Polk (Quark)]

Attachment: one liner: set 'T' as access key

Have Use Theme use the accesskey of 'T' instead, so that it doesn't overwrite
Update's accesskey.

Comment 4

[Asa Dotzler [:asa]]

Comment on attachment 156281 [details] [diff] [review]
one liner: set 'T' as access key

a=asa for aviary checkin.

Comment 5

[Jesse Ruderman]

I checked in Quark's patch, trunk and aviary.  Thanks Quark :)

Comment 6

[:deanis74]

*** Bug 253731 has been marked as a duplicate of this bug. ***

Comment 7

[Tracy Walker [:tracy]]

verified fixed with Firefox Branch build 2004-09-10-08-0.9