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

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

Issue 1981423002: [MD settings] change button padding and placement (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: added comment Created 4 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/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 01421172d609a035dfea033e5b9422bfc6b18ed9..8d337ea30180e85ffcf66c08831ef2e139b00198 100644
--- a/chrome/browser/resources/settings/people_page/people_page.html
+++ b/chrome/browser/resources/settings/people_page/people_page.html
@@ -79,10 +79,12 @@
</div>
<if expr="not chromeos">
<template is="dom-if" if="[[!syncStatus.signedIn]]">
- <paper-button class="action-button" on-tap="onSigninTap_"
- disabled="[[syncStatus.setupInProgress]]">
- $i18n{syncSignin}
- </paper-button>
+ <span class="secondary-action">
+ <paper-button class="action-button" on-tap="onSigninTap_"
+ disabled="[[syncStatus.setupInProgress]]">
+ $i18n{syncSignin}
+ </paper-button>
+ </span>
</template>
<template is="dom-if" if="[[syncStatus.signedIn]]">
<span class="secondary-action">
@@ -107,7 +109,7 @@
<template is="dom-if"
if="[[isAdvancedSyncSettingsVisible_(syncStatus)]]">
- <div class="settings-box">
+ <div class="settings-box button-strip">
<paper-button on-tap="onSyncTap_" class="primary-button">
$i18n{syncPageTitle}
</paper-button>
@@ -185,7 +187,7 @@
</template>
</if>
- <div class="settings-box">
+ <div class="settings-box button-strip">
<paper-button class="primary-button" on-tap="onManageOtherPeople_">
$i18n{manageOtherPeople}
</paper-button>

Powered by Google App Engine
This is Rietveld 408576698