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

Adds CSSCustomState to schema #128

Merged
merged 3 commits into from
May 6, 2024

Conversation

lukewarlow
Copy link
Contributor

@lukewarlow lukewarlow commented Nov 9, 2023

Fixes #113

See https://wicg.github.io/custom-state-pseudo-class/#exposing for the WICG proposal.

See whatwg/html#8467 for HTML PR and w3c/csswg-drafts#8213 for CSS PR (not a direct match for the WICG proposal)

While this is currently only supported in Chromium (in a form that doesn't match the latest spec) there's an effort to get this cross browser. See https://groups.google.com/a/mozilla.org/g/dev-platform/c/vDVxI6O3YqU for a Firefox I2P.

There's also interest within OpenUI to use this schema for our anatomies which would make use of this too.

schema.d.ts Show resolved Hide resolved
@MrHBS
Copy link

MrHBS commented Feb 10, 2024

Can someone review this PR?

@thepassle
Copy link
Collaborator

This looks fine to me 👍 Konnor was already working on a PR to add this to @custom-elements-manifest/analyzer

@justinfagnani can you also take a look? Seems like a quick win :)

@vospascal
Copy link

love to see the power of the opensource community at work !

Copy link
Collaborator

@justinfagnani justinfagnani left a comment

Choose a reason for hiding this comment

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

thanks @lukewarlow!

Just a couple of very small comments.

schema.d.ts Show resolved Hide resolved
schema.d.ts Show resolved Hide resolved
@justinfagnani
Copy link
Collaborator

@lukewarlow the scheme needs to be regenerated

@lukewarlow
Copy link
Contributor Author

That's done now sorry

@lukewarlow
Copy link
Contributor Author

@justinfagnani is there anything else you need me to change?

Copy link
Collaborator

@justinfagnani justinfagnani left a comment

Choose a reason for hiding this comment

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

Thanks @lukewarlow !

@justinfagnani justinfagnani merged commit 2570079 into webcomponents:main May 6, 2024
2 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.

Add support for CSS CustomState
5 participants