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

Unified Diff: options.html

Issue 8432019: Added acceptable ads list to the options UI (Closed)
Patch Set: Created Sept. 24, 2012, 7:52 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
Index: options.html
===================================================================
--- a/options.html
+++ b/options.html
@@ -34,16 +34,27 @@ button
#subscriptionTemplate,
#addSubscriptionContainer,
#customSubscriptionContainer
{
display: none;
}
+#acceptableAdsContainer
+{
+ margin-top: 10px;
+}
Felix Dahlke 2012/09/25 12:51:55 How about a newline here? I know, picky :)
+#acceptableAdsLink,
+#acceptableAdsDocs
+{
+ margin-left: 10px;
+ margin-right: 10px;
+}
+
.subscription
{
margin: 3px 0px;
}
.subscriptionRemoveButton,
.subscriptionEnabledContainer,
.subscriptionTitle
@@ -156,16 +167,23 @@ button
<span class="i18n_subscriptionSelection_title_label"></span><br />
<input type="text" id="customSubscriptionTitle" style="width:500px" />
</p>
<p>
<span class="i18n_subscriptionSelection_location_label"></span><br />
<input type="text" id="customSubscriptionLocation" style="width:500px" />
</p>
</div>
+
+ <div id="acceptableAdsContainer">
+ <input type="checkbox" id="acceptableAds" />
+ <span class="i18n_filters_acceptableAds2_label"></span>
+ <a id="acceptableAdsLink" href="#" class="i18n_filters_viewList_label"></a>
+ <a id="acceptableAdsDocs" href="#" class="i18n_filters_readMore_label"></a>
+ </div>
</p>
</div>
<div id="tab-whitelisted">
<p class="i18n_disabled_for_these_domains"></p>
<form id="whitelistForm" action="#">
<table>
« _locales/hr/messages.json ('K') | « lib/adblockplus_compat.js ('k') | options.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld