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

Unified Diff: new-options.html

Issue 29519669: Issue 5539 - Implement "Acceptable Ads notification" (Closed)
Patch Set: Rebased Created Sept. 14, 2017, 2:29 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 | « locale/en-US/new-options.json ('k') | new-options.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: new-options.html
===================================================================
--- a/new-options.html
+++ b/new-options.html
@@ -336,6 +336,7 @@
<h3 id="dialog-title-language-add" class="i18n_options_dialog_language_title"></h3>
<h3 id="dialog-title-language-change" class="i18n_options_dialog_language_title"></h3>
<h3 id="dialog-title-predefined" class="i18n_options_dialog_predefined_title"></h3>
+ <span id="dialog-title-tracking" class="i18n_options_dialog_tracking_title"></span>
saroyanm 2017/09/14 14:31:18 This needs to be switched to h3
saroyanm 2017/09/14 21:56:14 Done.
</span>
<button id="dialog-close" data-action="close-dialog"></button>
</header>
@@ -380,6 +381,26 @@
<button class="i18n_options_dialog_predefined_confirm default-focus primary" data-action="add-predefined-subscription"></button>
</div>
</div>
+ <!-- Acceptable Ads tracking notification -->
+ <div id="dialog-content-tracking" class="dialog-content">
+ <div class="dialog-content-block">
+ <p class="i18n_options_dialog_tracking_notification"></p>
+ <p>
+ <button class="i18n_options_dialog_tracking_relevent_ads" data-action="close-dialog"></button>
+ </p>
+ <p>
+ <button class="i18n_options_dialog_tracking_privacy_ads" data-action="close-dialog"></button>
+ </p>
+ <p>
+ <h3 class="i18n_options_dialog_tracking_note"></h3>
+ <ul>
+ <li class="i18n_options_dialog_tracking_note_1"></li>
+ <li class="i18n_options_dialog_tracking_note_2"></li>
+ <li class="i18n_options_dialog_tracking_note_3"></li>
+ </ul>
+ </p>
+ </div>
+ </div>
</div>
<!-- Placeholder element to determine when to wrap focus around -->
<span class="focus-last" tabindex="0"></span>
« no previous file with comments | « locale/en-US/new-options.json ('k') | new-options.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld