Bug 895921 - Defect - Tab bar not appearing when selecting links from flyouts.r=?

This commit is contained in:
Allison Naaktgeboren 2013-08-07 10:29:40 -07:00
parent 2102351272
commit 1e7daa1a28
3 changed files with 13 additions and 12 deletions

View File

@ -411,12 +411,21 @@ var BrowserUI = {
});
},
onAboutPolicyClick: function() {
FlyoutPanelsUI.hide();
let linkStr = Services.urlFormatter.formatURLPref("app.privacyURL");
BrowserUI.newTab(linkStr, Browser.selectedTab, true);
},
/*********************************
* Tab management
*/
newTab: function newTab(aURI, aOwner) {
newTab: function newTab(aURI, aOwner, aPeekTabs) {
aURI = aURI || kStartOverlayURI;
if (aPeekTabs) {
ContextUI.peekTabs(kNewTabAnimationDelayMsec);
}
let tab = Browser.addTab(aURI, true, aOwner);
return tab;
},
@ -1064,9 +1073,8 @@ var BrowserUI = {
this._closeOrQuit();
break;
case "cmd_newTab":
this.newTab();
this.newTab(null, null, true);
this._edit.beginEditing(false);
ContextUI.peekTabs(kNewTabAnimationDelayMsec);
break;
case "cmd_closeTab":
this.closeTab();
@ -1442,7 +1450,7 @@ var SettingsCharm = {
label: Strings.browser.GetStringFromName("helpOnlineCharm"),
onselected: function() {
let url = Services.urlFormatter.formatURLPref("app.support.baseURL");
BrowserUI.newTab(url, Browser.selectedTab);
BrowserUI.newTab(url, Browser.selectedTab, true);
}
});
},

View File

@ -1039,13 +1039,6 @@ var Browser = {
break;
}
},
onAboutPolicyClick: function() {
FlyoutPanelsUI.hide();
let linkStr = Services.urlFormatter.formatURLPref("app.privacyURL");
BrowserUI.newTab(linkStr, Browser.selectedTab);
}
};
Browser.MainDragger = function MainDragger() {

View File

@ -465,7 +465,7 @@
#endif
#endif
<label id="about-policy-label"
onclick="if (event.button == 0) { Browser.onAboutPolicyClick(); }"
onclick="if (event.button == 0) { BrowserUI.onAboutPolicyClick(); }"
class="text-link" value="&aboutHeader.policy.label;"/>
</flyoutpanel>