Closed Bug 259224 Opened 20 years ago Closed 19 years ago

Control j shortcut key appears 2 times in Tools menu

Categories

(Firefox :: Keyboard Navigation, defect, P3)

x86
Linux
defect

Tracking

()

RESOLVED FIXED
Firefox1.5

People

(Reporter: nagappan, Assigned: Gavin)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

In tools menu both web search and downloads menuitem in Tools menu has the shortcut key Control+j. I'm using GTK2 based Firefox.
Obviously these needs to be cleaned up if it's still a problem. Any takers for a trivial fix?
Blocks: firekey
Keywords: helpwanted
So Firefox doesn't have the same shortcut anymore. Can we just close this as invalid?
The fix for this was in bug 250396's attachment 157066 [details] [diff] [review], and was apparently undone by the aviary branch landing.
Assignee: aaronleventhal → gavin.sharp
Priority: -- → P3
Target Milestone: --- → Firefox1.1
Version: 1.0 Branch → Trunk
Attached patch Patch (deleted) — Splinter Review
Remove the shortcut key for downloads, in Linux only.
Attachment #175981 - Flags: review?(mconnor)
Status: NEW → ASSIGNED
Attachment #175981 - Flags: review?(mconnor) → review+
Whiteboard: [checkin needed]
Checking in browser-menubar.inc; /cvsroot/mozilla/browser/base/content/browser-menubar.inc,v <-- browser-menubar.inc new revision: 1.46; previous revision: 1.45 done
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Keywords: aviary-landing
Resolution: --- → FIXED
Whiteboard: [checkin needed]
Something is wrong here. In Linux, the download manager opens on pressing Ctrl+Y, but that shortcut is not shown in the menu.
Gah, what an obvious mistake. The ifdef in this patch gives the same result regardless of platform.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Er, no, I was wrong. There is no key defined on Linux. There is something wacky going on though, http://lxr.mozilla.org/mozilla/source/browser/base/content/browser-sets.inc#218 uses ifdef XP_GNOME.
XP_GNOME is defined in the same file as "XP_UNIX, but not XP_MACOSX". So we browser-sets.inc defines "key_openDownloads" for Linux except Mac (#ifdef XP_GNOME) and the same "key_openDownloads" for other platforms (#else). XP_GNOME seems to work fine because Ctrl+Y opens the download manager on Linux. Looks like we should use that on all platforms in browser-menubar.inc.
Depends on: 313572
The Ctrl+Y not showing up in the menu regression was fixed in bug 313572.
Status: REOPENED → RESOLVED
Closed: 20 years ago19 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: