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: Added 32x32 icon and fixed my own review comments Created Dec. 2, 2013, 10:55 a.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
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"
tschuster 2014/05/15 20:12:30 The new api calls this "area", maybe switch to tha
Wladimir Palant 2014/07/11 07:15:54 I don't really feel like renaming everything whene
};
exports.toolsMenu = {
parent: "menu_ToolsPopup"
};
exports.addBottomBar = function fx_addBottomBar(window, element)
{

Powered by Google App Engine
This is Rietveld