-
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
**:not()** with a selector list doesn't expand & symbol #4313
Comments
The issue you're encountering with the This behavior indicates that Less currently does not support expanding the If this is causing issues with linters, you might need to adjust the linter rules or continue using the workaround until a future update potentially addresses this limitation. To continue talking to Dosu, mention @dosu. Help Dosu learn! Give it feedback: Great Response | Irrelevant Answer | Incorrect Sources | Too Verbose | Hallucination | Other |
I am guessing @puckowski would be the best person for this type of issue? Due to his work on the latest fix for listed selectors? |
Thank you for the issue report. I'll spend some time this weekend on this issue and I will try to come up with a fix PR. |
I think I have a fix. Will work on adding more tests and try to get a PR submitted this weekend. |
* Fix an issue where selector with comma would not expand correctly.
* Make fix for issue less#4313 selector list not expanding correctly more flexible.
* Add more tests for fix for issue less#4313.
* Cleanup merge issues for fix for issue less#4313.
PR #4314 should resolve this issue, but it looks like we may be having new CI problems. |
FYI figured out the new CI issue so PR #4314 can be merged after review. |
:not(&--a):not(&--b)
does expand the & symbol into the parent's selector as expected.:not(&--a, &--b)
doesn't expand the & symbol into the parent's selector.To reproduce:
Current behavior:
Expected behavior:
Environment information:
less
version: 4.2.2nodejs
version: anyoperating system
: anyWe are currently in the middle of a battle of linters. We are told
:not(a):not(b)
is being classed ascomplex, and should be changed to
:not(a, b)but it appears less forgets to expand when this feature is being used.
4.2.2at least compiles the syntax (though incorrectly) our older version
4.1.3` just keels over completely.. So I am guessing this is a relatively new feature and hasn't had much usage yet.The text was updated successfully, but these errors were encountered: