-
Notifications
You must be signed in to change notification settings - Fork 27
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
Conversation
Can someone review this PR? |
This looks fine to me 👍 Konnor was already working on a PR to add this to @justinfagnani can you also take a look? Seems like a quick win :) |
love to see the power of the opensource community at work ! |
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.
thanks @lukewarlow!
Just a couple of very small comments.
@lukewarlow the scheme needs to be regenerated |
That's done now sorry |
@justinfagnani is there anything else you need me to change? |
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.
Thanks @lukewarlow !
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.