Unified Diff: desktop-options.html
Issue 29609587:
Issue 6031 - Implement Acceptable Ads notification (Closed)
Patch Set:
Created Nov. 15, 2017, 5:01 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with comments
|
« background.js ('k')
|
desktop-options.js » ('j')
|
desktop-options.js » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: desktop-options.html |
| =================================================================== |
| --- a/desktop-options.html |
| +++ b/desktop-options.html |
| @@ -97,6 +97,15 @@ |
| <p class="i18n_options_acceptableAds_description"></p> |
| </header> |
| <form id="acceptable-ads"> |
| + <div id="tracking-notification"> |
|
Thomas Greiner
2017/11/16 19:25:05
Suggestion: The naming is inconsistent. Here are t
saroyanm
2017/11/17 16:10:38
Agree, done.
|
| + <button id="hide-tracking-notification" type="button" data-pref="ui_warn_tracking" data-action="toggle-pref,hide-tracking-notification"> |
| + <span class="i18n_options_notification_hide sr-only"></span> |
| + </button> |
| + <p class="i18n_options_tracking_notification_1"></p> |
| + <p class="i18n_options_tracking_notification_2"></p> |
| + <p class="i18n_options_tracking_notification_3"></p> |
| + <button class="i18n_options_tracking_notification_acknowledgment link" type="button" data-pref="ui_warn_tracking" data-action="toggle-pref,hide-tracking-notification"></button> |
| + </div> |
| <ul> |
| <li> |
| <button id="acceptable-ads-allow" data-action="switch-acceptable-ads" type="button" role="checkbox" name="acceptable-ads" value="ads"></button> |
« no previous file with comments
|
« background.js ('k')
|
desktop-options.js » ('j')
|
desktop-options.js » ('J')
