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

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

Issue 2472083003: Increase .settings-box and .settigns-box.two-line height. (Closed)
Patch Set: reformat comments to fit style guide Created 4 years, 1 month 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 | « no previous file | chrome/browser/resources/settings/settings_vars_css.html » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 721823e59a4af5c4d19f8f685a8255c54a428ab5..26ebed41b63b1c2da4c99618a6ed5827578077e7 100644
--- a/chrome/browser/resources/settings/settings_shared_css.html
+++ b/chrome/browser/resources/settings/settings_shared_css.html
@@ -275,16 +275,26 @@
font-weight: 500;
}
+ /* A row with two lines of text. Often the lower line will be .secondary.
+ */
+ .two-line {
+ min-height: var(--settings-row-two-line-min-height);
+ }
+
/* A settings-box is a horizontal row of text or controls within a
* setting section (page or subpage). */
.settings-box {
align-items: center;
border-top: var(--settings-separator-line);
display: flex;
- min-height: var(--settings-row-min-height);
+ min-height: var(--settings-box-min-height);
padding: 0 var(--settings-box-row-padding);
}
+ .settings-box.two-line {
+ min-height: var(--settings-box-two-line-min-height);
+ }
+
/* We use an explicit tag to remove the top border, rather than a
* :first-of-type modifier. This is a conscious choice, please consult
* with dbeam@ or dschuyler@ prior to changing it. */
@@ -297,12 +307,6 @@
display: block;
}
- /* A row with two lines of text. Often the lower line will be .secondary.
- */
- .two-line {
- min-height: var(--settings-row-two-line-min-height);
- }
-
/* A start-aligned column. */
.single-column {
align-items: flex-start;
« no previous file with comments | « no previous file | chrome/browser/resources/settings/settings_vars_css.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698