Closed Bug 636149 Opened 11 years ago Closed 10 years ago

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

Categories

(Firefox Graveyard :: Panorama, defect, P3)

defect

Tracking

(blocking2.0 -)

VERIFIED FIXED
Firefox 6
Tracking Status
blocking2.0 --- -

People

(Reporter: bugzilla, Assigned: ttaubert)

References

Details

(Whiteboard: [4b12])

Attachments

(2 files, 1 obsolete file)

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
OS: Windows 7 → All
Hardware: x86_64 → All
Whiteboard: [4b12]
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: ? → -
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.
screenshot showing unsolicited spam tab from spam ISP when Use Current Pages selected
Blocks: 653099
Version: unspecified → Trunk
Attached patch patch v1 (obsolete) — Splinter Review
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+
(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
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
Closed: 10 years ago
Flags: in-testsuite?
Resolution: --- → FIXED
Whiteboard: [4b12][fixed-in-cedar] → [4b12]
 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.