-
-
Notifications
You must be signed in to change notification settings - Fork 6.3k
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
PWA: Can't edit icons in vue.config.js
#4069
Comments
same problem |
For versions before With version 4, the |
We currently don't have plugins documented directly on the documentation website, but simply link to their README files on github on the this is the reason for OPs mistake. They read the README and found docs there that they should be able to make the change, because the README on the Sorry for that :( I think we should dedicate some time to create some minimal docs for each core plugin on the documentation website to prevent such confusion in the future. |
@LinusBorg I would like to work on this 😅 |
That's awesome! Since this issue here is about a specific misunderstanding with one plugin, would you open a new issue where we can discuss the approach? Then we can close this issue here with reference to the new one. |
@LinusBorg done! |
Closing in favor of #4117 |
@LinusBorg Are you sure that |
Did you update all of the For reference: |
@LinusBorg Yes, I did. BTW, the implementation you attached is regarding the |
Right. Got those mixed up. However I now spotted the mistake:
|
@LinusBorg Gotcha, thanks, it works! |
@LinusBorg nice, thats works for me |
@LinusBorg Would it pe possible to add the icon parameter in the example of For people like me who start discovering the world of pwa directly with vue-cli its confusing on how is generated the icons parameter of the generated manifest.json I lost an hour to finally end up on this issue to find the solution. Besides this little issue the PWA integration in Vue CLI is really great ! You made a really great work !! |
I agree with ouraios, the documentation should mention the |
I still couldn't figure out how to change my favicon. Can someone help me? |
@Syiabonga the bug is still there. it is a bug, not a documentation issue. |
thanks man! |
Can someone tell me please why is this not written in the documentation? Wasted so much time on this... Once you know it is logic to set it in pwa.manifestOptions.icons, but since it doesn't mentioned in the doc, it's very hard to find out. |
Version
3.8.0
Environment info
Steps to reproduce
yarn serve
What is expected?
The icons in the manifest should be the same path you entered in the config file.
What is actually happening?
The icons that used are the default.
https://github.com/vuejs/vue-cli/blob/dev/packages/%40vue/cli-plugin-pwa/lib/HtmlPwaPlugin.js#L15
Chrome debugger:
Error while trying to use the following icon from the Manifest: http://localhost:8087/img/icons/android-chrome-192x192.png (Download error or resource isn't a valid image)
The text was updated successfully, but these errors were encountered: