Mirrors/uBlock
1
0
Fork 0
mirror of https://github.com/gorhill/uBlock.git synced 2024-07-08 04:49:12 +02:00
Code Issues Releases Wiki Activity

minor code review: move vars closer to their 1st use + reference links

Browse Source
This commit is contained in:
gorhill 2015-12-06 16:43:58 -05:00
parent 982748f71e
commit 2af72459fe
1 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
platform/firefox/vapi-background.js
View File

@ -2689,10 +2689,10 @@ vAPI.toolbarButton = {
return;
}
var palette = toolbox.palette;
var navbar = document.getElementById('nav-bar');
var toolbarButton = createToolbarButton(window);
// https://developer.mozilla.org/en-US/docs/Mozilla/Tech/XUL/toolbarpalette
var palette = toolbox.palette;
if ( palette && palette.querySelector('#' + tbb.id) === null ) {
palette.appendChild(toolbarButton);
}
@ -2723,6 +2723,7 @@ vAPI.toolbarButton = {
break;
}
}
// https://developer.mozilla.org/en-US/docs/Mozilla/Tech/XUL/Method/insertItem
toolbar.insertItem(tbb.id, before);
break;
}
@ -2734,6 +2735,7 @@ vAPI.toolbarButton = {
// No button yet so give it a default location. If forcing the button,
// just put in in the palette rather than on any specific toolbar (who
// knows what toolbars will be available or visible!)
var navbar = document.getElementById('nav-bar');
if ( navbar !== null && !vAPI.localStorage.getBool('legacyToolbarButtonAdded') ) {
// https://github.com/gorhill/uBlock/issues/264
// Find a child customizable palette, if any.