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

Support invalidation for location pseudo classes inside :has() #34051

Merged
merged 1 commit into from
May 13, 2022

Conversation

chromium-wpt-export-bot
Copy link
Collaborator

@chromium-wpt-export-bot chromium-wpt-export-bot commented May 13, 2022

Support invalidation for :link, :any-link and :target pseudo class
inside :has().

Invalidation for :visited inside :has() is not needed since :visited
is not matched inside :has() to prevent leaking visitedness.

Ignored invalidation of :scope inside :has() due to an issue in the
csswg-draft.

Bug: 669058, 1324834
Change-Id: Ice42f0113f0cdc69a8054c2778c9cab3cdd5ef10
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3645177
Reviewed-by: Rune Lillesveen <futhark@chromium.org>
Commit-Queue: Byungwoo Lee <blee@igalia.com>
Cr-Commit-Position: refs/heads/main@{#1003038}

Copy link
Collaborator

@wpt-pr-bot wpt-pr-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The review process for this patch is being conducted in the Chromium project.

Support invalidation for :link, :any-link and :target pseudo class
inside :has().

Invalidation for :visited inside :has() is not needed since :visited
is not matched inside :has() to prevent leaking visitedness.

Ignored invalidation of :scope inside :has() due to an issue in the
csswg-draft.
- w3c/csswg-drafts#7211

Bug: 669058, 1324834
Change-Id: Ice42f0113f0cdc69a8054c2778c9cab3cdd5ef10
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3645177
Reviewed-by: Rune Lillesveen <futhark@chromium.org>
Commit-Queue: Byungwoo Lee <blee@igalia.com>
Cr-Commit-Position: refs/heads/main@{#1003038}
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants