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

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

Issue 2356653002: [MD settings] icons for clickable rows (Closed)
Patch Set: wrapped line Created 4 years, 3 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/people_page.html
diff --git a/chrome/browser/resources/settings/people_page/people_page.html b/chrome/browser/resources/settings/people_page/people_page.html
index 55a3f6dc81c6bdff632298559c9a136a00ae005e..a0fe20924ecf34d84ab31881d2fbbfdf2553e688 100644
--- a/chrome/browser/resources/settings/people_page/people_page.html
+++ b/chrome/browser/resources/settings/people_page/people_page.html
@@ -95,6 +95,8 @@
</if>
</div>
<if expr="not chromeos">
+ <iron-icon icon="settings:arrow-drop-up" class="settings-arrow-right">
hcarmona 2016/09/20 21:33:57 Maybe this button shouldn't have the > because it
dschuyler 2016/09/23 17:29:18 The longer term plan is to change the UI further s
+ </iron-icon>
<template is="dom-if" if="[[!syncStatus.signedIn]]">
<span class="secondary-action">
<paper-button class="primary-button" on-tap="onSigninTap_"
@@ -149,6 +151,9 @@
[[syncStatus.statusText]]
</div>
</div>
+ <iron-icon icon="settings:arrow-drop-up"
+ class="settings-arrow-right">
+ </iron-icon>
</div>
</template>
@@ -162,6 +167,8 @@
<div>$i18n{personalizeGoogleServicesTitle}</div>
<div class="secondary">$i18n{personalizeGoogleServicesText}</div>
</div>
+ <iron-icon icon="settings:arrow-drop-up" class="settings-arrow-right">
+ </iron-icon>
</div>
<if expr="chromeos">
@@ -230,7 +237,8 @@
</if>
<div id="manage-other-people-subpage-trigger" class="settings-box">
- <paper-button class="primary-button" on-tap="onManageOtherPeople_">
+ <paper-button class="primary-button"
+ on-tap="onManageOtherPeople_">
$i18n{manageOtherPeople}
</paper-button>
</div>

Powered by Google App Engine
This is Rietveld 408576698