Skip to content

Fix sizes property name to size in README#80

Open
PetrDlouhy wants to merge 1 commit intosilviolleite:masterfrom
PetrDlouhy:patch-1
Open

Fix sizes property name to size in README#80
PetrDlouhy wants to merge 1 commit intosilviolleite:masterfrom
PetrDlouhy:patch-1

Conversation

@PetrDlouhy
Copy link
Contributor

Fix #38
The sizes property name in README doesn't match with the code, where is it named size:

<link rel="apple-touch-icon" href="{{ icon.src }}" sizes="{{ icon.size }}">

@JelleBootsma
Copy link
Contributor

I believe this should be fixed in pwa.html, instead of in the readme. The icon property should be called sizes, as this also injected into the manifest.json and here it should be sizes, as can be seen on MDN Web Docs

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

sizes not in html

2 participants