Closed
Bug 486280
Opened 16 years ago
Closed 16 years ago
Several things briefly appear and disappear when closing the last tab (closeWindowWithLastTab=false)
Categories
(Firefox :: Tabbed Browser, defect)
Tracking
()
VERIFIED
FIXED
Firefox 3.6a1
People
(Reporter: albert.brand, Assigned: dao)
References
Details
(Keywords: polish, verified1.9.1, Whiteboard: [polish-easy] [polish-interactive][polish-p1])
Attachments
(1 file, 1 obsolete file)
(deleted),
patch
|
Gavin
:
review+
beltzner
:
approval1.9.1+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9.2a1pre) Gecko/20090331 Minefield/3.6a1pre
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9.2a1pre) Gecko/20090331 Minefield/3.6a1pre
Reproducible: Always
Steps to Reproduce:
Set browser.tabs.closeWindowWithLastTab to false and close the last tab with Ctrl-W or middleclick on the tab.
Actual Results:
Several things briefly appear and disappear.
1. The tab closes and disappears from the strip, which moves the 'new tab' button briefly to the outer left
2. An empty tab appears with the 'x' or close button briefly present, the new tab button moves back to its previous place
3. The close button disappears from the tab.
Expected Results:
The tab shouldn't disappear briefly from the strip, the new tab button shouldn't move and the close button shouldn't appear briefly.
Reporter | ||
Updated•16 years ago
|
Comment 1•16 years ago
|
||
The tab closing should probably be stopped entirely in this case. Confirmed.
Severity: trivial → normal
Status: UNCONFIRMED → NEW
Ever confirmed: true
Assignee | ||
Comment 2•16 years ago
|
||
Attachment #371109 -
Flags: review?(gavin.sharp)
Assignee | ||
Comment 3•16 years ago
|
||
err, "&& !aCloseWindow && !this._windowIsClosing" was superfluous.
Attachment #371109 -
Attachment is obsolete: true
Attachment #371110 -
Flags: review?(gavin.sharp)
Attachment #371109 -
Flags: review?(gavin.sharp)
Comment 4•16 years ago
|
||
(In reply to comment #3)
> Created an attachment (id=371110) [details]
> patch
This works fine.
Bug 486129 also fixed.
However, .tab-close-button is still displayed on the new blank tab momentarily.
Updated•16 years ago
|
Attachment #371110 -
Flags: review?(gavin.sharp) → review+
Assignee | ||
Updated•16 years ago
|
Keywords: checkin-needed
Assignee | ||
Comment 5•16 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/5f4cbbe377fc
(In reply to comment #4)
> However, .tab-close-button is still displayed on the new blank tab momentarily.
I have an idea how to fix that, but it won't happen in this bug. If you file a new one and cc me, I won't forget to look into that.
Assignee | ||
Updated•16 years ago
|
Status: NEW → RESOLVED
Closed: 16 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 3.6a1
Assignee | ||
Updated•16 years ago
|
Attachment #371110 -
Flags: approval1.9.1?
Comment 6•16 years ago
|
||
> I have an idea how to fix that, but it won't happen in this bug. If you file a
> new one and cc me, I won't forget to look into that.
OK. I filed Bug 490166.
Comment 7•16 years ago
|
||
Comment on attachment 371110 [details] [diff] [review]
patch
a191=beltzner, please land with bug 457096
Updated•16 years ago
|
Attachment #371110 -
Flags: approval1.9.1? → approval1.9.1+
Assignee | ||
Updated•16 years ago
|
Keywords: checkin-needed
Assignee | ||
Comment 8•16 years ago
|
||
Keywords: checkin-needed → fixed1.9.1
Comment 9•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 10•16 years ago
|
||
With Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.2a1pre) Gecko/20090525 Minefield/3.6a1pre (.NET CLR 3.5.30729) I still see flickering of the whole page when trying to close the last tab...
Comment 11•16 years ago
|
||
...Alfred is this working for you now? Because it worked for me on two OS's on Shiretoko as well as Trunk:
verified FIXED on Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9.1pre)
Gecko/20090526 Shiretoko/3.5pre (.NET CLR 3.5.30729) ID:20090526042535
and
Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9.2a1pre) Gecko/20090526 Minefield/3.6a1pre (.NET CLR 3.5.30729) ID:20090526044156
Same goes for the same build ID's on OS X 10.5.
I'll keep this in the same state until the end of this week if there's no answer.
Comment 12•16 years ago
|
||
Never mind, I misread comment #10, his complaint is being resolved on bug 490166. Moving the bug to verified FIXED.
Status: RESOLVED → VERIFIED
Keywords: fixed1.9.1 → verified1.9.1
Comment 13•16 years ago
|
||
As long as the 'closing the last tab (closeWindowWithLastTab=false)' on empty page doesn't do nothing, but keeps on killing the tab and then recreating a new empty one, UI elements (not just the close button) will keep on flashing (in the default theme, and custom themes).
I agree that bug 490166 is very similar, so one could argue one is a DUP of another, or that bug 490166 is more specific, just pointing to one flickering UI element, and this bug is about all flickering...
I would propose to keep bug 490166 for the closebutton with its specific patch to just prevent the closebutton to be momentarily visible, and this bug for the whole unwanted behaviour of closing the last empty tab, and then recreating a new one. It should just keep the last empty tab open.
Assignee | ||
Comment 14•16 years ago
|
||
The last /empty/ tab is a special case, and not what this bug is about. In general, the last tab /should/ be replaced by a blank one, given closeWindowWithLastTab=false.
Comment 15•15 years ago
|
||
This bug is not fixed on my mac system, which has:
browser.tabs.closeWindowWithLastTab-false
browser.tabs.closeButtons=3
Clicking on the "Close Tab" icon when there's a single tab (empty or not)
causes a temporary display of a second tab (labeled "Untitled") before leaving
a single Untitled tab.
The whole idea of "Close Tab" being promoted to "Close Window" is foolish.
I couldn't believe it was intentional until someone wrote that it's copied
from Microsoft.
Comment 16•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.
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
•