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

need to save the background-image url with the whole query #785

Closed
zww-v5 opened this issue Oct 30, 2018 · 2 comments
Closed

need to save the background-image url with the whole query #785

zww-v5 opened this issue Oct 30, 2018 · 2 comments

Comments

@zww-v5
Copy link

zww-v5 commented Oct 30, 2018

I have a problem about the background-image url. There is a background-image url like "background-image: url('../../css/img/slice/icon-arrow-bottom.svg?fill=#444');" in
my css files. I need to retain the 'fill=#444' for the other handling. But when the files dealed with by css-loader, the background-image url change to
"background-image: url(src/css/img/slice/icon-arrow-bottom.svg?#444);". The 'fill=' was lost.
I find this problem caused by the 'idx = url.indexOf("?#");'(in css-loader/lib/loader.js ). In this code, it just retain the query like '#444'. If you can change the code to 'idx = url.indexOf("?");'?
It can retain the whole query with this code.

@alexander-akait
Copy link
Member

It is impossible. also it is bad practice use query for this purpose, please use issue template in future. Describe you use case, why you need query?

@alexander-akait
Copy link
Member

Closing due to inactivity. Please test with latest version and feel free to reopen if still regressions. Thanks!

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

2 participants