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

Unified Diff: chrome/browser/resources/settings/site_settings/site_data.html

Issue 2391253002: Site Settings Desktop: Polish the Site Data section. (Closed)
Patch Set: Address feedback Created 4 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/settings/site_settings/site_data.html
diff --git a/chrome/browser/resources/settings/site_settings/site_data.html b/chrome/browser/resources/settings/site_settings/site_data.html
index 2d95ea9c9c18d287b8be050d997c871d45c3aa53..d105e19cab116d2fcd54806c9afa84b669983873 100644
--- a/chrome/browser/resources/settings/site_settings/site_data.html
+++ b/chrome/browser/resources/settings/site_settings/site_data.html
@@ -1,4 +1,5 @@
<link rel="import" href="chrome://resources/html/polymer.html">
+<link rel="import" href="chrome://resources/cr_elements/cr_dialog/cr_dialog.html">
<link rel="import" href="chrome://resources/cr_elements/icons.html">
<link rel="import" href="chrome://resources/html/web_ui_listener_behavior.html">
<link rel="import" href="chrome://resources/polymer/v1_0/iron-flex-layout/iron-flex-layout-classes.html">
@@ -34,11 +35,26 @@
margin-top: -4px;
}
</style>
+ <!-- Confirm Delete dialog -->
+ <dialog is="cr-dialog" id="confirmDeleteDialog">
+ <div class="title">$i18n{siteSettingsCookieRemoveDialogTitle}</div>
+ <div class="body">[[confirmationDeleteMsg_]]</div>
+ <div class="button-container">
+ <paper-button class="cancel-button" on-tap="onCloseDialog_">
+ $i18n{cancel}
+ </paper-button>
+ <paper-button class="action-button" on-tap="onConfirmDelete_">
+ $i18n{siteSettingsCookieClearAll}
+ </paper-button>
+ </div>
+ </dialog>
+
<div class="layout horizontal">
<div class="flex site-header">$i18n{siteSettingsCookieHeader}</div>
<div class="secondary-action"
hidden$="[[!isRemoveButtonVisible_(sites, renderedItemCount)]]">
- <paper-button on-tap="onDeleteMultipleSites_" class="secondary-button">
+ <paper-button on-tap="onConfirmDeleteMultipleSites_"
+ class="secondary-button">
[[computeRemoveLabel_(filterString_)]]
</paper-button>
</div>
@@ -58,7 +74,7 @@
<div class="flex middle">[[item.site]]</div>
<div class="site">[[item.localData]]</div>
</div>
- <paper-icon-button on-tap="onDeleteSite_"
+ <paper-icon-button on-tap="onConfirmDeleteSite_"
icon="cr:delete"></paper-icon-button>
</div>
</template>

Powered by Google App Engine
This is Rietveld 408576698