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

Unified Diff: chrome/browser/resources/settings/privacy_page/privacy_page.html

Issue 2684853003: MD Settings: change how tap is handled on custom toggle rows (Closed)
Patch Set: if the compiler ain't happy, ain't nobody happy Created 3 years, 10 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/settings/privacy_page/privacy_page.html
diff --git a/chrome/browser/resources/settings/privacy_page/privacy_page.html b/chrome/browser/resources/settings/privacy_page/privacy_page.html
index c6c14a1e7eaf634b36fa131fafd34ff93636ba63..a43c81404a5c4c49336e1c681033551f1c435560 100644
--- a/chrome/browser/resources/settings/privacy_page/privacy_page.html
+++ b/chrome/browser/resources/settings/privacy_page/privacy_page.html
@@ -37,20 +37,6 @@
button[is='paper-icon-button-light'].icon-help {
background-image: url(../images/help_outline.svg);
}
-
- .settings-row-min-height {
- min-height: var(--settings-row-min-height);
- }
-
- #metricsReporting paper-tooltip {
- --paper-tooltip: var(--cr-policy-tooltip);
- }
-
- #metricsReportingControl,
- #indicator,
- #restart {
- -webkit-margin-start: var(--checkbox-spacing);
- }
</style>
<template is="dom-if" if="[[showClearBrowsingDataDialog_]]" restamp>
<settings-clear-browsing-data-dialog prefs="{{prefs}}"
@@ -82,11 +68,12 @@
</settings-toggle-button>
</div>
<div class="settings-box">
- <div class="start">$i18n{safeBrowsingEnableExtendedReporting}</div>
- <paper-toggle-button id="safeBrowsingExtendedReportingControl"
- on-tap="onSafeBrowsingExtendedReportingControlTap_"
- checked="[[safeBrowsingExtendedReportingEnabled_]]">
- </paper-toggle-button>
+ <settings-toggle-button class="start"
+ id="safeBrowsingExtendedReportingControl"
+ pref="[[safeBrowsingExtendedReportingPref_]]"
+ label="$i18n{safeBrowsingEnableExtendedReporting}"
+ no-set-pref>
+ </settings-toggle-button>
</div>
<div class="settings-box">
<settings-toggle-button class="start"
@@ -105,24 +92,18 @@
</if><!-- chromeos -->
<if expr="not chromeos">
<div class="settings-box">
- <div class="start">$i18n{enableLogging}</div>
- <template is="dom-if" if="[[showRestart_]]" restamp>
- <paper-button on-tap="onRestartTap_" id="restart">
- $i18n{restart}
- </paper-button>
- </template>
- <template is="dom-if" if="[[metricsReporting_.managed]]" restamp>
- <iron-icon id="indicator" tabindex=0 icon="cr:domain"></iron-icon>
- <paper-tooltip for="indicator" position="top"
- fit-to-visible-bounds>
- $i18n{controlledSettingPolicy}
- </paper-tooltip>
- </template>
- <paper-toggle-button id="metricsReportingControl"
- on-tap="onMetricsReportingControlTap_"
- checked="[[metricsReporting_.enabled]]"
- disabled="[[metricsReporting_.managed]]">
- </paper-toggle-button>
+ <settings-toggle-button class="start"
+ id="metricsReportingControl"
+ pref="[[metricsReportingPref_]]"
+ label="$i18n{enableLogging}"
+ no-set-pref>
+ <template is="dom-if" if="[[showRestart_]]" restamp>
+ <paper-button on-tap="onRestartTap_" id="restart"
+ class="more-actions">
+ $i18n{restart}
+ </paper-button>
+ </template>
+ </settings-toggle-button>
</div>
</if><!-- not chromeos -->
</if><!-- _google_chrome -->

Powered by Google App Engine
This is Rietveld 408576698