-
Notifications
You must be signed in to change notification settings - Fork 22.5k
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
CSSPageRule.style update to CSSPageDescriptors #34960
Conversation
Preview URLs
(comment last updated: 2024-07-31 09:55:42) |
0c3005f
to
2f2ece2
Compare
Thanks Hamish, a couple of comments for you to consider, but I think we're ready to go when you take a look at the feedback. I like the examples 👍🏻 |
5fb84c6
to
8a8066b
Compare
@bsmth Thanks for the review. I like the examples too. In general the live examples are much better than just having code because you know the code works. I've accepted your changes, and only open question is whether we can safely have properties side-by-side with our linter. Back to you. |
Co-authored-by: Brian Thomas Smith <brian@smith.berlin>
8a8066b
to
9f7413e
Compare
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.
Super, LGTM! Thank you, Hamish!
FF129 Adds support for
CSSPageRule.style
being aCSSPageDescriptors
rather than aCSSStyleDeclaration
.This adds docs.
Related work can be tracked in #34702