Closed Bug 413658 Opened 18 years ago Closed 18 years ago

UI for offline apps: missing accesskey for Remove button

Categories

(Firefox :: Settings UI, defect)

defect
Not set
normal

Tracking

()

VERIFIED FIXED
Firefox 3 beta3

People

(Reporter: wladow, Assigned: wladow)

References

Details

(Keywords: access, polish)

Attachments

(1 file, 1 obsolete file)

Attached patch add accesskey for Remove button (obsolete) — Splinter Review
UI introduced by bug 394392 Remove button lacks accesskey
Attachment #298709 - Flags: review?(mconnor)
And we should replace "..." with ellipsis to be consistent with other UI
Attachment #298709 - Attachment is obsolete: true
Attachment #298715 - Flags: review?(mconnor)
Attachment #298709 - Flags: review?(mconnor)
Attachment #298715 - Flags: review?(mconnor) → review+
Attachment #298715 - Flags: approval1.9?
Comment on attachment 298715 [details] [diff] [review] add accesskey for Remove button and replace "..." a=beltzner for 1.9
Attachment #298715 - Flags: approval1.9? → approval1.9+
Keywords: checkin-needed
Checking in browser/components/preferences/advanced.xul; /cvsroot/mozilla/browser/components/preferences/advanced.xul,v <-- advanced.xul new revision: 1.47; previous revision: 1.46 done Checking in browser/locales/en-US/chrome/browser/preferences/advanced.dtd; /cvsroot/mozilla/browser/locales/en-US/chrome/browser/preferences/advanced.dtd,v <-- advanced.dtd new revision: 1.31; previous revision: 1.30 done
Status: NEW → RESOLVED
Closed: 18 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Verified with Mozilla/5.0 (Windows; U; Windows NT 6.0; sk; rv:1.9b3pre) Gecko/2008012704 Minefield/3.0b3pre
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: