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

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

Issue 2815783002: [MD settings] vertical align left edge of icons and text in ChromeOS People (Closed)
Patch Set: 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
« no previous file with comments | « chrome/browser/resources/settings/people_page/user_list.html ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/resources/settings/people_page/users_page.html
diff --git a/chrome/browser/resources/settings/people_page/users_page.html b/chrome/browser/resources/settings/people_page/users_page.html
index 6316f4a2869c09bbc8782ab8c03a061e90b3e24a..318d936e1f20e77f91608e55be2e298a65cf4db5 100644
--- a/chrome/browser/resources/settings/people_page/users_page.html
+++ b/chrome/browser/resources/settings/people_page/users_page.html
@@ -11,20 +11,13 @@
<dom-module id="settings-users-page">
<template>
<style include="settings-shared action-link">
- .users {
- /* The users box must line up with the checkbox text. */
- -webkit-margin-start: var(--settings-indent-width);
- }
-
.settings-box:first-of-type {
border-top: none;
}
#add-user-button {
/* Add user button must be lined up with the start of users' names. */
- /* TODO(dschuyler): Should this be var(--settings-box-row-indent) rather
- than 56px? */
- -webkit-margin-start: 56px;
+ -webkit-margin-start: var(--settings-box-row-indent);
}
</style>
<template is="dom-if" if="[[isWhitelistManaged_]]">
@@ -58,18 +51,18 @@
disabled="[[isEditingDisabled_(isOwner_, isWhitelistManaged_)]]"
inverted>
</settings-toggle-button>
- <div class="users">
- <settings-user-list prefs="[[prefs]]"
- disabled="[[isEditingUsersDisabled_(isOwner_, isWhitelistManaged_,
- prefs.cros.accounts.allowGuest.value)]]">
- </settings-user-list>
- <div id="add-user-button" class="list-item"
- hidden="[[isEditingUsersDisabled_(isOwner_, isWhitelistManaged_,
- prefs.cros.accounts.allowGuest.value)]]">
- <a is="action-link" class="list-button" on-tap="openAddUserDialog_">
- $i18n{addUsers}
- </a>
- </div>
+ </div>
+ <div class="list-frame">
+ <settings-user-list prefs="[[prefs]]"
+ disabled="[[isEditingUsersDisabled_(isOwner_, isWhitelistManaged_,
+ prefs.cros.accounts.allowGuest.value)]]">
+ </settings-user-list>
+ <div id="add-user-button" class="list-item"
+ hidden="[[isEditingUsersDisabled_(isOwner_, isWhitelistManaged_,
+ prefs.cros.accounts.allowGuest.value)]]">
+ <a is="action-link" class="list-button" on-tap="openAddUserDialog_">
+ $i18n{addUsers}
+ </a>
</div>
</div>
<settings-users-add-user-dialog id="addUserDialog"
« no previous file with comments | « chrome/browser/resources/settings/people_page/user_list.html ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698