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

Fix underinvalidation of :nth-child using invalidation sets #3360

Conversation

kalenikaliaksandr
Copy link
Member

@kalenikaliaksandr kalenikaliaksandr commented Jan 24, 2025

For all invalidation properties nested into nth-child argument list we need to invalidate whole subtree to make sure style of sibling elements will be recalculated.

There are tests where absence of newlines at EOF is important and
required to check certain behavior.
@kalenikaliaksandr kalenikaliaksandr force-pushed the fix-nth-child-pseudo-class-underinvalidation branch from ce5b036 to ec5643a Compare January 24, 2025 21:52
@kalenikaliaksandr kalenikaliaksandr force-pushed the fix-nth-child-pseudo-class-underinvalidation branch 2 times, most recently from 7c7600c to 484b6e2 Compare January 25, 2025 01:52
For all invalidation properties nested into nth-child argument list we
need to invalidate whole subtree to make sure style of sibling elements
will be recalculated.
@kalenikaliaksandr kalenikaliaksandr force-pushed the fix-nth-child-pseudo-class-underinvalidation branch from 484b6e2 to 719678b Compare January 25, 2025 02:54
@awesomekling awesomekling merged commit d5a8204 into LadybirdBrowser:master Jan 25, 2025
7 checks passed
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