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

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

Issue 2726113003: MD Settings: re-fix relative paper-input-label font-size. (Closed)
Patch Set: hardcode line-height so clear-icon doesn't push content down when appearing Created 3 years, 10 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 | « chrome/browser/resources/settings/settings_page/settings_subpage_search.html ('k') | 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 1fa88f765cc21564c6c15657f9ea0aa01428392b..b234dfe52ef0bf1b83a17a18969d7e80f327058f 100644
--- a/chrome/browser/resources/settings/settings_vars_css.html
+++ b/chrome/browser/resources/settings/settings_vars_css.html
@@ -103,18 +103,23 @@
color: inherit;
font-size: inherit;
font-weight: inherit;
- line-height: inherit;
+ line-height: 154%;
vertical-align: baseline;
};
--paper-input-container-label: {
font-size: inherit;
- line-height: inherit;
- }
+ line-height: 154%;
+ };
+ --paper-input-container-label-floating: {
+ /* Using "rem" to make it easier to equalize with other labels. */
+ font-size: 1rem;
+ line-height: 1.25rem;
dpapad 2017/03/03 22:23:27 I am not familiar enough with rem units. It sounds
Dan Beam 2017/03/03 22:33:39 yes, rem are good
scottchen 2017/03/06 18:29:54 Yeah, I think we should try to use rem where we ca
+ };
--paper-input-error: {
font-size: 92.31%; /* Should be 12px when 100% is 13px. */
- line-height: inherit;
+ line-height: 154%;
overflow: visible; /* Half-visible error message is not useful at all. */
- }
+ };
--paper-input-max-width: 264px;
--paper-radio-button-ink-size: 40px;
--paper-radio-button-label-color: inherit;
@@ -135,7 +140,7 @@
--settings-input-underline: {
border-color: var(--paper-grey-300);
- }
+ };
--paper-input-container-underline: var(--settings-input-underline);
}
« no previous file with comments | « chrome/browser/resources/settings/settings_page/settings_subpage_search.html ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698