-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
Media query express support #3735
Comments
I think I can work on this issue. Can you please give some details about the issue? |
@mud1tx Thanks for your reply. I'd like to provide something, but I don't know what to provide. |
@mud1tx This isn't a bug, it's technically a feature request. So what needs to be added is support for range syntax in media queries. This would need to support merging as well. That is, right now Less transforms this: @media screen {
@media (min-width: 1023px) {
.selector {
color: blue;
}
}
} ...into... @media screen and (min-width: 1023px) {
.selector {
color: blue;
}
} So, tests would need to be added to convert: @media (400px <= width) {
@media (width <= 700px) {
.main-box {
display: flex;
}
}
} into: @media (400px <= width) and (width <= 700px) {
.main-box {
display: flex;
}
} |
me too
|
The range syntax seems has been supported in version 4.2.0 by PR #3811 |
A new range syntax introduced in Media Queries Level 4.
The syntax allow query expresses, includes range express.
But i got an error when using the syntax.
To reproduce:
Current behavior:
Expected behavior:
Convert to css with the syntax correctly
Environment information:
less
version: 4.1.3nodejs
version: 16.14.0operating system
: winThe text was updated successfully, but these errors were encountered: