Skip to content
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

Update tests for fixing unifyCompound() and unifyComplex() ordering of pseudo class/element selectors #2018

Merged
merged 7 commits into from
Sep 30, 2024

Conversation

Goodwine
Copy link
Member

@Goodwine Goodwine commented Sep 18, 2024

See sass/dart-sass#2335
See sass/dart-sass#2350

[skip dart-sass]

@Goodwine Goodwine requested a review from nex3 September 18, 2024 22:00
@Goodwine
Copy link
Member Author

I added some changes to the tests to catch more potentially erroneous behaviors (or maybe those behaviors are intended haha), PTAL

@Goodwine Goodwine force-pushed the fix-unify-compound-changing-order branch from 0053ae9 to 92da5eb Compare September 19, 2024 01:32
@Goodwine Goodwine force-pushed the fix-unify-compound-changing-order branch from 92da5eb to d0def70 Compare September 27, 2024 21:42
spec/core_functions/selector/unify/compound.hrx Outdated Show resolved Hide resolved
@Goodwine Goodwine merged commit 58ae7fd into main Sep 30, 2024
12 checks passed
@Goodwine Goodwine deleted the fix-unify-compound-changing-order branch September 30, 2024 20:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants