Closed Bug 475802 Opened 16 years ago Closed 15 years ago

remove undiscoverable accesskeys in Library

Categories

(Firefox :: Bookmarks & History, defect)

defect
Not set
normal

Tracking

()

VERIFIED FIXED
Firefox 3.6a1

People

(Reporter: zeniko, Assigned: zeniko)

References

Details

Attachments

(1 file)

AFAICT backCmd.accesskey and forwardCmd.accesskey are not discoverable at all except if in a translation the same accesskey is used in a different place (e.g. in the German localization, see bug 475628).

And since bug 408663 was fixed, not even a11y is an argument to keep these...
Blocks: 475628
Assignee: nobody → zeniko
Status: NEW → ASSIGNED
Attachment #359357 - Flags: review?(mak77)
Comment on attachment 359357 [details] [diff] [review]
remove the undiscoverable accesskeys

since this is a change in a behavior, forwarding to dietrich.
Attachment #359357 - Flags: review?(mak77) → review?(dietrich)
Whiteboard: [needs review dietrich]
Comment on attachment 359357 [details] [diff] [review]
remove the undiscoverable accesskeys

yeah this should be fine, r=me
Attachment #359357 - Flags: review?(dietrich) → review+
Whiteboard: [needs review dietrich]
Keywords: checkin-needed
http://hg.mozilla.org/mozilla-central/rev/9ab600c487a7
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 3.6a1
Comment on attachment 359357 [details] [diff] [review]
remove the undiscoverable accesskeys

Approval requested only for the places.xul bits (no l10n change required).
Attachment #359357 - Flags: approval1.9.1?
verified per code check in
Status: RESOLVED → VERIFIED
Comment on attachment 359357 [details] [diff] [review]
remove the undiscoverable accesskeys

Don't really see the advantage of taking the XUL only fix on branch.
Attachment #359357 - Flags: approval1.9.1? → approval1.9.1-
Bug 451915 - move Firefox/Places bugs to Firefox/Bookmarks and History. Remove all bugspam from this move by filtering for the string "places-to-b-and-h".

In Thunderbird 3.0b, you do that as follows:
Tools | Message Filters
Make sure the correct account is selected. Click "New"
Conditions: Body   contains   places-to-b-and-h
Change the action to "Delete Message".
Select "Manually Run" from the dropdown at the top.
Click OK.

Select the filter in the list, make sure "Inbox" is selected at the bottom, and click "Run Now". This should delete all the bugspam. You can then delete the filter.

Gerv
Component: Places → Bookmarks & History
QA Contact: places → bookmarks
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: