-
Notifications
You must be signed in to change notification settings - Fork 4
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
Suggestion: Add section about overriding icons #4
Comments
Interesting, I haven't looked at this yet - what is your recommendation for a method to override an icon? Data URIs? Or using a Chrome Local Override? What I have done on macOS for PWAs is to right-click -> Get Info on the app created by Chrome and then drag a new But I'm not sure if that always works for every use case and unsure how to do this for Windows and Linux. |
Sure, you can replace the desktop icons after the fact, but it's quite a bit of hassle, and you wouldn't be able to install the app in the first place if your app manifest fails the installability check (it's mandatory to have an icon of at least 144x144). Local override doesn't seem to work for replacing app manifest icons. Data URIs directly in the For Linux, Chrome / Chromium would install the icons to e.g. |
Nice, sounds like a good trick to add! Want to try doing a PR for adding an icon via a data URI? I'm assuming that you'll be able to copy+paste some of the content from the other tricks :) |
To provide your own icons, use data URIs. Trying to override non-existent paths does not work (it will not pass the installability check). Serving files from localhost or another origin would not work well either, as it is very likely to be blocked by CSP.
The text was updated successfully, but these errors were encountered: