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:
Created Aug. 22, 2017, 12:02 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with comments
|
« no previous file
|
options.js » ('j')
|
no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | 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 |
| +{ |
| + font-size: 120%; |
|
Thomas Greiner
2017/08/22 17:13:25
Coding style: "CSS rule declaration order should f
Wladimir Palant
2017/08/22 18:44:06
Not very specific but I tried my best...
|
| + border: 1px solid #cccccc; |
| + border-radius: 3px; |
| + background-color: #ffcccc; |
| + margin-bottom: 20px; |
| + padding: 10px; |
| +} |
| + |
| #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">We neither</a>. |
|
Thomas Greiner
2017/08/22 17:13:25
I assume that we decided against using documentati
Thomas Greiner
2017/08/22 17:13:25
Detail: All existing external links open in a sepa
Wladimir Palant
2017/08/22 18:44:06
Yes, this is a temporary thing.
On 2017/08/22 17:
|
| + We will improve this page soon. |
|
Thomas Greiner
2017/08/22 17:13:25
It's understandable that this doesn't need to be t
Wladimir Palant
2017/08/22 18:44:06
We simply cannot translate it, by the time we'll h
|
| +</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 »
