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

Unified Diff: chrome/browser/resources/settings/people_page/change_picture.css

Issue 1610973002: Settings People Revamp: Implement Change People preview pane (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@0072-settings-add-tests-for-cros-choose-picture
Patch Set: Created 4 years, 11 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/change_picture.css
diff --git a/chrome/browser/resources/settings/people_page/change_picture.css b/chrome/browser/resources/settings/people_page/change_picture.css
index 914164484c73af9287d92b54589906565fc20a0b..8d410db0b0aa5ba3635e62ee60428b9ad8c14130 100644
--- a/chrome/browser/resources/settings/people_page/change_picture.css
+++ b/chrome/browser/resources/settings/people_page/change_picture.css
@@ -7,6 +7,24 @@
margin-top: 16px;
}
+#availableIcons paper-button {
dpapad 2016/01/21 18:11:40 All other CSS classes and IDs in this file are usi
tommycli 2016/01/21 18:47:05 Done.
Dan Beam 2016/01/21 19:33:28 fwiw: I dropped the presubmit check for ids-like-t
+ padding: 4px;
+}
+
+#preview-pane {
+ flex-shrink: 0;
+ margin: 18px 10px 0 0;
+ width: 228px;
+}
+
+#preview-pane img {
tommycli 2016/01/21 01:35:10 These CSS Changes are inspired by the existing CSS
+ border: solid 1px #cacaca;
+ border-radius: 4px;
+ max-height: 220px;
+ max-width: 220px;
+ padding: 3px;
+}
+
.user-image {
height: 64px;
width: 64px;

Powered by Google App Engine
This is Rietveld 408576698