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
Implement loading of CSS Modules. Resolves #1991. #1997
Conversation
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.
We just need to be very clear on the compatibility here, I've included some notes.
Great to see the new docs pages, that is very useful.
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.
Seems ready to merge to me. The docs page is looking great.
Will be a very nice feature to mention for v6!
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.
Just pushed a small whitespace change to the docs. Is there anything else we need to wait on before merging?
I think it's pretty clear, and we can improve the docs over time as necessary. Will write a clear release description and ship a v6 shortly. Still on the fence over whether it should be a blog post or not, wdyt? |
See #1991. Note that in addition to the tests in this PR I verified in a test repo that the css actually gets loaded and can be applied to dom elements.