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

Unified Diff: chrome/browser/resources/settings/people_page/user_list.html

Issue 2848973003: MD Settings: convert paper-icon-button to paper-icon-button-light. (Closed)
Patch Set: add missing files Created 3 years, 7 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/people_page/user_list.html
diff --git a/chrome/browser/resources/settings/people_page/user_list.html b/chrome/browser/resources/settings/people_page/user_list.html
index ba5a87a4e01bb6f452818b5bfb7d365ab1fd5119..c8b0d124c84ba5f85c3d3cd0b1b9f14a7e15d0f9 100644
--- a/chrome/browser/resources/settings/people_page/user_list.html
+++ b/chrome/browser/resources/settings/people_page/user_list.html
@@ -3,7 +3,7 @@
<link rel="import" href="chrome://resources/html/i18n_behavior.html">
<link rel="import" href="chrome://resources/html/polymer.html">
<link rel="import" href="chrome://resources/polymer/v1_0/iron-flex-layout/iron-flex-layout-classes.html">
-<link rel="import" href="chrome://resources/polymer/v1_0/paper-icon-button/paper-icon-button.html">
+<link rel="import" href="chrome://resources/polymer/v1_0/paper-icon-button/paper-icon-button-light.html">
<link rel="import" href="../route.html">
<link rel="import" href="../settings_shared_css.html">
@@ -33,6 +33,10 @@
.user-info {
-webkit-padding-start: 20px;
}
+
+ [scrollable] .icon-clear {
+ background-color: transparent;
+ }
stevenjb 2017/05/23 22:16:02 I'd still prefer to make this change globally and
stevenjb 2017/05/23 23:19:24 I just re-read my first suggestion and actually li
scottchen 2017/05/27 00:18:03 Cool, I'll remove it here.
</style>
<div class="user-list" scrollable>
<template is="dom-repeat" items="[[users_]]">
@@ -44,10 +48,10 @@
<div class="secondary">[[item.email]]</div>
</template>
</div>
- <paper-icon-button icon="cr:clear" class="clear-icon"
+ <button is="paper-icon-button-light" class="icon-clear"
on-tap="removeUser_"
hidden="[[shouldHideCloseButton_(disabled, item.isOwner)]]">
- </paper-icon-button>
+ </button>
</div>
</template>
</div>

Powered by Google App Engine
This is Rietveld 408576698