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

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

Issue 2971823002: Cleanup Tool WebUI: Add logs upload checkbox and minor polishing (Closed)
Patch Set: Address review comments on #3, simplify idle error cases Created 3 years, 5 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
« no previous file with comments | « no previous file | chrome/browser/resources/settings/chrome_cleanup_page/chrome_cleanup_page.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/resources/settings/chrome_cleanup_page/chrome_cleanup_page.html
diff --git a/chrome/browser/resources/settings/chrome_cleanup_page/chrome_cleanup_page.html b/chrome/browser/resources/settings/chrome_cleanup_page/chrome_cleanup_page.html
index 3457c8af4549b7adabf31599869851751799e6a8..ed9e1d6649596b9bb1a022f9d7951e784041db23 100644
--- a/chrome/browser/resources/settings/chrome_cleanup_page/chrome_cleanup_page.html
+++ b/chrome/browser/resources/settings/chrome_cleanup_page/chrome_cleanup_page.html
@@ -20,8 +20,10 @@
}
#show-files-button {
- color: var(--google-blue-700);
--paper-button-ink-color: white;
+ /* Left-align the text of the button with the rest of the card's text */
+ -webkit-margin-start: calc(var(--settings-button-edge-spacing) * -1);
+ color: var(--google-blue-700);
text-transform: inherit;
}
@@ -31,8 +33,8 @@
}
.status-icon-container {
+ -webkit-padding-end: 12px;
min-width: 28px;
- padding-right: 12px;
}
.status-icon-remove {
@@ -68,9 +70,22 @@
</div>
<div id="details-container"
class="settings-box two-line" hidden="[[!showDetails_]]">
- <div>[[detailsDescription]]</div>
+ <div>
+ $i18n{chromeCleanupExplanation}
+ <a href="$i18n{chromeCleanupLearnMoreUrl}" target="_blank">
+ $i18n{learnMore}
+ </a>
+ </div>
+ </div>
+ <div class="settings-box continuation" hidden="[[!showLogsPermission_]]">
+ <settings-toggle-button class="start"
+ id="chromeCleanupLogsUploadControl"
+ label="$i18n{chromeCleanupLogsUploadPermission}"
+ pref="[[logsUploadPref_]]"
+ on-settings-boolean-control-change="changeLogsPermission_">
+ </settings-toggle-button>
</div>
- <div class="settings-box continuation" hidden="[[!showDetails_]]">
+ <div class="settings-box" hidden="[[!showDetails_]]">
<div class="show-files-container start" hidden="[[showFilesToRemove_]]">
<paper-button id="show-files-button" on-tap="showFiles_">
$i18n{chromeCleanupLinkShowFiles}
« no previous file with comments | « no previous file | chrome/browser/resources/settings/chrome_cleanup_page/chrome_cleanup_page.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698