-
Notifications
You must be signed in to change notification settings - Fork 154
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
[CLEANUP] Move CSS parsing to a separate class #1014
Conversation
This still needs a |
This change is analogous to the second commit in #1015, so you can see how it might turn out when we introduce |
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 like this refactoring a lot.
6ff10cf
to
37bb360
Compare
Also clarify the class description in the DocBlock.
This clarifies that not all at-rules are rendered. Also rename the associated property, and add note in the DocBlock for the method that `@charset` rules are discarded.
bdd6ac0
to
d73124d
Compare
I've made the changes initially suggested. I've also finished off writing the |
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.
Already looks very good! I've added my comments.
Have commited changes as suggested. |
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.
Beautiful!
Part of #734 and #544. Relates to #994.