Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

PostCSS should run on @import statements. #929

Merged
merged 1 commit into from
Oct 19, 2016

Conversation

nhunzaker
Copy link
Contributor

Looks like PostCSS isn't running on stylesheets included via @import within another stylesheet. The postcss-loader docs use ?importLoaders=1 in there configuration. Adding it fixed the issue.

Steps to reproduce:

  1. Create a new app with create-react-app
  2. Import App.css from src/index.css
  3. Remove import './App.css' from src/App.js

The -webkit- prefix for the animation style rule is gone! I've pushed up an example here:

https://github.com/nhunzaker/no-autoprefixer-issue

@nhunzaker nhunzaker force-pushed the nh-fix-autoprefixer-imports branch from a3035b9 to 0562acc Compare October 19, 2016 03:22
@gaearon gaearon added this to the 0.7.0 milestone Oct 19, 2016
@gaearon
Copy link
Contributor

gaearon commented Oct 19, 2016

LGTM, thanks!

@gaearon gaearon merged commit 59acae6 into facebook:master Oct 19, 2016
arunoda added a commit to storybookjs/storybook that referenced this pull request Oct 24, 2016
arunoda added a commit to storybookjs/storybook that referenced this pull request Oct 24, 2016
feiqitian pushed a commit to feiqitian/create-react-app that referenced this pull request Oct 25, 2016
jarlef pushed a commit to jarlef/create-react-app that referenced this pull request Nov 28, 2016
alexdriaguine pushed a commit to alexdriaguine/create-react-app that referenced this pull request Jan 23, 2017
@lock lock bot locked and limited conversation to collaborators Jan 22, 2019
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants