-
Notifications
You must be signed in to change notification settings - Fork 186
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
Merging is broken for :where selectors #454
Comments
@ygoe Which version of CSSO do you use? CSSO recognises |
Oh, I see, I have version 4.2.0. Need to update that again. |
Okay, how can I update this? It doesn't do with |
Any idea how I should follow your suggestion? It seems impossible with npm. Or there's a packaging bug in csso. |
@lahmatiy Looks like the version you suggested is not available. See css/csso-cli#25 |
It looks like |
:where()
selectors have zero specificity so their order is the only thing that matters. csso will merge multiple selectors into a single comma-separated selector if they have the same content. But then, it will also change their order and it renders differently in browsers.Example SASS:
Compiled to CSS:
Result after csso: (inserted line breaks for readability)
If anything, the second line needs to come last (merge to last occurrence, not to first). But then again, it's still a different order than declared in the source.
Similar to #301?
The text was updated successfully, but these errors were encountered: