Closed Bug 33735 Opened 25 years ago Closed 24 years ago

[Mac] Keyboard shortcuts broken when no browser windows open

Categories

(Core :: DOM: UI Events & Focus Handling, defect, P1)

PowerPC
Mac System 8.6
defect

Tracking

()

VERIFIED FIXED
Future

People

(Reporter: n.i.meara, Assigned: mikepinkerton)

References

Details

(Keywords: platform-parity, regression, relnote)

Overview Description: On Mac OS the keyboard shortcuts do not work when Mozilla is running with no windows open. Steps to Reproduce: 1) Launch Mozilla. 2) Close the initial browser window. 3) Type a keyboard shortcut to perform an action, such as quitting the application (Command Q) or opening a new browser window (Command N). Actual Results: The keyboard shortcuts have no effect.Nothing happens. Expected Results: Appropriate action for keyboard shortcut is performed. Build ID and Platform: 2000032808 build on Mac OS 8.6
i keep thinking there's already a bug on this, but i cannot seem to find it. anyhow, it's a definite regression w/respect to 4.x.
Severity: normal → major
Keywords: 4xp, pp
There is another bug on this, I think Pink has it (and a proposed fix!).
saari, do you already have this bug? i never checked in my presShell changes, though they are on my machine.
Assignee: pinkerton → saari
I'll put this in M15 with the assumption that I can just use the code pink wrote. Sure, I'll take the credit ;-)
Status: NEW → ASSIGNED
Target Milestone: --- → M15
Mass-moving most M15 bugs to M16
Target Milestone: M15 → M16
Priority: P3 → P1
Keywords: nsbeta2
Putting on [nsbeta2+] radar.
Whiteboard: [nsbeta2+]
Command shortcuts also do not work when a view source window is active. Arrow keys for navigation and pageup/pagedown also don't work when view source is active. (Is this part of the same problem?)
hi John, the problem you're seeing w/keyboard shortcuts in the source window is tracked in bug 26593.
Blocks: 40158
Fixed.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
verif on Mac OS 9.0, 2000.05.24.08, opt commercial bits.
Status: RESOLVED → VERIFIED
This has been reoccurring on the nightly Mac builds lately. Verified with 2000-09-13 build on Mac OS 8.6. Reopening.
Status: VERIFIED → REOPENED
Resolution: FIXED → ---
->future
Target Milestone: M16 → Future
Keywords: helpwanted, relnote
*** Bug 51414 has been marked as a duplicate of this bug. ***
*sigh*
Component: XP Toolkit/Widgets: Menus → Keyboard Navigation
Keywords: nsbeta2nsbeta3, regression
Whiteboard: [nsbeta2+]
The reason that they don't work is because of the test for an empty window content rgn in nsMacMessagePump::DispatchOSEventToRaptor(). Because the hidden window is (I presume) 0,0 pixels, it counds as empty. We just need some special casing here. Reassign to pinkerton in danm's absence.
Assignee: saari → pinkerton
Status: REOPENED → NEW
This should be an easy kill for nsbeta3.
Keywords: helpwanted
I'm not 100% convinced that is all there is to it Simon (although that would be nice). Did you try this fix? If this doesn't work, it is related to changes in activation <sigh>
It works fine if I omit that check for an empty window.
Cool! I must have been chasing a dead end before.
pink noted that backing out that change will cause tooltips to show up for windowshaded windows, which is pretty brain-dead. So can we filter what events are passed through in this case? Maybe we should send keys, but not mouse moves, for example.
That sounds reasonable. Or is there a better check for being window shaded than "is empty"?
We're now filtering only mousemove events for windowshaded windows, which returns key events to their normal programming. All better.
Status: NEW → RESOLVED
Closed: 24 years ago24 years ago
Resolution: --- → FIXED
vrfy fixed using 2000.10.03.12-n6 bits.
Status: RESOLVED → VERIFIED
Component: Keyboard: Navigation → User events and focus handling
You need to log in before you can comment on or make changes to this bug.