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

Unified Diff: ui/webui/resources/cr_elements/shared_style_css.html

Issue 2902363002: [MD settings] adjust button layout (Closed)
Patch Set: restore overflow-y auto 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: ui/webui/resources/cr_elements/shared_style_css.html
diff --git a/ui/webui/resources/cr_elements/shared_style_css.html b/ui/webui/resources/cr_elements/shared_style_css.html
index b5464b7dfbd504235071d70a7b05c8017fd89627..75de21919735a7e7deaa284d01d6cecd8368f671 100644
--- a/ui/webui/resources/cr_elements/shared_style_css.html
+++ b/ui/webui/resources/cr_elements/shared_style_css.html
@@ -49,7 +49,8 @@
}
button[is='paper-icon-button-light'] {
- @apply(--cr-paper-icon-button-margin);
+ -webkit-margin-end: calc(var(--cr-icon-ripple-padding) * -1);
hcarmona 2017/05/25 23:42:58 This is used in 4 places, can we make a variable f
dschuyler 2017/05/26 00:07:03 Done.
+ -webkit-margin-start: 16px;
background-position: center;
background-repeat: no-repeat;
background-size: var(--cr-icon-size);
@@ -58,6 +59,10 @@
width: var(--cr-icon-ripple-size);
}
+ .separator + button[is='paper-icon-button-light'] {
+ -webkit-margin-start: calc(var(--cr-icon-ripple-padding) * -1);
+ }
+
button[is='paper-icon-button-light'].subpage-arrow {
background-image: url(chrome://resources/images/arrow_right.svg);
}
@@ -80,6 +85,20 @@
display: none;
}
+ paper-icon-button {
hcarmona 2017/05/25 23:42:58 Can this
dschuyler 2017/05/26 00:07:03 Done.
+ color: var(--paper-grey-600);
+ flex-shrink: 0;
+ }
+
+ paper-icon-button {
hcarmona 2017/05/25 23:42:58 and this can be combined b/c they use the same sel
dschuyler 2017/05/26 00:07:03 Done.
+ -webkit-margin-end: calc(var(--cr-icon-ripple-padding) * -1);
+ -webkit-margin-start: 16px;
+ }
+
+ .separator + paper-icon-button {
+ -webkit-margin-start: calc(var(--cr-icon-ripple-padding) * -1);
+ }
+
paper-icon-button.subpage-arrow {
background-image: url(../images/arrow_right.svg);
}

Powered by Google App Engine
This is Rietveld 408576698