Skip to content
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

Autoprefix styles #2

Closed
vadimdemedes opened this issue Jun 30, 2017 · 4 comments
Closed

Autoprefix styles #2

vadimdemedes opened this issue Jun 30, 2017 · 4 comments

Comments

@vadimdemedes
Copy link
Owner

Would be cool to use inline-style-prefixer to autoprefix style props.

@fregante
Copy link
Collaborator

fregante commented Jul 2, 2017

Style shouldn't be in the JS in the first place. It might make sense when the JSX is dynamic but not when it's static. Plus it's not a very small dependency: 70KB unminified

@vadimdemedes
Copy link
Owner Author

Now that I think of it, it might be an over bundling it with features. Let's postpone it and see if other people have different thoughts.

@mDibyo
Copy link

mDibyo commented Jul 2, 2017

Agree with @bfred-it. One thing worth considering might be creating automatic classes for elements and generating css placed inside style tags in the head.

@vadimdemedes
Copy link
Owner Author

That's definitely out of scope of dom-chef and is a job for libs like glamor.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants