-
Notifications
You must be signed in to change notification settings - Fork 807
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
IMPORTANT: Replace all rem units to px
- Loading branch information
1 parent
ef49185
commit 78de8f1
Showing
55 changed files
with
1,039 additions
and
1,024 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
Oops, something went wrong.
78de8f1
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.
If you don't mind me asking, what is the reason for this change? I'm curious why you're going to
px
when most frameworks I've seen useem
/rem
?78de8f1
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.
Using rem and em is much better for responsiveness and to propagate dimensions changes just by changing font size on body.
78de8f1
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.
@valorin @christophermh44 The problem here is about the minimum font size in Chinese/Japanese/Korean Chrome is 12px, which breaks some
rem
sizes. You can read about it on https://bugs.chromium.org/p/chromium/issues/detail?id=36429 .Another solution is just changing html
font-size
to 20px or any other larger than 12px. But Spectre.css is not using any rem-only feature. I thinkpx
is better. Any thought?