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

Unified Diff: html/templates/index.html

Issue 5427527162003456: Issue 1086 - Add acceptable ads checkbox to the settings page (Closed)
Patch Set: Created July 21, 2014, 11:42 a.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
Index: html/templates/index.html
===================================================================
--- a/html/templates/index.html
+++ b/html/templates/index.html
@@ -32,7 +32,12 @@
<p class="feature-text">
<span id="localeLanguageLabel" class="feature-title">Language:</span><span id="localeLanguageDescription" class="feature-title">Select your language to get the most out of Adblock Plus so that it can get rid of all ads on the websites you visit.</span>
</p>
+ <p>
<select id="language"></select>
+ </p>
+ <p>
+ <input type="checkbox" id="acceptableAdsCheckbox" onchange="acceptableAdsChange(this)"/><label id="acceptableAdsLabel" for="acceptableAdsCheckbox">Enable some non-intrusive advertising</label>
Felix Dahlke 2014/07/21 13:15:37 onchange is not pretty, can you register the handl
+ </p>
</div>
</li>
<li class="feature">
« no previous file with comments | « html/static/js/IESettings.js ('k') | locales/en.ini » ('j') | src/plugin/PluginUserSettings.cpp » ('J')

Powered by Google App Engine
This is Rietveld