Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Add custom state pseudo class #8467
Add custom state pseudo class #8467
Changes from 2 commits
17f9aae
a7189ff
aed027c
6237810
c0b2fc5
912494b
dee29ca
c7e5e7b
9bf2de8
6595d14
bd965c1
1797b58
e068994
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm, potentially-significant issue: what about customized built-in (i.e., non-autonomous) custom elements? I don't see anything prohibiting them from working, so I believe right now the spec will crash when you do
element.internals.states
.We might need to make the getter steps more complicated? Or we could just expand this to work on customized built-ins? I'm not sure why that restriction was added in the first place...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is because we can't use ElementInternals on custom builtins: #5166
But yeah I don't see any reason why that means we also have to exclude them here. I removed "autonomous", what do you think?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How is this observable if the states set isn't exposed? I'd rather stay aligned with
ElementInternals
.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah I agree, you can't get ElementInternals on a custom builtin so I guess it doesn't matter what we say here