-
Notifications
You must be signed in to change notification settings - Fork 461
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
Switch from radium to styled-components #112
Comments
Between those, Definitely styled-components. Emotion is fine as well but you will no have the same support and some missing features compare to styled-components. SASS, to be honest, I have almost a year without touching anything outside of JSCSS and unless I have to jump to something outside of React I will keep using styled-components or any other implementation of JSCSS |
I used When you say "missing features" about |
Seems like everyone is on-board with styled-components. As for styled-components vs emotion, if they are on par, I think it would be wiser to use styled-components as it has a much bigger user base, with 13 462 stars on Github vs 2 822 for emotion. |
@gabrielbull also the version |
Let's go for |
I can help with |
Happy to review and merge PRs if anyone wants to make a start. |
As mentioned in issue-129 , lets keep mind theming for the two operating systems. |
I this suggestion still under consideration? Since I'm currently building an OS X component library with styled-components (supporting light & dark mode) I might be open to contribute. |
@pixelass Yes, I would be happy to review any contributions. |
Right now I'm building things from scratch but I'll look into this library and see if I can at least add a DRAFT for some new components. |
What are the pros and cons of those three solutions?Which one is the most adapted for a component library?The new library should allow for extending proposed in #113Switch react-desktop from radium to styled-components. Can be done progressively.
Need to provide multiple ways for the user to extend the style of a component, including className property, style property, etc.
The text was updated successfully, but these errors were encountered: