-
Notifications
You must be signed in to change notification settings - Fork 3
Keep CSS rules static #179
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
Changes from 11 commits
316a068
0e9aeff
bae36fb
465dbcc
1db36a6
dfa3f6e
eabe0c4
cb422ba
8167227
8467b91
fb6534b
9b0d320
9731e07
e5bb43d
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Please keep the rules available to projects currently using them, not just in the demo.
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. As @flo-dup suggest, either as default that can be override or at least as importable rules or css.
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. About the first suggestion, in fact I don't think we should put any default CSS in the component. That would mean injecting some CSS? Then how would a webpage with two components work, there would be conflicts with the two CSS? Instead as you said we could give one or several CSS examples. But to me it's not clear how we could include a CSS file in the release. Another simpler way to do this would be to leave the examples in the demo, and just add in the next migration guide the CSS which corresponds to the previous default
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Well ok. |
This file was deleted.
Uh oh!
There was an error while loading. Please reload this page.