Skip to content

Conversation

@lsanwick
Copy link
Contributor

@lsanwick lsanwick commented Jun 9, 2016

Right now, if I need to use assets like images, CSS, or fonts, Webpack won't load them and include them in the bundle. To make it possible to continue using Quik alongside these things, add the ability for the user to specify additional loaders to be included in the Webpack config.

@satya164
Copy link
Owner

satya164 commented Jun 9, 2016

@lsanwick Hey, thanks a lot!

I would like to add css and assets support to quik itself too. Just haven't got enough time for that yet. If you could a send a PR for that as well, it'll be awesome.

Also regarding this PR, can you add a line to the API doc also? Thanks :)

@lsanwick
Copy link
Contributor Author

lsanwick commented Jun 9, 2016

If you want to make it support them by default, I'd be happy to change to that. I'm sure most people would like to have at least basic CSS and image support when prototyping. :)

Right now, if I need to use assets like images, CSS, or fonts, Webpack won't load them and include them in the bundle.  To make it possible to continue using Quik alongside these things, add the ability for the user to specify additional loaders to be included in the Webpack config.
@lsanwick lsanwick force-pushed the ls/configurable-loaders branch from 5886162 to 71db8ee Compare June 9, 2016 20:53
@lsanwick lsanwick changed the title Allow user to provide additional loaders for Webpack feat(quik): Include CSS and image loaders in the webpack config Jun 9, 2016
@satya164 satya164 changed the title feat(quik): Include CSS and image loaders in the webpack config feat(loaders): Include CSS and image loaders in the webpack config Jun 11, 2016
@satya164 satya164 merged commit 4315f3b into satya164:master Jun 11, 2016
@satya164
Copy link
Owner

Sorry, this was delayed. Thanks a lot for the PR :D

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