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

Side by Side 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.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View unified diff | Download patch
OLDNEW
1 <?xml version="1.0"?> 1 <?xml version="1.0"?>
2 2
3 <!-- 3 <!--
4 - This file is part of Adblock Plus <http://adblockplus.org/>, 4 - This file is part of Adblock Plus <http://adblockplus.org/>,
5 - Copyright (C) 2006-2013 Eyeo GmbH 5 - Copyright (C) 2006-2013 Eyeo GmbH
6 - 6 -
7 - Adblock Plus is free software: you can redistribute it and/or modify 7 - Adblock Plus is free software: you can redistribute it and/or modify
8 - it under the terms of the GNU General Public License version 3 as 8 - it under the terms of the GNU General Public License version 3 as
9 - published by the Free Software Foundation. 9 - published by the Free Software Foundation.
10 - 10 -
(...skipping 14 matching lines...) Expand all
25 height="1" 25 height="1"
26 onload="window.close();"> <!-- Close window if it gets opened directly for some reason --> 26 onload="window.close();"> <!-- Close window if it gets opened directly for some reason -->
27 <setting type="control"> 27 <setting type="control">
28 <button id="adblockplus-filters" label="&filters.label;…"/> 28 <button id="adblockplus-filters" label="&filters.label;…"/>
29 </setting> 29 </setting>
30 <setting pref="extensions.adblockplus.enabled" type="bool" inverted="true" tit le="&disable.label;"/> 30 <setting pref="extensions.adblockplus.enabled" type="bool" inverted="true" tit le="&disable.label;"/>
31 <setting pref="extensions.adblockplus.frameobjects" type="bool" title="&object tabs.label;"/> 31 <setting pref="extensions.adblockplus.frameobjects" type="bool" title="&object tabs.label;"/>
32 <setting pref="extensions.adblockplus.fastcollapse" type="bool" inverted="true " title="&hideplaceholders.label;"/> 32 <setting pref="extensions.adblockplus.fastcollapse" type="bool" inverted="true " title="&hideplaceholders.label;"/>
33 <setting id="adblockplus-savestats" type="bool" title="&counthits.label;"/> 33 <setting id="adblockplus-savestats" type="bool" title="&counthits.label;"/>
34 <setting id="adblockplus-sync" type="bool" title="&sync.label;"/> 34 <setting id="adblockplus-sync" type="bool" title="&sync.label;"/>
35 <setting id="adblockplus-showinaddonbar" type="bool" title="&showinaddonbar.la bel;"/>
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
36 <setting id="adblockplus-showintoolbar" type="bool" title="&showintoolbar.labe l;"/> 35 <setting id="adblockplus-showintoolbar" type="bool" title="&showintoolbar.labe l;"/>
37 <setting id="adblockplus-showinstatusbar" pref="extensions.adblockplus.showins tatusbar" type="bool" title="&showinstatusbar.label;"/> 36 <setting id="adblockplus-showinstatusbar" pref="extensions.adblockplus.showins tatusbar" 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
38 </window> 37 </window>
OLDNEW

Powered by Google App Engine
This is Rietveld