-
Notifications
You must be signed in to change notification settings - Fork 3.4k
Add support for import at-rule layer functionality #4340
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
Conversation
* Add support for layer at-rule. * Add tests for layer at-rule.
* Add support for layer import at-rule syntax. See: https://developer.mozilla.org/en-US/docs/Web/CSS/@import
* Cleanup layer import at-rule solution before merge.
Hi @puckowski! Do you have updates on this? Thanks! |
@luangong Hello, this PR is currently awaiting peer review from another Less.js contributor before it can be merged. @matthew-dean What do you think of this PR to enhance |
@puckowski Sorry for missing this. So this just adds support for |
Yes, this PR bolsters
which are documented here: https://developer.mozilla.org/en-US/docs/Web/CSS/@import |
* Add fix for media query list parsing in import. * Add test for import with media query list syntax.
Cool, thanks! |
Is there a ticket about this (I couldn't find one) or it's simply not planned? |
@jmpri3 I have made a note to take a look at expanding support of Less |
For now I'm successfully using the @layer directive for my needs. Thanks for all the hard work, @puckowski! |
What:
Add support for
@import
layer functionality documented in https://developer.mozilla.org/en-US/docs/Web/CSS/@importBuilds on top of #4337
The above syntax has widely available browser support according to MDN.
Why:
Supporting the additional
@import
CSS syntax may benefit Less users.Checklist:
I did not venture into
.less
imports using this syntax as existingimport
code makes a clear distinction between.css
and.less
and the two have significantly different behavior.