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

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

Issue 1875413002: MD Settings: polish top-level bluetooth page (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: test fixes Created 4 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
Index: chrome/browser/resources/settings/settings_shared_css.html
diff --git a/chrome/browser/resources/settings/settings_shared_css.html b/chrome/browser/resources/settings/settings_shared_css.html
index 33e504aaa0eebfc2d66ce9d3d688ed6f27e828fc..ee62edbdc22d04f941c9c57646ef272f91d3c3e5 100644
--- a/chrome/browser/resources/settings/settings_shared_css.html
+++ b/chrome/browser/resources/settings/settings_shared_css.html
@@ -8,12 +8,17 @@
--checkbox-spacing: 18px;
--iron-icon-fill-color: var(--paper-grey-600);
--iron-icon-height: 20px;
+ --iron-icon-spacing: 16px;
--iron-icon-width: 20px;
--paper-icon-button: {
width: 36px;
height: 36px;
};
--paper-input-max-width: 264px;
+ --paper-item: {
+ font-size: inherit;
+ };
+
--settings-background-color: var(--paper-blue-grey-50);
--settings-secondary: {
color: var(--paper-grey-600);
@@ -25,6 +30,9 @@
/* Some colors use non-MD colors. These custom colors are specified by
* UX design (bettes@). */
--settings-nav-grey: rgb(90, 90, 90);
+
+ --settings-row-min-height: 40px;
+
--settings-title-bar-background-color: rgb(52, 73, 94);
--settings-title-bar-color: rgb(255, 255, 255);
}
@@ -36,7 +44,7 @@
font-size: 100%;
font-weight: 500;
margin: 0;
- min-height: 40px;
+ min-height: var(--settings-row-min-height);
}
iron-icon[icon=check],
@@ -144,7 +152,7 @@
.list-item {
@apply(--layout-center);
display: flex;
- min-height: 40px;
+ min-height: var(--settings-row-min-height);
padding: 0;
}
@@ -188,7 +196,7 @@
@apply(--layout-center);
border-top: 1px solid var(--paper-grey-300);
display: flex;
- min-height: 40px;
+ min-height: var(--settings-row-min-height);
padding: 0 20px;
}
@@ -239,11 +247,7 @@
padding: 8px;
}
- .vertical-list > div:first-of-type {
- border-top: none;
- }
-
- .vertical-list > div {
+ .vertical-list > *:not(:first-of-type) {
border-top: 1px solid var(--paper-grey-300);
}

Powered by Google App Engine
This is Rietveld 408576698