-
Notifications
You must be signed in to change notification settings - Fork 20
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
em units are not working #17
Comments
OK, I figured out why this is impossible to do... might need some documentation for other people to avoid confusion maybe? |
Yep the exact combination of units that can be used needs to be documented somewhere. You can use div {
font-size: responsive 1.2em 1.4em;
font-range: 26em 80em;
} |
Wait, actually, there's something I am failing to understand. Why can't: .foo {
font-size: responsive 3rem 6rem;
font-range: 420px 1280px;
} be converted simply to: .foo {
font-size: calc(3rem + 3 * ((100vw - 420px) / 860));
}
@media screen and (min-width: 1280px) {
.foo {
font-size: 6rem;
}
}
@media screen and (max-width: 420px) {
.foo {
font-size: 3rem;
}
} |
there was definitely a reason for this, but it's been quite a while since I've worked on this repo and can't remember what it was for the life of me. I'll do a bit of digging... |
I found a work around here: |
I reproduced the problem here: https://github.com/perrin4869/responsive_type_test
I am getting the warning
this combination of units is not supported [postcss-responsive-type]
while attempting to process the following:The text was updated successfully, but these errors were encountered: