Closed
Bug 619866
Opened 14 years ago
Closed 14 years ago
Holding Shift should only trigger safe mode if other modifiers are not held (avoid conflicts with custom shortcuts)
Categories
(Toolkit :: Startup and Profile System, defect)
Tracking
()
RESOLVED
FIXED
mozilla2.0b9
Tracking | Status | |
---|---|---|
blocking2.0 | --- | - |
People
(Reporter: quatrixj, Assigned: robert.strong.bugs)
References
Details
(Keywords: regression)
Attachments
(1 file)
1.47 KB,
patch
|
mossop
:
review+
mossop
:
approval2.0+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.0b8) Gecko/20100101 Firefox/4.0b8 Build Identifier: See Bug 602562. This interferes with some Windows functionality. I specify a hot key Ctrl+Shift+N in the Firefox shortcut properties and use that key to launch it. As of Beta 8, using that shortcut starts Firefox in safe mode. The Shift key is also used in Windows to launch a program minimized, so holding Shift and clicking a Firefox shortcut starts it in safe mode but minimized. Is there at least a way to disable this behavior? Reproducible: Always Steps to Reproduce: 1. Find Firefox shortcut in the Start Menu and open properties. 2. Change Shortcut Key to Ctrl+Shift+N. OK/Apply. 3. Press Ctrl+Shift+N. Actual Results: Firefox safe mode dialog pops up. Expected Results: Firefox opens normally.
Comment 1•14 years ago
|
||
This is exactly what I was afraid of! Why are we using shift instead of control, or something less likely to be accidental?
Keywords: regression
![]() |
Assignee | |
Comment 2•14 years ago
|
||
I was afraid of using shift exactly for this same reason! as well but I don't think using ctrl would have prevented this - see bug 602562 as to why we are using shift. As for starting minimized using shift, I tried to reproduce on Win7 and Win2K and programs (e.g. IE, notepad, wordpad, Firefox prior to this change) did not start minimized when holding the shift key down so more details as to how to reproduce that specific issue would be helpful. We can likely fix the issue regarding ctrl + shift + <any other key> in the "Steps to Reproduce: but more info is required to figure out why I was unable to reproduce the start minimized. There is still plenty of time to fix the issue with shortcut keys but without the ability to reproduce the start minimized case I can't say whether this is a significant issue or whether there is a reasonable way to resolve it.
Re: Programs starting minimized: http://support.microsoft.com/kb/78373
Apologies for bugspam, but just saw that that link was for the following (really old) versions of Windows: Microsoft Windows 3.0 Standard Edition Microsoft Windows 3.1 Standard Edition Microsoft Windows 3.11 Standard Edition So...
![]() |
Assignee | |
Comment 5•14 years ago
|
||
Yep, I've *read* that. ;) I've also tried that on a couple of systems... one of them being a MozQA Win2K system... and notepad, wordpad, IE, and Firefox B7 prior to this change did not start minimized.
![]() |
Assignee | |
Comment 6•14 years ago
|
||
So, if we can exclude the "start minimized" case it would be fairly simple to handle the shortcut key case. James, can you confirm that it starts minimized when holding the shift key down... I haven't been able to reproduce.
![]() |
Assignee | |
Comment 7•14 years ago
|
||
(In reply to comment #4) > Apologies for bugspam, but just saw that that link was for the following > (really old) versions of Windows: > Microsoft Windows 3.0 Standard Edition > Microsoft Windows 3.1 Standard Edition > Microsoft Windows 3.11 Standard Edition > > So... Just for my own sanity, where did you find this info since the page did not reference the versions?
I clicked "Applies to", and those versions popped up. Regarding Comment 0's steps to reproduce, step 3 can still launch Firefox in not-safe-mode if you let go of Shift immediately after pressing 'N'. Holding for more than a second launches in safe mode.
![]() |
Assignee | |
Comment 9•14 years ago
|
||
Thanks... clicking "expand all" doesn't expand all as in sub items. :/
![]() |
Assignee | |
Updated•14 years ago
|
Summary: "Hold Shift key for safe mode" conflicts with Windows functions → Shift key for safe mode conflicts with Windows shortcut keys
![]() |
Assignee | |
Comment 10•14 years ago
|
||
I'm going to wait on a response from James to comment #2 / comment #5 before requesting review
Assignee: nobody → robert.bugzilla
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Reporter | ||
Comment 11•14 years ago
|
||
The minimize behavior is inconsistent depending on the location of the shortcut (Start Menu programs, pinned, desktop, etc.), but it happens every time for me with a desktop shortcut. Try holding Shift while double-clicking on a desktop shortcut. I get the safe mode dialog minimized to the task bar.
Reporter | ||
Comment 12•14 years ago
|
||
Regarding the press/release timing, I'm pretty quick on a keyboard but still can't release Shift fast enough to avoid going into safe mode. I'm letting go easily < 0.5 s after pressing "N". Seems like it would vary widely by first/cold start vs. subsequent start and other variables.
Updated•14 years ago
|
![]() |
Assignee | |
Comment 13•14 years ago
|
||
Thanks James, I am able to reproduce it starting minimized using shift with a desktop shortcut but it also starts minimized using ctrl with a desktop shortcut. Can you confirm that this happens when launching with ctrl as well for you?
![]() |
Assignee | |
Comment 14•14 years ago
|
||
try build with the patch http://ftp.mozilla.org/pub/mozilla.org/firefox/tryserver-builds/rstrong@mozilla.com-174931132be1/try-w32/firefox-4.0b9pre.en-US.win32.installer.exe
Reporter | ||
Comment 15•14 years ago
|
||
Confirmed, either Shift or Ctrl in Windows 7, but neither in XP. However, I tried the same thing with a few other apps, and none of them started minimized in 7. It's only happening for me with Firefox and Thunderbird, suggesting that something Mozilla-specific is going on here. The KB article also applies only to Windows 3, further evidence that I was mistaken and it has nothing to do with Windows itself. Still something to look into if Mozilla is doing something weird.
![]() |
Assignee | |
Comment 16•14 years ago
|
||
Thanks again James, I think a separate bug on the minimized issue should be filed and that we should be able to move forward with just check if ctrl or alt is pressed during startup and if so not launch in safe mode. If you get a chance, could you confirm that the try build I linked to does the right thing for you?
Comment 17•14 years ago
|
||
>This interferes with some Windows functionality ... >I specify a hot key It sounds like this is interfering with the potential of a user specified hot key, as opposed to a built in windows feature. Even if this did directly conflict with an existing windows feature, I would still argue in favor of the behavior, since it streamlines support, and having two shortcuts is non-negotiable. I recommend wontfix.
![]() |
Assignee | |
Comment 18•14 years ago
|
||
Alex, the patch makes it so if the shift key is pressed without the ctrl or alt key being pressed we launch in safe mode. If the shift key is pressed and the ctrl and / or alt key is pressed it doesn't start in safe mode. This is the behavior we should have and I highly recommend not wontfixing this especially since there is a patch attached.
Comment 19•14 years ago
|
||
Yeah, that's fine. Sorry, thought we were heading towards the suggestion of getting rid of the feature entirely due to an obscure boundary case.
![]() |
Assignee | |
Updated•14 years ago
|
Attachment #498293 -
Flags: review?(dtownsend)
Reporter | ||
Comment 20•14 years ago
|
||
Confirmed, test build resolves my problem. Thanks for the quick fix, Robert. As a programmer of business apps myself, I know it's tempting sometimes to call something a corner case and say the software "works as designed" even when it's a simple fix.
Comment 21•14 years ago
|
||
Fun, I remember the shift issue on Windows 3.x somewhat. The shift key was important then because the concept of the taskbar didn't exist till windows 95 to deal with both a desktop, a start menu and a taskbar that could handle such cases, as all programs were launched either from Program Manager or File Manager as the KB article refers to but if you didn't watch out, not using shift made the program take focus and take up the screen, and I believe that shift to minimized was a way to keep it out of site till you wanted to use.
Updated•14 years ago
|
Attachment #498293 -
Flags: review?(dtownsend)
Attachment #498293 -
Flags: review+
Attachment #498293 -
Flags: approval2.0+
Updated•14 years ago
|
blocking2.0: ? → -
![]() |
Assignee | |
Comment 22•14 years ago
|
||
Pushed to mozilla-central http://hg.mozilla.org/mozilla-central/rev/875e5ea75241
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Flags: in-testsuite-
Flags: in-litmus?
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.0b9
Updated•14 years ago
|
Summary: Shift key for safe mode conflicts with Windows shortcut keys → Holding Shift should only trigger safe mode if other modifiers are not held (avoid conflicts with custom shortcuts)
![]() |
Assignee | |
Updated•13 years ago
|
Flags: in-litmus?
You need to log in
before you can comment on or make changes to this bug.
Description
•