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

Unified Diff: chrome/content/ui/settings.xul

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: chrome/content/ui/settings.xul
===================================================================
--- a/chrome/content/ui/settings.xul
+++ b/chrome/content/ui/settings.xul
@@ -27,12 +27,11 @@
<setting type="control">
<button id="adblockplus-filters" label="&filters.label;…"/>
</setting>
<setting pref="extensions.adblockplus.enabled" type="bool" inverted="true" title="&disable.label;"/>
<setting pref="extensions.adblockplus.frameobjects" type="bool" title="&objecttabs.label;"/>
<setting pref="extensions.adblockplus.fastcollapse" type="bool" inverted="true" title="&hideplaceholders.label;"/>
<setting id="adblockplus-savestats" type="bool" title="&counthits.label;"/>
<setting id="adblockplus-sync" type="bool" title="&sync.label;"/>
- <setting id="adblockplus-showinaddonbar" type="bool" title="&showinaddonbar.label;"/>
tschuster 2014/05/15 20:12:30 I wonder if we should keep this for all the people
Wladimir Palant 2014/07/11 07:15:54 I don't think we can support the various customiza
<setting id="adblockplus-showintoolbar" type="bool" title="&showintoolbar.label;"/>
<setting id="adblockplus-showinstatusbar" pref="extensions.adblockplus.showinstatusbar" type="bool" title="&showinstatusbar.label;"/>
tschuster 2014/05/15 20:12:30 Aside: Isn't the statusbar long gone?
Wladimir Palant 2014/07/11 07:15:54 In Firefox - yes. This option is only being shown
</window>

Powered by Google App Engine
This is Rietveld