superreview granted: [Bug 156082] Don't hide the tab bar when clicking the close box (pref for showing single tab ['Hide tab bar when only one tab is open'] is ignored) : [Attachment 233414] blank las

neil@parkwaycc.co.uk  has granted Chris Thomas (CTho) [formerly
cst@andrew.cmu.edu] 's request for superreview:
Bug 156082: Don't hide the tab bar when clicking the close box (pref for
showing single tab ["Hide tab bar when only one tab is open"] is ignored)
https://bugzilla.mozilla.org/show_bug.cgi?id=156082

Attachment 233414: blank last tab
https://bugzilla.mozilla.org/attachment.cgi?id=233414&action=edit

------- Additional Comments from neil@parkwaycc.co.uk 
This will cause me dataloss, because I'm used to X hiding the tab bar...

>	      var l = this.mTabs.length;
>-	      if (l == 1) {
>-		// hide the tab bar
>-		this.mPrefs.setBoolPref("browser.tabs.forceHide", true);
>-		this.mStrip.collapsed = true;
>-		return;
>-	      }
Move the var l to where you use it. sr=me with this fixed.

0 bugzilla 8/16/2006 2:42:04 PM