The default bug view has changed. See this FAQ.

Home Page -> Use Current Pages; Does not respect Tab groups

VERIFIED FIXED in Firefox 6

Status

Firefox Graveyard
Panorama
P3
normal
VERIFIED FIXED
6 years ago
a year ago

People

(Reporter: bugzilla, Assigned: ttaubert)

Tracking

Trunk
Firefox 6
Bug Flags:
in-testsuite ?

Details

(Whiteboard: [4b12])

Attachments

(2 attachments, 1 obsolete attachment)

(Reporter)

Description

6 years ago
User-Agent:       Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.0b12pre) Gecko/20110222 Firefox/4.0b12pre
Build Identifier: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.0b12pre) Gecko/20110222 Firefox/4.0b12pre

Only the current pages in the current tab group should be set as home, instead every page from all groups are, and they are all brought into the active group when clicking on the home button.

Perhaps there should be a url scheme to allow tab groups to be bookmarks or set as home pages etc.

Reproducible: Always

Steps to Reproduce:
1. separate some pages into groups
2. options > home page > use current pages
3. all pages are added
I can see the problem as described here. If the user's intention is to use the current pages as viewed in the current state in panorama, then this doesn't work as expected, because all the tabs come up in the same group. Nomination for clarification.
Status: UNCONFIRMED → NEW
blocking2.0: --- → ?
Ever confirmed: true

Updated

6 years ago
OS: Windows 7 → All
Hardware: x86_64 → All

Updated

6 years ago
Whiteboard: [4b12]

Comment 2

6 years ago
Technically we're doing exactly what the button says even though some of your pages are not in your current view. I agree that we could do better here in conjunction with Panorama.

Unless it's blocking, I think this is a post-Fx4 issue.
Priority: -- → P3
Target Milestone: --- → Future
Yeah, not a blocker.
blocking2.0: ? → -

Comment 4

6 years ago
I have an issue that may be related.  When I choose Use Current Pages, Firefox adds a spam ISP site and a blank Tab, neither of which is a current page when I choose Use Current Pages.

Below is what was saved:
http://mail.derekwilliams.net/zimbra/#7|http://community.vsl.co.at/forums/|http://www.facebook.com/|http://commonsenseatheism.com/?p=15167|http://by148w.bay148.mail.live.com/default.aspx?wa=wsignin1.0|https://mail.google.com/mail/?hl=en&shva=1#inbox|http://extranet.inpho.com.au/livr/dw/dw_status.php?action=line_status|http://timeanddate.com/worldclock/|http://translate.google.com/#auto|en||http://dictionary.reference.com/

I am attaching an image of what opens.  I contacted their Abuse team, to no avail.

Comment 5

6 years ago
Created attachment 529113 [details]
screenshot showing unsolicited spam tab from spam ISP when Use Current Pages selected

screenshot showing unsolicited spam tab from spam ISP when Use Current Pages selected
(Assignee)

Updated

6 years ago
Blocks: 653099
Version: unspecified → Trunk
(Assignee)

Comment 6

6 years ago
Created attachment 529276 [details] [diff] [review]
patch v1
Assignee: nobody → tim.taubert
Status: NEW → ASSIGNED
Attachment #529276 - Flags: review?(gavin.sharp)
Comment on attachment 529276 [details] [diff] [review]
patch v1

While you're at it, you could update the XXX comment to be a "FIXME: Bug 244192".

You could also replace all of this code with:

var tabs = win.gBrowser.visibleTabs;
function getTabURI(t) t.linkedBrowser.currentURI.spec;
var newVal = tabs.map(getTabURI).join("|");
Attachment #529276 - Flags: review?(gavin.sharp) → review+
(Assignee)

Comment 8

6 years ago
Created attachment 529611 [details] [diff] [review]
patch for checkin

(In reply to comment #7)
> While you're at it, you could update the XXX comment to be a "FIXME: Bug
> 244192".

Done.

> You could also replace all of this code with:
> 
> var tabs = win.gBrowser.visibleTabs;
> function getTabURI(t) t.linkedBrowser.currentURI.spec;
> var newVal = tabs.map(getTabURI).join("|");

That's much better :) Done.
Attachment #529276 - Attachment is obsolete: true
(Assignee)

Updated

6 years ago
Keywords: checkin-needed
http://hg.mozilla.org/projects/cedar/rev/ec5747dbbfed
Keywords: checkin-needed
Whiteboard: [4b12] → [4b12][fixed-in-cedar]
Target Milestone: Future → Firefox 6
http://hg.mozilla.org/mozilla-central/rev/ec5747dbbfed
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Flags: in-testsuite?
Resolution: --- → FIXED
Whiteboard: [4b12][fixed-in-cedar] → [4b12]

Comment 11

6 years ago
 Mozilla/5.0 (X11; Linux i686; rv:6.0a1) Gecko/20110519 Firefox/6.0a1

Verified issue using steps from Comment 0 on Ubuntu 10.10, WinXP, Mac OS X 10.6, Win 7. Bug is no longer present.
Status: RESOLVED → VERIFIED
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.