Issue 2975 - add spaces after colons on the settings page.

[Eric, 2015-10-13 17:08:10]

https://codereview.adblockplus.org/29329035/diff/29329036/html/templates/inde...
File html/templates/index.html (left):

https://codereview.adblockplus.org/29329035/diff/29329036/html/templates/inde...
html/templates/index.html:33: <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>
It seems you have source and target versions swapped, since this version has a
space and the other doesn't.

Is there really a reason to wrap the extra space in a <span> tag? Seems
extraneous.

[Oleksandr, 2015-10-15 11:04:06]

https://codereview.adblockplus.org/29329035/diff/29329036/html/templates/inde...
File html/templates/index.html (left):

https://codereview.adblockplus.org/29329035/diff/29329036/html/templates/inde...
html/templates/index.html:33: <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>
On 2015/10/13 17:08:10, Eric wrote:
> It seems you have source and target versions swapped, since this version has a
> space and the other doesn't.
> 
> Is there really a reason to wrap the extra space in a <span> tag? Seems
> extraneous.

I think using &nbsp; instead of a space would be better, if only for visual
clarity. And yes, the patch sets seem to be reversed. Otherwise makes sense.

[sergei, 2015-10-15 11:15:52]

https://codereview.adblockplus.org/29329035/diff/29329036/html/templates/inde...
File html/templates/index.html (left):

https://codereview.adblockplus.org/29329035/diff/29329036/html/templates/inde...
html/templates/index.html:33: <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>
On 2015/10/15 11:04:06, Oleksandr wrote:
> On 2015/10/13 17:08:10, Eric wrote:
> > It seems you have source and target versions swapped, since this version has
a
> > space and the other doesn't.
> > 
> > Is there really a reason to wrap the extra space in a <span> tag? Seems
> > extraneous.
> 
> I think using &nbsp; instead of a space would be better, if only for visual
> clarity. And yes, the patch sets seem to be reversed. Otherwise makes sense.

Done.

[Oleksandr, 2015-10-15 11:49:27]

LGTM

[Eric, 2015-10-15 15:34:18]

LGTM