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

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

Issue 2960803002: [MD settings] change single line row height (Closed)
Patch Set: Created 3 years, 6 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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/resources/settings/settings_vars_css.html
diff --git a/chrome/browser/resources/settings/settings_vars_css.html b/chrome/browser/resources/settings/settings_vars_css.html
index 0ab064f189d4570f91e4bd20e16d6cfec0b7c206..c93cc87051b15d2bc213e84e9d703c88f397e473 100644
--- a/chrome/browser/resources/settings/settings_vars_css.html
+++ b/chrome/browser/resources/settings/settings_vars_css.html
@@ -37,13 +37,13 @@
/* TODO (scottchen): re-implement with paddings instead; */
/* These are used for row items such as radio buttons, check boxes, list
* items etc. */
- --settings-row-min-height: 45px;
+ --settings-row-min-height: 48px;
--settings-row-two-line-min-height: 64px;
/* TODO (scottchen): re-implement with paddings instead; */
/* These are used for the settings-box containers, which may contain one or
* more "row items". */
- --settings-box-min-height: 45px;
+ --settings-box-min-height: 48px;
dpapad 2017/06/27 01:19:32 Nit: Does the following work? --settings-box-min-
dschuyler 2017/06/27 21:47:04 I'm going to look into combining lines 37 to 48 in
--settings-box-two-line-min-height: 64px;
--settings-box-three-line-min-height: 84px;
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698