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

Side by Side Diff: options.html

Issue 29523647: Issue 5555 - [webextensions] Add a message for Firefox users on the options page (Closed) Base URL: https://hg.adblockplus.org/adblockpluschrome
Patch Set: Adjusted wording as suggested by Lisa Created Aug. 22, 2017, 7:06 p.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | options.js » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 <!DOCTYPE html> 1 <!DOCTYPE html>
2 2
3 <!-- 3 <!--
4 - This file is part of Adblock Plus <https://adblockplus.org/>, 4 - This file is part of Adblock Plus <https://adblockplus.org/>,
5 - Copyright (C) 2006-present eyeo GmbH 5 - Copyright (C) 2006-present 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 35 matching lines...) Expand 10 before | Expand all | Expand 10 after
46 p 46 p
47 { 47 {
48 margin-top: 0.5em; 48 margin-top: 0.5em;
49 } 49 }
50 50
51 button 51 button
52 { 52 {
53 white-space: pre; 53 white-space: pre;
54 } 54 }
55 55
56 #firefox-warning
57 {
58 margin-bottom: 20px;
59 padding: 10px;
60 border: 1px solid #cccccc;
61 border-radius: 3px;
62 background-color: #ffcccc;
63 font-size: 120%;
64 }
65
56 #subscriptionTemplate, 66 #subscriptionTemplate,
57 #addSubscriptionContainer, 67 #addSubscriptionContainer,
58 #customSubscriptionContainer 68 #customSubscriptionContainer
59 { 69 {
60 display: none; 70 display: none;
61 } 71 }
62 72
63 #acceptableAdsContainer 73 #acceptableAdsContainer
64 { 74 {
65 margin-top: 10px; 75 margin-top: 10px;
(...skipping 71 matching lines...) Expand 10 before | Expand all | Expand 10 after
137 <tr> 147 <tr>
138 <td style="padding-right: 8px; vertical-align: middle;"> 148 <td style="padding-right: 8px; vertical-align: middle;">
139 <img src="icons/detailed/abp-32.png" srcset="icons/detailed/abp-64.png 2x" > 149 <img src="icons/detailed/abp-32.png" srcset="icons/detailed/abp-64.png 2x" >
140 </td> 150 </td>
141 <td> 151 <td>
142 <h1><span class="i18n_options"></span></h1> 152 <h1><span class="i18n_options"></span></h1>
143 </td> 153 </td>
144 </tr> 154 </tr>
145 </table> 155 </table>
146 156
157 <div id="firefox-warning" hidden>
158 Don't like what you see?
159 <a href="https://adblockplus.org/blog/the-plan-towards-offering-adblock-plus- for-firefox-as-a-web-extension" target="_blank">​We neither</a>.
hub 2017/08/23 12:45:52 Lisa wrote "Neither do we." https://issues.adblo
Wladimir Palant 2017/08/23 13:30:46 Right, fixed that.
160 This page will be improved soon.
161 </div>
162
147 <div id="tabs"> 163 <div id="tabs">
148 <ul> 164 <ul>
149 <li><a href="#tab-filterlists"><span class="i18n_filter_lists"></span></a></ li> 165 <li><a href="#tab-filterlists"><span class="i18n_filter_lists"></span></a></ li>
150 <li><a href="#tab-userfilters"><span class="i18n_add_your_own_filters"></spa n></a></li> 166 <li><a href="#tab-userfilters"><span class="i18n_add_your_own_filters"></spa n></a></li>
151 <li><a href="#tab-whitelisted"><span class="i18n_whitelisted_domains"></span ></a></li> 167 <li><a href="#tab-whitelisted"><span class="i18n_whitelisted_domains"></span ></a></li>
152 <li><a href="#tab-general"><span class="i18n_general"></span></a></li> 168 <li><a href="#tab-general"><span class="i18n_general"></span></a></li>
153 </ul> 169 </ul>
154 170
155 <div id="tab-filterlists"> 171 <div id="tab-filterlists">
156 <p> 172 <p>
(...skipping 126 matching lines...) Expand 10 before | Expand all | Expand 10 after
283 <input type="checkbox" id="shouldShowNotifications" /><label for="should ShowNotifications" class="i18n_overlay_shownotifications_label"></label> 299 <input type="checkbox" id="shouldShowNotifications" /><label for="should ShowNotifications" class="i18n_overlay_shownotifications_label"></label>
284 </div> 300 </div>
285 </p> 301 </p>
286 302
287 <p id="found-a-bug" class="i18n_found_a_bug"></p> 303 <p id="found-a-bug" class="i18n_found_a_bug"></p>
288 </div> 304 </div>
289 </div> 305 </div>
290 306
291 </body> 307 </body>
292 </html> 308 </html>
OLDNEW
« no previous file with comments | « no previous file | options.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld