Closed
Bug 602562
Opened 14 years ago
Closed 14 years ago
Add keyboard modifier to start in safe mode for windows
Categories
(Toolkit :: Startup and Profile System, defect)
Tracking
()
VERIFIED
FIXED
mozilla2.0b8
Tracking | Status | |
---|---|---|
blocking2.0 | --- | betaN+ |
People
(Reporter: robert.strong.bugs, Assigned: robert.strong.bugs)
References
Details
(Keywords: relnote, user-doc-needed)
Attachments
(1 file, 3 obsolete files)
1.17 KB,
patch
|
robert.strong.bugs
:
review+
robert.strong.bugs
:
ui-review+
jimm
:
feedback+
|
Details | Diff | Splinter Review |
Similar to what was implemented for Mac OS X.
Comment 1•14 years ago
|
||
What about Linux?
![]() |
Assignee | |
Comment 2•14 years ago
|
||
(In reply to comment #1)
> What about Linux?
Please file a separate bug
![]() |
Assignee | |
Comment 3•14 years ago
|
||
note: bug 311466 added this for Mac OS X
![]() |
Assignee | |
Comment 4•14 years ago
|
||
Assignee: nobody → robert.bugzilla
Status: NEW → ASSIGNED
Attachment #488621 -
Flags: review?(dtownsend)
![]() |
Assignee | |
Comment 5•14 years ago
|
||
no value in checking the current value of gSafeMode and this makes it consistent with the Mac implementation
Attachment #488621 -
Attachment is obsolete: true
Attachment #488626 -
Flags: review?(dtownsend)
Attachment #488621 -
Flags: review?(dtownsend)
![]() |
Assignee | |
Comment 6•14 years ago
|
||
Comment on attachment 488626 [details] [diff] [review]
patch rev2
Alex, are you ok with using the ctrl key as the modifier to launch in safe mode on Windows?
Attachment #488626 -
Flags: ui-review?(faaborg)
![]() |
Assignee | |
Comment 7•14 years ago
|
||
Comment on attachment 488626 [details] [diff] [review]
patch rev2
Jim, running this by you since you might now of issues I don't with doing this. Thanks!
Attachment #488626 -
Flags: review?(jmathies)
Comment 8•14 years ago
|
||
Comment on attachment 488626 [details] [diff] [review]
patch rev2
I think we should go with shift on windows (and possibly add it for consistency on the other platforms in addition to option). Main reason is that it is easier to find than control (very novice users won't be familiar with alt/control since you don't really have to use those keys), and additionally there is some limited platform consistency with using shift in this way:
>Microsoft uses _shift_ to bypass startup routines/macros for Office Apps and to
>bypass Windows Startup programs.
https://bugzilla.mozilla.org/show_bug.cgi?id=542122#c14
Attachment #488626 -
Flags: ui-review?(faaborg) → ui-review-
![]() |
Assignee | |
Comment 9•14 years ago
|
||
Filed Bug 610086 for changing the Mac OS X modifier
Commented in bug 602566 that the modifier should be the shift key
Carrying forward ui-r+
Attachment #488626 -
Attachment is obsolete: true
Attachment #488627 -
Flags: ui-review+
Attachment #488627 -
Flags: review?(dtownsend)
Attachment #488627 -
Flags: feedback?(jmathies)
Attachment #488626 -
Flags: review?(jmathies)
Attachment #488626 -
Flags: review?(dtownsend)
![]() |
||
Comment 10•14 years ago
|
||
(In reply to comment #7)
> Comment on attachment 488626 [details] [diff] [review]
> patch rev2
>
> Jim, running this by you since you might now of issues I don't with doing this.
> Thanks!
I would add a comment explaining what this code does.
There's a chance that users might trigger this mistakenly, do we notify users when entering private mode so they understand what's going on?
Updated•14 years ago
|
Attachment #488627 -
Flags: review?(dtownsend) → review+
![]() |
Assignee | |
Comment 11•14 years ago
|
||
(In reply to comment #10)
> (In reply to comment #7)
> > Comment on attachment 488626 [details] [diff] [review] [details]
> > patch rev2
> >
> > Jim, running this by you since you might now of issues I don't with doing this.
> > Thanks!
>
> I would add a comment explaining what this code does.
Sure
> There's a chance that users might trigger this mistakenly, do we notify users
> when entering private mode so they understand what's going on?
It does... it displays the safe mode ui with the safe mode options.
Updated•14 years ago
|
blocking2.0: --- → betaN+
![]() |
Assignee | |
Comment 12•14 years ago
|
||
Carrying forward ui-r and r.
Jim, is this comment adequate?
Attachment #488627 -
Attachment is obsolete: true
Attachment #489266 -
Flags: ui-review+
Attachment #489266 -
Flags: review+
Attachment #489266 -
Flags: feedback?(jmathies)
Attachment #488627 -
Flags: feedback?(jmathies)
![]() |
||
Comment 13•14 years ago
|
||
(In reply to comment #12)
> Created attachment 489266 [details] [diff] [review]
> patch updated to comments
>
> Carrying forward ui-r and r.
>
> Jim, is this comment adequate?
yeah I'd say that covers it. :)
![]() |
||
Updated•14 years ago
|
Attachment #489266 -
Flags: feedback?(jmathies) → feedback+
![]() |
Assignee | |
Comment 14•14 years ago
|
||
Pushed to mozilla-central
http://hg.mozilla.org/mozilla-central/rev/c5fb2f1485cd
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Flags: in-testsuite-
Flags: in-litmus?
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.0b8
Comment 15•14 years ago
|
||
Man, this is slick! \o/ V. Fixed.
Mozilla/5.0 (Windows NT 6.1; rv:2.0b8pre) Gecko/20101118 Firefox/4.0b8pre ID:20101118191737
Status: RESOLVED → VERIFIED
![]() |
Assignee | |
Updated•14 years ago
|
Keywords: user-doc-needed
Comment 16•14 years ago
|
||
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?
Comment 17•14 years ago
|
||
Created Bug 619866.
Comment 18•14 years ago
|
||
I have the same issue. Using both Ctrl + Shift + ... makes for a suitable Global Hotkey.
If Shift is to be used could there at least be an option to deactivate it or, better yet, to have it user defined?
![]() |
Assignee | |
Comment 19•14 years ago
|
||
(In reply to comment #18)
> I have the same issue. Using both Ctrl + Shift + ... makes for a suitable
> Global Hotkey.
> If Shift is to be used could there at least be an option to deactivate it or,
> better yet, to have it user defined?
See Bug 619866 - it is fixed for the next beta and in the nightly builds.
Comment 20•14 years ago
|
||
Would you consider adding a setting to allow this behavior to be disabled?
Although it's useful for regular users, it is bit of a problem for those of us who automatically run browser tests with Firefox using tools such as selenium-webdriver. As the tests run Firefox gets opened up a number of times automatically, in the background, during which time many developers carry on editing their work or checking their email or what have you.
But if it happens that they have the shift key down at one of the times when a new Firefox instance is started, then it will start in safe mode, which of course doesn't work with the browser control mechanisms that these tools use.
So again, perfectly reasonable for the default behavior, but it'd be great if we could supply a profile setting to bypass this, much as we supply profile settings to turn off various UI prompts and warnings (browser.tabs.warnOnClose and security.warn_entering_secure for example).
Comment 21•14 years ago
|
||
That's a good point, and you already have my vote. Although the situation was improved so that it has to be the Shift key alone without Ctrl or Alt, my Ctrl+Shift+N hot key still triggers safe mode about 10% of the time if I happen to release Ctrl a fraction of a second before Shift. I don't see any harm in providing a hidden setting to disable the Shift key modifier while still allowing -safe-mode from the command line.
![]() |
Assignee | |
Comment 22•14 years ago
|
||
The shift key check is currently done early during startup before profile selection. If someone wanted to add this functionality, filed a bug, provided a patch, and it didn't hurt startup performance which it most likely wouldn't I highly suspect the patch would be accepted. I definitely won't have time to look at implementing that in the foreseeable future.
![]() |
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
•