Bommels05/Mozilla
Bommels05/Mozilla
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix for bug #342105: Close button not shown if there is only one tab

Browse Source 
per beltzner:

"The close button should be shown, with the effect being that it disposes the
tab and immediately opens a new one. Strange behaviour surely, but I think it's
the easiest and most reliably understandable case."

thanks to Simon <zeniko@gmail.com> for the patch.

r=sspitzer, sr=mconnor


git-svn-id: svn://10.0.0.236/trunk@232753 18797224-902f-48f8-a5cc-f745e15eee43
This commit is contained in:
sspitzer%mozilla.org 2007-08-22 05:04:19 +00:00
parent a79c56d295
commit 7f2a788fb9
1 changed files with 9 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

25
mozilla/browser/base/content/tabbrowser.xml
View File

@ -2583,27 +2583,20 @@
// 3 - closeatend = close button at the end of the tabstrip
switch (this.mCloseButtons) {
case 0:
if (this.childNodes.length == 1)
this.setAttribute("closebuttons", "noclose");
else
this.setAttribute("closebuttons", "activetab");
this.setAttribute("closebuttons", "activetab");
break;
case 1:
try {
// if we have only one tab, hide the closebutton
if (this.childNodes.length == 1)
this.setAttribute("closebuttons", "noclose");
else {
var width = this.firstChild.boxObject.width;
// 0 width is an invalid value and indicates
// an item without display, so ignore.
if (width > this.mTabClipWidth || width == 0)
this.setAttribute("closebuttons", "alltabs");
else
this.setAttribute("closebuttons", "activetab");
}
var width = this.firstChild.boxObject.width;
// 0 width is an invalid value and indicates
// an item without display, so ignore.
if (width > this.mTabClipWidth || width == 0)
this.setAttribute("closebuttons", "alltabs");
else
this.setAttribute("closebuttons", "activetab");
}
catch (e) {
// XXXzeniko what error are we catching here?
}
break;
case 2: