Closed Bug 475798 Opened 16 years ago Closed 16 years ago

Library: Alt+E only focuses the More button, doesn't trigger it

Categories

(Firefox :: Bookmarks & History, defect)

defect
Not set
normal

Tracking

()

VERIFIED FIXED
Firefox 3.6a1

People

(Reporter: zeniko, Assigned: zeniko)

References

Details

(Keywords: access, regression, verified1.9.1)

Attachments

(1 file)

... due to bug 438306.

This regressed on Windows due to bug 403151 (probably never worked on the other platforms).
NB: The code to correctly update the accesskey on the <button type="image"> was removed while the accesskey was added.
Assignee: nobody → zeniko
Status: NEW → ASSIGNED
Attachment #359352 - Flags: review?(mak77)
Attachment #359352 - Flags: review?(mak77) → review+
Comment on attachment 359352 [details] [diff] [review]
obvious fix: remove the offending lines

thanks!
Keywords: checkin-needed
Attachment #359352 - Flags: approval1.9.1?
Comment on attachment 359352 [details] [diff] [review]
obvious fix: remove the offending lines

Drivers: This trivial patch only removes dead/obstructing code that shouldn't be there in the first place.
http://hg.mozilla.org/mozilla-central/rev/76540a65adc0
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Keywords: checkin-neededaccess
Resolution: --- → FIXED
Target Milestone: --- → Firefox 3.2a1
Verified with Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9.2a1pre) Gecko/20090131 Minefield/3.2a1pre
Status: RESOLVED → VERIFIED
Comment on attachment 359352 [details] [diff] [review]
obvious fix: remove the offending lines

a191=beltzner
Attachment #359352 - Flags: approval1.9.1? → approval1.9.1+
Keywords: checkin-needed
verified FIXED on Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1b4pre) Gecko/20090331 Shiretoko/3.5b4pre ID:20090331041754
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: