Closed
Bug 636149
Opened 14 years ago
Closed 13 years ago
Home Page -> Use Current Pages; Does not respect Tab groups
Categories
(Firefox Graveyard :: Panorama, defect, P3)
Firefox Graveyard
Panorama
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)
(deleted),
image/png
|
Details | |
(deleted),
patch
|
Details | Diff | Splinter Review |
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
Comment 1•14 years ago
|
||
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•14 years ago
|
OS: Windows 7 → All
Hardware: x86_64 → All
Updated•14 years ago
|
Whiteboard: [4b12]
Comment 2•14 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
Comment 4•13 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•13 years ago
|
||
screenshot showing unsolicited spam tab from spam ISP when Use Current Pages selected
Assignee | ||
Comment 6•13 years ago
|
||
Assignee: nobody → tim.taubert
Status: NEW → ASSIGNED
Attachment #529276 -
Flags: review?(gavin.sharp)
Comment 7•13 years ago
|
||
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•13 years ago
|
||
(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•13 years ago
|
Keywords: checkin-needed
Comment 9•13 years ago
|
||
Keywords: checkin-needed
Whiteboard: [4b12] → [4b12][fixed-in-cedar]
Target Milestone: Future → Firefox 6
Comment 10•13 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Flags: in-testsuite?
Resolution: --- → FIXED
Whiteboard: [4b12][fixed-in-cedar] → [4b12]
Comment 11•13 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
Updated•9 years ago
|
Product: Firefox → Firefox Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•