Closed
Bug 13908
Opened 26 years ago
Closed 26 years ago
Can't use cmd-m until I click in the bookmarks menu for the first time
Categories
(Core :: XUL, defect, P1)
Tracking
()
VERIFIED
FIXED
M10
People
(Reporter: mikepinkerton, Assigned: waterson)
Details
(Whiteboard: okay, maybe fix not understood)
Open the browser window and click in it to set focus.
hit cmd-shift-m. Nothing happens.
Click the bookmarks menu, but don't pick anything.
hit cmd-shift-m again. Now the cmd-key equivalent works.
Updated•26 years ago
|
Status: NEW → ASSIGNED
Priority: P3 → P1
Target Milestone: M11
Comment 1•26 years ago
|
||
If I remove the observer/broadcaster from this equation by just putting
onkeypress on the key node, this works correctly. Same thing with oncommand.
So, something strage is happening with the observer/broadcaster... both with the
menu needing to be pulled down, and with the shift key needing to be pressed.
Both of these problems only manifest with broadcasters/observers in the mix.
Comment 2•26 years ago
|
||
Didn't I file this same bug a while back? Bug 13378 is the one I'm thinking of.
Comment 3•26 years ago
|
||
13378 looks similar, but was really a different issue.
There is something even more interesting happening here
Updated•26 years ago
|
Assignee: saari → waterson
Status: ASSIGNED → NEW
Target Milestone: M11 → M10
Comment 4•26 years ago
|
||
Assigning to waterson and marking M10 as it prevents keybinding from functioning
properly
Assignee | ||
Updated•26 years ago
|
Status: NEW → ASSIGNED
Whiteboard: fix understood. typing.
Updated•26 years ago
|
QA Contact: beppe → claudius
Assignee | ||
Updated•26 years ago
|
Whiteboard: fix understood. typing. → okay, maybe fix not understood
Assignee | ||
Comment 5•26 years ago
|
||
okay, i fixed the "forward declaration of observers" problem; however, i'm
still seeing the problem that bug originally reported. Namely, I can't seem to
run "Ctrl+B" until the bookmarks menu has been created once. At least this
seems to be XP now: it's happening on Win32.
Assignee | ||
Updated•26 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 6•26 years ago
|
||
changed landed in trunk and M10 branch.
Updated•26 years ago
|
Status: RESOLVED → VERIFIED
Comment 7•26 years ago
|
||
I looked at 19990930 builds on all platforms and 'Ctrl-B' or command-b on
the mac all work without first creating the Bookmarks menu. marking VERIFIED.
You need to log in
before you can comment on or make changes to this bug.
Description
•