search

nuxt-community / svg-module

Super simple svg loading module for Nuxt.js
MIT License
338 stars 35 forks source link

Allow to configure SVGo options #82

Closed mrleblanc101 closed 3 years ago

mrleblanc101 commented 3 years ago

@sam3d Could you please look into this when you have some time ?

manuelodelain commented 3 years ago

Thanks @mrleblanc101 for the PR! As the svgo option is used only by the vue-svg-loader I think it should be nested into another level of option, for example: 

{
  svg: {
    inline: {
      svgo: {}
    }
  }
}

@sam3d @mrleblanc101 What are your thoughts on that?

mrleblanc101 commented 3 years ago

I'll go with vueSvgLoader since that's the name of the underlying package. I'll update the PR tonight

{
  svg: {
    vueSvgLoader: {
      svgo: {}
    }
  }
}
mrleblanc101 commented 3 years ago

@manuelodelain I've updated the PR