gridsome-plugin-pwa icon indicating copy to clipboard operation
gridsome-plugin-pwa copied to clipboard

Bug: msapplication head meta tags read as options and result as undefined

Open chorijan opened this issue 5 years ago • 6 comments

There is a bit of code that sets head meta tags for msapplication by reading options Screenshot 2019-11-12 at 16 56 12

But the dont get exposed publicly, can't be set and don't have defaults Screenshot 2019-11-12 at 16 56 35

which results in undefined values being output Screenshot 2019-11-12 at 17 00 54

chorijan avatar Nov 12 '19 16:11 chorijan

@chorijan would you like to provide default options for them in a PR ? And these properties should work if you supply those properties in your project's gridsome.config.js.

rishabh3112 avatar Nov 13 '19 15:11 rishabh3112

@rishabh3112 I did supply the options in my gridsome.config.js in the end, but that resulted with them still being undefined. I then tried also adding them to my main.js (push the head meta tags) which did end up adding them, but also left the previous two undefined(each meta tag was present twice)

chorijan avatar Dec 04 '19 09:12 chorijan

gridsome-plugin-pwa v0.0.10 should fix it. Update and let me know.

rishabh3112 avatar Dec 10 '19 13:12 rishabh3112

@rishabh3112 updated to the latest version and it does seem to create the props now, but there are duplicates as well - like I manually created them which i haven't (the only one i have manually created was the theme-color meta tag)

Screenshot 2020-01-10 at 10 51 38

chorijan avatar Jan 10 '20 09:01 chorijan

@chorijan I have tried locally and is working as expected. Please create a minimum reproducible repository for further investigation.

rishabh3112 avatar Jan 11 '20 16:01 rishabh3112

Similar issue: I couldn't access msTileImage from PWA options.

I noticed that the path to icon is generated using icon value in Manifest Options, but then adds -144x144.png, which result in the wrong path /assets/static/favicon.png-144x144.png (first .png is incorrect)

Unfortunately, plugin checks if the favicon exists, so I can't place a random icon value, e.g. icon: 'favicon', to generate correct path.

Wrong path to favicon:

Screenshot 2020-12-12 at 16 29 34

vitalikda avatar Dec 12 '20 14:12 vitalikda