Skip to content

[Encore] Encore.configureUrlLoader() documentation #9606

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 3 commits into from
Apr 19, 2018

Conversation

Lyrkan
Copy link
Contributor

@Lyrkan Lyrkan commented Apr 16, 2018

This PR adds the documentation for the Encore.configureUrlLoader() method (symfony/webpack-encore#296).

Not sure if it should be more detailed since the other options of the URL loader are typically not needed...

Copy link
Member

@javiereguiluz javiereguiluz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Lyrkan thanks a lot for this contribution! I love it (and I love the described feature too!! I needed a while ago and I really missed it!).

As you can see, I've made some rewords ... but they are only made to make the article more consistent with the style used in other parts of the Symfony Docs. I also update the article title to make it more SEO-friendly: Inlining files in CSS with Webpack URL Loader

Thanks again for your work ... and congrats on your first Symfony Docs contribution!

@javiereguiluz
Copy link
Member

It's merged now! Thank you.

@javiereguiluz javiereguiluz merged commit 5c87901 into symfony:3.4 Apr 19, 2018
javiereguiluz added a commit that referenced this pull request Apr 19, 2018
…n, javiereguiluz)

This PR was merged into the 3.4 branch.

Discussion
----------

[Encore] Encore.configureUrlLoader() documentation

This PR adds the documentation for the `Encore.configureUrlLoader()` method (symfony/webpack-encore#296).

Not sure if it should be more detailed since the other options of the URL loader are typically not needed...

Commits
-------

5c87901 Use the default article title
4d8700f Reword
6820867 Encore.configureUrlLoader() documentation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants