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

Unified Diff: options.html

Issue 29339231: Issue 3882 - Make checkbox labels sensitive on the option page (Closed)
Patch Set: Created March 31, 2016, 4:39 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 | « no previous file | options.js » ('j') | options.js » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: options.html
===================================================================
--- a/options.html
+++ b/options.html
@@ -98,7 +98,7 @@
<ul id="blocking-languages-table" class="table list">
<template>
<button role="checkbox" class="control"></button>
- <span class="display"></span>
+ <label class="display"></label>
</template>
</ul>
<div class="controls">
@@ -123,7 +123,7 @@
<ul id="recommend-list-table" class="table list">
<template>
<button role="checkbox" class="control"></button>
- <span class="display"></span>
+ <label class="display"></label>
<span class="i18n_options_popular popular"></span>
</template>
</ul>
@@ -162,7 +162,7 @@
<ul id="acceptableads-table" class="table list">
<template>
<button role="checkbox" class="control"></button>
- <span class="display"></span>
+ <label class="display"></label>
</template>
</ul>
</div>
@@ -219,20 +219,28 @@
</h1>
<ul id="tweaks" class="table">
<li data-pref="shouldShowBlockElementMenu">
- <button role="checkbox" data-action="toggle-pref"></button>
- <span class="i18n_options_tweaks_blockElement"></span>
+ <label>
+ <button role="checkbox" data-action="toggle-pref"></button>
+ <span class="i18n_options_tweaks_blockElement"></span>
+ </label>
</li>
<li data-pref="show_devtools_panel">
- <button role="checkbox" data-action="toggle-pref"></button>
- <span class="i18n_options_tweaks_showDevToolsPanel"></span>
+ <label>
+ <button role="checkbox" data-action="toggle-pref"></button>
+ <span class="i18n_options_tweaks_showDevToolsPanel"></span>
+ </label>
</li>
<li data-pref="notifications_ignoredcategories">
- <button role="checkbox" data-action="toggle-pref"></button>
- <span class="i18n_options_tweaks_showNotifications"></span>
+ <label>
+ <button role="checkbox" data-action="toggle-pref"></button>
+ <span class="i18n_options_tweaks_showNotifications"></span>
+ </label>
</li>
<li data-pref="safari_contentblocker">
- <button role="checkbox" data-action="toggle-pref"></button>
- <span class="i18n_options_tweaks_safariContentBlocker"></span>
+ <label>
+ <button role="checkbox" data-action="toggle-pref"></button>
+ <span class="i18n_options_tweaks_safariContentBlocker"></span>
+ </label>
<span id="restart-safari" class="i18n_options_tweaks_restartSafari"></span>
</li>
</ul>
« no previous file with comments | « no previous file | options.js » ('j') | options.js » ('J')

Powered by Google App Engine
This is Rietveld