Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: lib/appSupport.js

Issue 5741004535627776: Fix toolbar icon customization in Australis (Closed)
Patch Set: Fixed unintentional code duplication Created Nov. 22, 2013, 5:11 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/locale/en-US/overlay.dtd ('k') | lib/customizableUI.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: lib/appSupport.js
===================================================================
--- a/lib/appSupport.js
+++ b/lib/appSupport.js
@@ -61,19 +61,18 @@ exports.getCurrentLocation = function ge
/**
* The ID (or a list of possible IDs) of the content area context menu.
* @type String|String[]
*/
exports.contentContextMenu = null;
/**
- * The properties parent, before, after determine the default placement of the
- * toolbar icon, the property isAddonBar indicates that it is an add-on bar
- * (different options text is displayed then).
+ * Determines the default placement of the toolbar icon via object properties
+ * parent, before and after.
* @type Object
*/
exports.defaultToolbarPosition = null;
/**
* The properties parent, before, after determine the placement of the status
* bar icon.
* @type Object
@@ -272,19 +271,17 @@ switch (application)
window.openNewTabWith(url, exports.getBrowser(window).contentDocument, null, event, false);
else
window.gBrowser.loadOneTab(url, {inBackground: false});
};
exports.contentContextMenu = "contentAreaContextMenu";
exports.defaultToolbarPosition = {
- parent: "addon-bar",
- before: "addonbar-closebutton",
- isAddonBar: true
+ parent: "nav-bar"
};
exports.toolsMenu = {
parent: "menu_ToolsPopup"
};
exports.addBottomBar = function fx_addBottomBar(window, element)
{
« no previous file with comments | « chrome/locale/en-US/overlay.dtd ('k') | lib/customizableUI.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld