vue-skeleton icon indicating copy to clipboard operation
vue-skeleton copied to clipboard

FaviconsWebpackPlugin

Open nossienl opened this issue 6 years ago • 2 comments

when setting this to true, you also need to set title in webpack.partial.conf.plugins.js if you don't do this it tries to guess the name of app/website thru the function guessAppName and this tries to looks that up in package.json. I don't think everybody changes the package.json to match the website/app name.

maybe we can retreive website title from index.html and set that in webpack.partial.conf.plugins.js

nossienl avatar Aug 15 '19 06:08 nossienl

I don't think using the html is a feasible solution. The title in the html can also be changed at runtime, which won't be picked up yet at build time. My suggestions:

  1. Don't guess the title, but error indicating that you need to add a title
  2. Use the package.json value but error if that still equals "Vue Skeleton". Added advantage is that devs will more often update the name in the package json :)

flut1 avatar Aug 15 '19 08:08 flut1

Having a wrong favicon title is something you might not spot early, so using an automatic value from the package.json might also be dangerous.

ThaNarie avatar Aug 15 '19 09:08 ThaNarie