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

Unified Diff: html/templates/index.html

Issue 29329035: Issue 2975 - add spaces after colons on the settings page. (Closed)
Patch Set: Created Oct. 12, 2015, 10:07 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: html/templates/index.html
diff --git a/html/templates/index.html b/html/templates/index.html
index 3b24ac4a9830b39b15cfdf59beea76fc4d483d72..06fa943d1d1a09686fc8807a040f7ded98b06490 100644
--- a/html/templates/index.html
+++ b/html/templates/index.html
@@ -30,7 +30,7 @@
<div id="feature-language" class="feature-image"></div>
<div class="feature-description">
<p class="feature-text">
- <span id="localeLanguageLabel" class="feature-title">Language:</span><span class="feature-title"> </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>
Eric 2015/10/13 17:08:10 It seems you have source and target versions swapp
Oleksandr 2015/10/15 11:04:06 I think using &nbsp; instead of a space would be b
sergei 2015/10/15 11:15:52 Done.
+ <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>
@@ -44,7 +44,7 @@
<div id="feature-exceptions" class="feature-image"></div>
<div class="feature-description">
<p class="feature-text">
- <span id="localeBlockingLabel" class="feature-title">Manage Blocking:</span><span class="feature-title"> </span><span id="localeBlockingDescription" class="feature-title">You can disable adblocking for some websites. Manage your list of websites for which you want to see ads.</span>
+ <span id="localeBlockingLabel" class="feature-title">Manage Blocking:</span><span id="localeBlockingDescription" class="feature-title">You can disable adblocking for some websites. Manage your list of websites for which you want to see ads.</span>
</p>
<div id="exceptions">
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld