Checkbox for restore_on_demand should always be accessible

RESOLVED FIXED in Firefox 15

Status

()

Firefox
Preferences
--
enhancement
RESOLVED FIXED
6 years ago
4 years ago

People

(Reporter: sdrocking, Assigned: ahurle)

Tracking

Trunk
Firefox 15
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [good first bug][mentor=zpao][lang=js])

Attachments

(2 attachments, 1 obsolete attachment)

(Reporter)

Description

6 years ago
Bug 648683 allows users to enable restore_on_demand when they choose "Show my windows and tabs from last time" for Firefox startup. This should always be accessible irrespective of the users selection in the other menu. A user may need to restart Firefox for an update or an addon installation and may not want to restore all tabs.
(Reporter)

Updated

6 years ago
Blocks: 648683

Comment 1

6 years ago
Dugar, this issue seems to be an enhancement request rather than a bug. Please mark it correspondingly.
Version: Trunk → 13 Branch

Comment 2

6 years ago
Sorry, the version got changed accidentally.
Version: 13 Branch → Trunk
Yea, that's probably a good idea. It sort of made sense when I first did it but now I'm less sure. Let's see what UX says - Limi, can you comment?
Whiteboard: [good first bug][mentor=zpao][lang=js]
(Reporter)

Comment 4

6 years ago
Changing importance to "enhancement" as requested.
Severity: normal → enhancement

Updated

6 years ago
Status: UNCONFIRMED → NEW
Ever confirmed: true
(Reporter)

Updated

6 years ago
Blocks: 711193

Updated

6 years ago
Component: Session Restore → Preferences
Keywords: uiwanted
QA Contact: session.restore → preferences
(In reply to Paul O'Shannessy [:zpao] from comment #3)
> Yea, that's probably a good idea. It sort of made sense when I first did it
> but now I'm less sure. Let's see what UX says - Limi, can you comment?

Sounds good, and it should probably be relocated to the Tabs page in that case. 

(I wonder if people will expect us to not load tabs opened in the background if we do that, but it seems like something that wouldn't make a lot of sense, maybe I'm overthinking it :)
Keywords: uiwanted
No longer blocks: 711193

Updated

6 years ago
Duplicate of this bug: 727106
Duplicate of this bug: 732260

Comment 8

6 years ago
was about to create this exact artifact.

This setting as it is, us slightly ineffective for several reasons.

1. A user might not choose to have firefox configured to load the last session when starting a new session, but will use an extension or the add=on manager to restart the browser either for a fresh start or extension updates, which will generally reload with the same tabs as prior to the restart.

In this case, the user may expect all tabs to reload.

2. Session managers:  a user would probably expect all tabs to reload in the case of loading a saved session, not all of them have a setting in their options to disable reload on demand (Session Manager does)
Assignee: nobody → fracture91
We'll move the checkbox to the tabs page then.  Note that the same change needs to be made in two places since we are porting to in-content preferences:

browser/components/preferences and
browser/components/preferences/in-content (for when browser.preferences.inContent = true)
Status: NEW → ASSIGNED
(Assignee)

Comment 10

5 years ago
Created attachment 624238 [details] [diff] [review]
v1 - Move checkbox to Tabs page
Attachment #624238 - Flags: review?(paul)
Comment on attachment 624238 [details] [diff] [review]
v1 - Move checkbox to Tabs page

Please rename restoreOnDemand.* to restoreTabsOnDemand.*
(Assignee)

Comment 12

5 years ago
Created attachment 624438 [details] [diff] [review]
v2 - restoreOnDemand.* renamed to restoreTabsOnDemand.*
Attachment #624238 - Attachment is obsolete: true
Attachment #624238 - Flags: review?(paul)
Attachment #624438 - Flags: review?(paul)
Comment on attachment 624438 [details] [diff] [review]
v2 - restoreOnDemand.* renamed to restoreTabsOnDemand.*

I know Limi said we should do this and on the surface it all looks good. But I'd like to get somebody from UX to chime in and see if we should reword the string now that it doesn't have the same context.
Attachment #624438 - Flags: review?(paul) → review+
Created attachment 624505 [details]
screenshot

Poking Boriss for ui-review since she ui-r'ed the original string.
Attachment #624505 - Flags: ui-review?(jboriss)
Attachment #624505 - Flags: ui-review?(jboriss) → ui-review+
(Assignee)

Updated

5 years ago
Keywords: checkin-needed
https://hg.mozilla.org/integration/mozilla-inbound/rev/2a027b72537a
Flags: in-testsuite-
Keywords: checkin-needed
Target Milestone: --- → Firefox 15
https://hg.mozilla.org/mozilla-central/rev/2a027b72537a
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED

Comment 17

4 years ago
(In reply to Paul O'Shannessy [:zpao] (not reading much bugmail, email directly) from comment #13)
> I'd like to get somebody from UX to chime in and see if we should reword the
> string now that it doesn't have the same context.
Because bug 792210 has yet to be fixed, users expect this option in its new "Tabs" location to prevent loading of background tabs opened via 'Open All in Tabs' bookmarks subfolder context menu (bug 851775) and when just middle-clicking/Apple-clicking many links to open them in background tabs (bug 911427).
Depends on: 792210, 851775, 911427
You need to log in before you can comment on or make changes to this bug.