yuguo / gulp-import-css

Import several css files into a single file
MIT License
15 stars 6 forks source link

0.1.3 breaks data: URLs #7

Open josip opened 7 years ago

josip commented 7 years ago

In 0.1.3 url("data:...") URLs are transformed into:

src:url("../../some/random/path/data:...");

Behaviour was correct in 0.1.2 where these would go unchanged.

yuguo commented 7 years ago

I write a simple test case, and everything just work fine.(Maybe I need to submit a test for the module later).

.home {background:url("data:image/png;base64,......")}

Could you add your complete test file, please?

kikhtenko commented 7 years ago

the same issue... unsure what and when was changed but behavior is the same background-image: url("data:image/png .... is transformed to background-image: url("../../bower_components/angular-toastr/dist/data:image/png

kikhtenko commented 7 years ago

As far as I can see from the code previously you were transforming only css files. The new version transforms other links which causes this issue.

ingro commented 7 years ago

0.1.3 also breaks import on windows:

Things like src:url(../fonts/fontawesome-webfont.eot) became src:url(..\fonts\fontawesome-webfont.eot)