Bug 524469: Opening enough tabs to create additional columns cuts off the initial column [r=mark.finkle]

This commit is contained in:
Vivien Nicolas 2009-12-01 23:20:58 -05:00
Родитель 66d7743a64
Коммит 4643ac9cf0
1 изменённых файлов: 12 добавлений и 6 удалений

Просмотреть файл

@ -653,10 +653,18 @@ var BrowserUI = {
this._tabSelect(aEvent); this._tabSelect(aEvent);
break; break;
case "TabOpen": case "TabOpen":
if (!this.isTabsVisible() && {
Browser.selectedTab.chromeTab != aEvent.target) if (!this.isTabsVisible() && Browser.selectedTab.chromeTab != aEvent.target)
NewTabPopup.show(aEvent.target); NewTabPopup.show(aEvent.target);
// Workaround to hide the tabstrip if it is partially visible
// See bug 524469
let [tabstripV,,,] = Browser.computeSidebarVisibility();
if (tabstripV > 0 && tabstripV < 1)
Browser.hideSidebars();
break; break;
}
// URL textbox events // URL textbox events
case "click": case "click":
this.doCommand("cmd_openLocation"); this.doCommand("cmd_openLocation");
@ -837,6 +845,8 @@ var NewTabPopup = {
}, },
show: function(aTab) { show: function(aTab) {
BrowserUI.pushPopup(this, this.box);
this._tabs.push(aTab); this._tabs.push(aTab);
this._updateLabel(); this._updateLabel();
@ -849,8 +859,6 @@ var NewTabPopup = {
this._timeout = setTimeout(function(self) { this._timeout = setTimeout(function(self) {
self.hide(); self.hide();
}, 2000, this); }, 2000, this);
BrowserUI.pushPopup(this, this.box);
}, },
selectTab: function() { selectTab: function() {
@ -882,8 +890,6 @@ var BookmarkPopup = {
this.box.hidden = false; this.box.hidden = false;
let [,,,controlsW] = Browser.computeSidebarVisibility(); let [,,,controlsW] = Browser.computeSidebarVisibility();
Components.reportError("bar: " + controlsW);
Components.reportError("box: " + this.box.getBoundingClientRect().width)
this.box.left = window.innerWidth - (this.box.getBoundingClientRect().width + controlsW + margin); this.box.left = window.innerWidth - (this.box.getBoundingClientRect().width + controlsW + margin);
this.box.top = BrowserUI.starButton.getBoundingClientRect().top + margin; this.box.top = BrowserUI.starButton.getBoundingClientRect().top + margin;