Closed
Bug 917020
Opened 11 years ago
Closed 11 years ago
Remove "startup" section from MetroFx options
Categories
(Firefox for Metro Graveyard :: Flyouts, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
Firefox 27
People
(Reporter: TimAbraldes, Assigned: kjozwiak)
References
Details
(Whiteboard: [preview])
Attachments
(2 files)
140.78 KB,
image/png
|
Details | |
3.38 KB,
patch
|
mbrubeck
:
review+
|
Details | Diff | Splinter Review |
This was recently discussed on the metro mailing list:
https://mail.mozilla.org/pipermail/metro/2013-September/000282.html
We will remove the "startup" section from our options panel, and will always act as if "show tabs from last time" is selected.
Reporter | ||
Comment 1•11 years ago
|
||
This is a screenshot highlighting the option that will be removed
Comment 2•11 years ago
|
||
I agree we should remove the option, but the default option should be to show start page?
I think that the MS guideline for Metro apps is to resume the session if the app was suspended and killed, but to start fresh if the previous execution state was a fresh close.
Comment 3•11 years ago
|
||
Yes, I think Brian is correct. For those users who do discover "kill it for real" we should start up fresh. That respects the Windows guidelines and gives the user a simple "get rid of this mess and start fresh" gesture.
Reporter | ||
Updated•11 years ago
|
Summary: Remove option to show start page on startup → Remove "startup" section from MetroFx options
Reporter | ||
Comment 4•11 years ago
|
||
SGTM. I see the benefit of defaulting to either option and I don't have a strong preference for one over the other
Comment 5•11 years ago
|
||
Kamil, interested in taking this one?
Assignee | ||
Comment 7•11 years ago
|
||
Removed the "Startup" labels and menu items from the "Options" panel. Doubled checked metro.js and ensured that when Firefox Metro is killed incorrectly (crashes), that all the tabs are restored.
Went through the following test cases:
- Closed Firefox Metro (killed properly) and ensured that a "fresh slate" was used when re-launching the browser
- Closed Firefox Metro using the Task Manager (incorrect kill) and ensured that all the tabs where restored when the browser was re-launched
Please let me know if I missed something!
Attachment #814696 -
Flags: review?(mbrubeck)
Assignee | ||
Comment 8•11 years ago
|
||
Forgot to mention that I left the pref there so it can be used in the future if needed but is not supported by the UI
Comment 9•11 years ago
|
||
Comment on attachment 814696 [details] [diff] [review]
removing "Startup" from the "Options" flyout
Review of attachment 814696 [details] [diff] [review]:
-----------------------------------------------------------------
Thanks!
Attachment #814696 -
Flags: review?(mbrubeck) → review+
Assignee | ||
Comment 10•11 years ago
|
||
Thanks Matt,
Would you be able to push this to the try server if you have some time? Just let me know!
Comment 11•11 years ago
|
||
Metro tests passed locally; landing directly to fx-team:
https://hg.mozilla.org/integration/fx-team/rev/10824701bc80
Status: NEW → ASSIGNED
Hardware: x86_64 → All
Whiteboard: [preview-triage] → [preview]
Comment 12•11 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 27
Updated•10 years ago
|
OS: Windows 8 Metro → Windows 8.1
You need to log in
before you can comment on or make changes to this bug.
Description
•