Permalink
Show file tree
Hide file tree
5 comments
on commit
sign in to comment.
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Browse files
More explicitly define font resets on form controls (#607)
- Loading branch information
Jonathan Neal
committed
Aug 8, 2016
1 parent
db6567c
commit 5d728b83a5ee6ee3e9e3158a28d6369718014e0c
Showing
3 changed files
with
8 additions
and
12 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
5d728b8
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can I ask what the reason behind the more explicit font definitions is? Thing is, that I usually set a base
font-family
,font-size
andline-height
on thebody
tag, which usually do apply to input elements as well. This leads to me basically just adding the old rule back into my styles, overriding the one set here again which does not make too much sense in my mind. That is why I would be grateful to understand the reasoning behind this decision.5d728b8
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@rafhun See #591 and #607
5d728b8
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The
font-family
should stayinherit
IMO.5d728b8
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree, the
input
font-family
should beinherit
.Since its set to
sans-serif
onhtml
anyway (#591 (comment)), wouldn't usingfont-family: inherit
oninput
just inherit thefont-family: sans-serif
from thehtml
while still allowing the user to override it by just specifying afont-family
onhtml
orbody
and not having to explicitly specify thefont-family
forinput
?5d728b8
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@rafhun, @jovilog, @MethodGrab, as @Florian-R pointed out, you can see the reasons behind this decision on #591 and #607. If you consider that we should reconsider this, please, open an issue or comment in the original issues, so we can track this. Comments on commits are hard to track. Thanks!