Closed
Bug 457096
Opened 16 years ago
Closed 16 years ago
When closing the last tab, two tabs are briefly visible (browser.tabs.closeWindowWithLastTab = false)
Categories
(Firefox :: Tabbed Browser, defect)
Tracking
()
VERIFIED
FIXED
Firefox 3.6a1
People
(Reporter: Portfolioso, Assigned: dao)
References
Details
(Keywords: polish, verified1.9.1, Whiteboard: [polish-easy] [polish-interactive][polish-p1])
Attachments
(2 files, 1 obsolete file)
(deleted),
patch
|
Gavin
:
review+
beltzner
:
approval1.9.1+
|
Details | Diff | Splinter Review |
(deleted),
image/png
|
Details |
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1b1pre) Gecko/20080925033548 Minefield/3.1b1pre
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1b1pre) Gecko/20080925033548 Minefield/3.1b1pre
When browser.tabs.closeWindowWithLastTab is set to False and the last tab is closed, another tab briefly appears then disappears. This shouldn't happen
Reproducible: Always
Steps to Reproduce:
1. Set browser.tabs.closeWindowWithLastTab to False
2. Open a webpage in one tab
3. Middle click the tab to close it
Actual Results:
Another tab briefly appears then disappears. This shouldn't happen.
Expected Results:
The tab should close the page and revert to a blank tab without creating a second tab and having that disappear.
Assignee | ||
Updated•16 years ago
|
Status: UNCONFIRMED → NEW
Ever confirmed: true
Assignee | ||
Updated•16 years ago
|
Assignee: nobody → dao
I can confirm this using Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1b1pre) Gecko/20080925033548 Minefield/3.1b1pre
Assignee | ||
Comment 2•16 years ago
|
||
Attachment #344782 -
Flags: review?(gavin.sharp)
Assignee | ||
Updated•16 years ago
|
Bug is still there on the latest trunk/branch nightlies:
1) Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.2a1pre) Gecko/20090130 Minefield/3.2a1pre
2) Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1b3pre) Gecko/20090130 Shiretoko/3.1b3pre
Is it going to be fixed in upcoming Firefox 3.1 release?
Assignee | ||
Comment 5•16 years ago
|
||
now without hacking addTab
Attachment #344782 -
Attachment is obsolete: true
Attachment #369064 -
Flags: review?(gavin.sharp)
Attachment #344782 -
Flags: review?(gavin.sharp)
Assignee | ||
Updated•16 years ago
|
Keywords: polish
OS: Windows XP → All
Hardware: x86 → All
Summary: Strange Behavior when Closing Last Tab (browser.tabs.closeWindowWithLastTab set to False) → When closing the last tab, two tabs are briefly visible (browser.tabs.closeWindowWithLastTab = false)
Whiteboard: [polish-easy] [polish-interactive]
Version: unspecified → 3.1 Branch
Updated•16 years ago
|
Attachment #369064 -
Flags: review?(gavin.sharp) → review+
Assignee | ||
Comment 6•16 years ago
|
||
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 3.6a1
Assignee | ||
Updated•16 years ago
|
Attachment #369064 -
Flags: approval1.9.1?
Comment 7•16 years ago
|
||
With this patch, if you close the last tab I can see the close-button on the empty tab for a split second. Seeing this on vista. Would be nice if the X wouldn't appear.
Assignee | ||
Comment 8•16 years ago
|
||
(In reply to comment #7)
> With this patch, if you close the last tab I can see the close-button on the
> empty tab for a split second. Seeing this on vista. Would be nice if the X
> wouldn't appear.
Please file a new bug.
Comment 9•16 years ago
|
||
Filed bug #486280.
Comment 10•16 years ago
|
||
Tab B (Grey) is actually collapsed.
The image is only for reference.
I filed Bug 486129.
The problem is a time lag (3).
Now, the patch v2 causes serious regression.
This patch break many third party themes.
And, this patch greatly limits the tab design of the third party theme.
https://bug486129.bugzilla.mozilla.org/attachment.cgi?id=370202
Piro told me that the more large patch is necessary to fix Bug 486129 and Bug 486280.
Comment 11•16 years ago
|
||
Comment on attachment 369064 [details] [diff] [review]
patch v2
I think we need to wait for the followup patch from bug 486280 before we can land this. Please feel free to renominate then.
Attachment #369064 -
Flags: approval1.9.1? → approval1.9.1-
Assignee | ||
Updated•16 years ago
|
Attachment #369064 -
Flags: approval1.9.1- → approval1.9.1?
Comment 12•16 years ago
|
||
This is not fixed in Firefox 3.5b4
Comment 13•16 years ago
|
||
That's because it hasn't been checked into that branch at this time (you can see that the patch attached to this bug is still pending approval). You can verify that it's fixed with a recent mozilla-central nightly (version 3.6a1pre).
Comment 14•16 years ago
|
||
Comment on attachment 369064 [details] [diff] [review]
patch v2
a191=beltzner, please land with bug 486280
Updated•16 years ago
|
Attachment #369064 -
Flags: approval1.9.1? → approval1.9.1+
Assignee | ||
Updated•16 years ago
|
Keywords: checkin-needed
Assignee | ||
Comment 15•16 years ago
|
||
Keywords: checkin-needed → fixed1.9.1
Comment 16•16 years ago
|
||
This checkin was in a range identified with a Ts Shutdown regression on Windows:
Regression: Ts Shutdown increase 27.64% on WINNT 5.1 Firefox3.5
Previous results:
357.263 from build 20090506145316 of revision 486b76052a94 at 2009-05-06 14:53:00
New results:
456.0 from build 20090506155401 of revision 7aa4483585bd at 2009-05-06 15:54:00
http://graphs-new.mozilla.org/graph.html#tests=[{"machine":32,"test":36,"branch":3},{"machine":33,"test":36,"branch":3},{"machine":34,"test":36,"branch":3},{"machine":35,"test":36,"branch":3},{"machine":48,"test":36,"branch":3}]&sel=1241564040,1241736840
Comment 17•16 years ago
|
||
verified FIXED on builds:
Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.2a1pre) Gecko/20090527 Minefield/3.6a1pre ID:20090527031500
and
Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.1pre) Gecko/20090527 Shiretoko/3.5pre ID:20090527031214
Status: RESOLVED → VERIFIED
Keywords: fixed1.9.1 → verified1.9.1
Comment 18•15 years ago
|
||
In Firefox3.5b99 an X is still briefly visible when closing the last tab. Additionally if you undo the last closed tab a second tab briefly appears. Unsure if I should open a new bug for this or not.
Comment 19•15 years ago
|
||
(In reply to comment #18)
That's bug 490166.
Comment 20•15 years ago
|
||
Er, the "X is still briefly visible when closing the last tab" is bug 490166.
The "Undo Close Tab with a last blank tab briefly shows 2 tabs" part is bug 458011.
Comment 21•15 years ago
|
||
This bug's priority relative to the set of other polish bugs is:
P1 - Polish issue that appears in the main window, or is something that the user may encounter several times a day.
nice diagram :)
Whiteboard: [polish-easy] [polish-interactive] → [polish-easy] [polish-interactive][polish-p1]
You need to log in
before you can comment on or make changes to this bug.
Description
•