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)
Tracking
()
RESOLVED
FIXED
Firefox1.5
People
(Reporter: nagappan, Assigned: Gavin)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
(deleted),
patch
|
mconnor
:
review+
|
Details | Diff | Splinter Review |
In tools menu both web search and downloads menuitem in Tools menu has the
shortcut key Control+j. I'm using GTK2 based Firefox.
Comment 1•20 years ago
|
||
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?
Assignee | ||
Comment 3•20 years ago
|
||
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
Keywords: helpwanted → aviary-landing
Priority: -- → P3
Target Milestone: --- → Firefox1.1
Version: 1.0 Branch → Trunk
Assignee | ||
Comment 4•20 years ago
|
||
Remove the shortcut key for downloads, in Linux only.
Attachment #175981 -
Flags: review?(mconnor)
Assignee | ||
Updated•20 years ago
|
Status: NEW → ASSIGNED
Updated•20 years ago
|
Attachment #175981 -
Flags: review?(mconnor) → review+
Assignee | ||
Updated•20 years ago
|
Whiteboard: [checkin needed]
Comment 5•20 years ago
|
||
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]
Comment 6•19 years ago
|
||
Something is wrong here. In Linux, the download manager opens on pressing
Ctrl+Y, but that shortcut is not shown in the menu.
Assignee | ||
Comment 7•19 years ago
|
||
Gah, what an obvious mistake. The ifdef in this patch gives the same result
regardless of platform.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 8•19 years ago
|
||
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.
Comment 9•19 years ago
|
||
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.
Assignee | ||
Comment 10•19 years ago
|
||
The Ctrl+Y not showing up in the menu regression was fixed in bug 313572.
Status: REOPENED → RESOLVED
Closed: 20 years ago → 19 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•