Skip to content

Add Encore.disableCssExtraction() to the public API #539

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

Merged
merged 1 commit into from
Mar 25, 2019

Conversation

Lyrkan
Copy link
Collaborator

@Lyrkan Lyrkan commented Mar 11, 2019

This PR adds an Encore.disableCssExtraction() method that allows to disable the mini-css-extract-plugin and use the style-loader instead.

It can be used to solve various problems that, until now, required a really ugly workaround that relied on our internal implementation (for instance the following commit probably broke some builds that used previous versions of it: 6867443#diff-8beacd21a12ca072bafa4e8e3f1aae6b).

Related issues: #3, #256, #348, #527

@Lyrkan Lyrkan force-pushed the disable-css-extract branch from 725833d to 965e83e Compare March 14, 2019 20:06
@Lyrkan Lyrkan force-pushed the disable-css-extract branch from 965e83e to 347feed Compare March 17, 2019 11:55
@weaverryan
Copy link
Member

Yea, this IS useful. I fully realize that people may want to opt-in to the "inconsistent" experience between dev & prod because of nice things like HMR. Good idea!

@weaverryan weaverryan merged commit 347feed into symfony:master Mar 25, 2019
weaverryan added a commit that referenced this pull request Mar 25, 2019
…kan)

This PR was merged into the master branch.

Discussion
----------

Add Encore.disableCssExtraction() to the public API

This PR adds an `Encore.disableCssExtraction()` method that allows to disable the `mini-css-extract-plugin` and use the `style-loader` instead.

It can be used to solve various problems that, until now, required a really ugly workaround that relied on our internal implementation (for instance the following commit probably broke some builds that used previous versions of it: 6867443#diff-8beacd21a12ca072bafa4e8e3f1aae6b).

Related issues: #3, #256, #348, #527

Commits
-------

347feed Add Encore.disableCssExtraction() to the public API
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants