fix(edge): apply bable-loader on modern builds too

nuxt-community / svg-module

Super simple svg loading module for Nuxt.js

MIT License

340 stars 35 forks source link

fix(edge): apply bable-loader on modern builds too #35

Closed suits-at closed 4 years ago

suits-at commented 4 years ago

Due to MS Edge uses modern build, but does not properly work with the rest operator used in the svg-to-vue package, the current version of this package does not work in MS Edge out of the box. By applying bable-loader on modern builds too, this issue could be solved. This would also fix https://github.com/nuxt-community/svg-module/issues/11.

suits-at commented 4 years ago

Actually babel does not transpile modern mode at all at the moment. This could also lead to others issues than the current one regarding the rest operator in MS Edge.

simllll commented 4 years ago

voting for this, pinging @sam3d :+1:

sam3d commented 4 years ago

Thanks for the PR!

suits-at commented 4 years ago

Thanks for merging the PR @sam3d. But unfortunately I can't find the code inside the master branch. Also in the history it seems to be missing. Is it possible it got lost somehow?