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

Unified 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.
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
« no previous file with comments | « no previous file | options.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: options.html
===================================================================
--- a/options.html
+++ b/options.html
@@ -48,16 +48,26 @@ p
margin-top: 0.5em;
}
button
{
white-space: pre;
}
+#firefox-warning
+{
+ margin-bottom: 20px;
+ padding: 10px;
+ border: 1px solid #cccccc;
+ border-radius: 3px;
+ background-color: #ffcccc;
+ font-size: 120%;
+}
+
#subscriptionTemplate,
#addSubscriptionContainer,
#customSubscriptionContainer
{
display: none;
}
#acceptableAdsContainer
@@ -139,16 +149,22 @@ button
<img src="icons/detailed/abp-32.png" srcset="icons/detailed/abp-64.png 2x">
</td>
<td>
<h1><span class="i18n_options"></span></h1>
</td>
</tr>
</table>
+<div id="firefox-warning" hidden>
+ Don't like what you see?
+ <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.
+ This page will be improved soon.
+</div>
+
<div id="tabs">
<ul>
<li><a href="#tab-filterlists"><span class="i18n_filter_lists"></span></a></li>
<li><a href="#tab-userfilters"><span class="i18n_add_your_own_filters"></span></a></li>
<li><a href="#tab-whitelisted"><span class="i18n_whitelisted_domains"></span></a></li>
<li><a href="#tab-general"><span class="i18n_general"></span></a></li>
</ul>
« no previous file with comments | « no previous file | options.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld