Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(18)

Unified Diff: chrome/browser/resources/options/browser_options.html

Issue 11193002: Add third batch of controlled setting indicators (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Comments addressed. Created 8 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/options/browser_options.html
diff --git a/chrome/browser/resources/options/browser_options.html b/chrome/browser/resources/options/browser_options.html
index e6294e074a2b88b4a1ab93286cd8caa68d34d6df..2a7c52391fbf9a53dae7454f660fcbe64aada7ee 100644
--- a/chrome/browser/resources/options/browser_options.html
+++ b/chrome/browser/resources/options/browser_options.html
@@ -255,11 +255,18 @@
</span>
</div>
<div class="checkbox">
- <label>
+ <span class="controlled-setting-with-label">
<input id="safeBrowsingEnabled" pref="safebrowsing.enabled"
metric="Options_SafeBrowsingCheckbox" type="checkbox">
- <span i18n-content="safeBrowsingEnableProtection"></span>
- </label>
+ <span>
+ <label for="safeBrowsingEnabled">
+ <span i18n-content="safeBrowsingEnableProtection"></span>
James Hawkins 2012/10/17 03:29:28 Just set the i18n-content of the label element.
bartfab (slow) 2012/10/18 13:05:04 Done.
+ </label>
+ <span class="controlled-setting-indicator"
+ pref="safebrowsing.enabled">
+ </span>
+ </span>
+ </span>
</div>
<if expr="pp_ifdef('_google_chrome')">
<div class="checkbox">
@@ -270,7 +277,7 @@
</label>
</div>
<div id="metricsReportingSetting" class="checkbox">
- <label>
+ <span class="controlled-setting-with-label">
<if expr="pp_ifdef('chromeos')">
<input id="metricsReportingEnabled"
pref="cros.metrics.reportingEnabled" type="checkbox">
@@ -279,8 +286,15 @@
<input id="metricsReportingEnabled"
pref="user_experience_metrics.reporting_enabled" type="checkbox">
</if>
- <span i18n-content="enableLogging"></span>
- </label>
+ <span>
+ <label for="metricsReportingEnabled">
+ <span i18n-content="enableLogging"></span>
+ </label>
+ <span class="controlled-setting-indicator"
+ pref="user_experience_metrics.reporting_enabled">
+ </span>
+ </span>
+ </span>
</div>
</if> <!-- pp_ifdef('_google_chrome') -->
<div class="checkbox">
@@ -323,25 +337,38 @@
<section id="passwords-and-autofill-section">
<h3 i18n-content="passwordsAndAutofillGroupName"></h3>
<div class="checkbox">
- <label>
+ <span class="controlled-setting-with-label">
<input id="autofill-enabled" pref="autofill.enabled"
metric="Options_FormAutofill" type="checkbox">
- <span i18n-content="autofillEnabled"></span>
- </label>
- <button id="autofill-settings" class="link-button"
- pref="autofill.enabled"
- i18n-content="manageAutofillSettings"></button>
+ <span>
+ <label for="autofill-enabled">
+ <span i18n-content="autofillEnabled"></span>
+ </label>
+ <span class="controlled-setting-indicator" pref="autofill.enabled">
+ </span>
+ <button id="autofill-settings" class="link-button"
+ i18n-content="manageAutofillSettings">
+ </button>
+ </span>
+ </span>
</div>
<div class="checkbox">
- <label>
+ <span class="controlled-setting-with-label">
<input id="password-manager-enabled"
pref="profile.password_manager_enabled"
metric="Options_PasswordManager" type="checkbox">
- <span i18n-content="passwordManagerEnabled"></span>
- </label>
- <button id="manage-passwords" class="link-button"
- i18n-content="managePasswords"
- pref="profile.password_manager_enabled"></button>
+ <span>
+ <label for="password-manager-enabled">
+ <span i18n-content="passwordManagerEnabled"></span>
+ </label>
+ <span class="controlled-setting-indicator"
+ pref="profile.password_manager_enabled">
+ </span>
+ <button id="manage-passwords" class="link-button"
+ i18n-content="managePasswords">
+ </button>
+ </span>
+ </span>
</div>
<div class="checkbox" id="password-generation-checkbox">
<label>
@@ -427,21 +454,33 @@
<div>
<div class="settings-row">
<label>
- <span i18n-content="downloadLocationBrowseTitle"></span>
+ <span id="download-location-label"
+ i18n-content="downloadLocationBrowseTitle">
+ </span>
<input id="downloadLocationPath" class="weakrtl" type="text"
- size="36" readonly>
+ size="36" readonly>
</label>
<button id="downloadLocationChangeButton"
- pref="download.prompt_for_download"
- i18n-content="downloadLocationChangeButton"></button>
+ i18n-content="downloadLocationChangeButton">
+ </button>
+ <span class="controlled-setting-indicator"
+ pref="download.default_directory">
+ </span>
</div>
<div class="checkbox">
- <label>
- <input type="checkbox"
- pref="download.prompt_for_download"
- metric="Options_AskForSaveLocation">
- <span i18n-content="downloadLocationAskForSaveLocation"></span>
- </label>
+ <span class="controlled-setting-with-label">
+ <input id="prompt-for-download" type="checkbox"
+ pref="download.prompt_for_download"
+ metric="Options_AskForSaveLocation">
+ <span>
+ <label for="prompt-for-download">
+ <span i18n-content="downloadLocationAskForSaveLocation"></span>
+ </label>
+ <span class="controlled-setting-indicator"
+ pref="download.prompt_for_download">
+ </span>
+ </span>
+ </span>
</div>
<if expr="pp_ifdef('chromeos')">
<div class="checkbox" guest-visibility="disabled">
« no previous file with comments | « chrome/browser/resources/options/browser_options.css ('k') | chrome/browser/resources/options/browser_options.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698