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

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

Issue 2716333002: Implement important sites dialog for desktop. (Closed)
Patch Set: extract importantsites dialog to separate dialog Created 3 years, 8 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/clear_browsing_data_dialog/clear_browsing_data_dialog.html
diff --git a/chrome/browser/resources/settings/clear_browsing_data_dialog/clear_browsing_data_dialog.html b/chrome/browser/resources/settings/clear_browsing_data_dialog/clear_browsing_data_dialog.html
index fed8c2db992397ad7e5b86e29c41376b05f286e4..23f68a0370b63159d41384c894ddcd9414032356 100644
--- a/chrome/browser/resources/settings/clear_browsing_data_dialog/clear_browsing_data_dialog.html
+++ b/chrome/browser/resources/settings/clear_browsing_data_dialog/clear_browsing_data_dialog.html
@@ -6,6 +6,7 @@
<link rel="import" href="../i18n_setup.html">
<link rel="import" href="clear_browsing_data_browser_proxy.html">
<link rel="import" href="history_deletion_dialog.html">
+<link rel="import" href="../controls/important_site_checkbox.html">
<link rel="import" href="../controls/settings_checkbox.html">
<link rel="import" href="../controls/settings_dropdown_menu.html">
<link rel="import" href="../icons.html">
@@ -14,11 +15,11 @@
<dom-module id="settings-clear-browsing-data-dialog">
<template>
<style include="settings-shared">
- #dialog:not(.fully-rendered) {
+ #clearBrowsingDataDialog:not(.fully-rendered) {
visibility: hidden;
}
- #dialog .footer {
+ #clearBrowsingDataDialog .footer {
color: var(--paper-grey-600);
}
@@ -79,22 +80,29 @@
-webkit-margin-start: 0.5em;
}
+ .title .secondary {
+ font-size: 86.67%; /* (13px / 15px) * 100 */
+ padding-top: 8px;
+ }
+
/* Cap the height on smaller screens to avoid unfavorable clipping.
* Replace the bottom margin with padding to avoid the gap between
* the scrollbar and the bottom separator. */
@media all and (max-height: 724px) {
- #dialog {
+ #clearBrowsingDataDialog {
/* crbug.com/652027: Show four and a *half* items in the list. */
--cr-dialog-body-container: {
- max-height: 280px;
+ max-height: 280px;
};
}
}
</style>
- <dialog is="cr-dialog" id="dialog" close-text="$i18n{close}"
- show-scroll-borders ignore-popstate>
- <div class="title">$i18n{clearBrowsingData}</div>
+ <dialog is="cr-dialog" id="clearBrowsingDataDialog"
+ close-text="$i18n{close}" show-scroll-borders ignore-popstate>
+ <div class="title">
+ $i18n{clearBrowsingData}
+ </div>
<div class="body">
<div class="row">
$i18n{clearFollowingItemsFrom}
@@ -115,13 +123,13 @@
sub-label="[[counters_.download_history]]"
disabled="[[clearingInProgress_]]">
</settings-checkbox>
- <settings-checkbox
+ <settings-checkbox id="cacheCheckbox"
pref="{{prefs.browser.clear_data.cache}}"
label="$i18n{clearCache}"
sub-label="[[counters_.cache]]"
disabled="[[clearingInProgress_]]">
</settings-checkbox>
- <settings-checkbox
+ <settings-checkbox id="cookiesCheckbox"
pref="{{prefs.browser.clear_data.cookies}}"
label="$i18n{clearCookies}"
sub-label="$i18n{clearCookiesCounter}"
@@ -158,7 +166,7 @@
on-tap="onCancelTap_">$i18n{cancel}</paper-button>
<paper-button class="action-button" disabled="[[clearingInProgress_]]"
on-tap="onClearBrowsingDataTap_">
- $i18n{clearBrowsingData}
+ $i18n{clearBrowsingData}
dschuyler 2017/04/27 23:10:25 nit: indented too far
</paper-button>
</div>
<div class="footer">
@@ -179,6 +187,36 @@
</div>
</dialog>
+ <dialog is="cr-dialog" id="importantSitesDialog" close-text="$i18n{close}"
+
+ show-scroll-borders ignore-popstate>
+ <div class="title">
+ $i18n{clearBrowsingData}
+ <div class="secondary">
+ $i18n{importantSitesSubtitle}
+ </div>
+ </div>
+ <div class="body">
+ <template is="dom-repeat" items="[[importantSites_]]">
+ <div class="row">
+ <important-site-checkbox
+ site="[[item]]"
+ disabled="[[clearingInProgress_]]">
+ </important-site-checkbox>
+ </div>
+ </template>
+ </div>
+ <div class="button-container">
+ <paper-spinner active="[[clearingInProgress_]]"></paper-spinner>
+ <paper-button class="cancel-button" disabled="[[clearingInProgress_]]"
+ on-tap="onImportantSitesCancelTap_">$i18n{cancel}</paper-button>
+ <paper-button class="action-button" disabled="[[clearingInProgress_]]"
+ on-tap="onImportantSitesConfirmTap_">
+ $i18n{importantSitesConfirm}
+ </paper-button>
+ </div>
+ </dialog>
+
<template is="dom-if" if="[[showHistoryDeletionDialog_]]" restamp>
<settings-history-deletion-dialog id="notice"
on-close="onHistoryDeletionDialogClose_">

Powered by Google App Engine
This is Rietveld 408576698