-
Notifications
You must be signed in to change notification settings - Fork 11.4k
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
Add support for apple-touch-icon #2774
Comments
I don't get it. Can you explain it further? |
Sure. First sorry for the poor wording that reflect my ignorance of html. When using Safari on an iOS device, one has the option to However, this icon needs to be referenced as After building, and deploying my webpage (to my own domain not to gihub), I added the icon on iOS and obtained the icon shared above on the left (with just my initial). So what I did was to modify the file
below this line
and deploy again to my own domain. And now I am able to get the icon above on the right (with my short cut picture) when adding a shortcut to my iPhone home screen. I hope this makes sense. Best, |
Now it makes sense. Tbh what doesn't make sense for me (but I am not against it) is creating a shortcut for your portfolio website on your phone. But it can be a nice addition. Would you mind sending a PR for this? Maybe create a field for it in |
Sure I will. Indeed I am not using a self-referred shortcut on my phone but it is nice to be able to share it with relatives. |
Sorry if my question is naive but the only place in the code where I can see a reference to Does it mean that I just need to duplicate this |
Yes, but pointing to the file defined in |
Have you checked that your feature request isn't already filed?
Description & Motivation
Adding this line:
<link rel="apple-touch-icon" href=link_to_my_icon_asset />
below theshortcut icon
line insideindex.html
will make that: adding a shortcut on an iOS device will use the chosen icon instead of a generic one.The change is very simple.
Pitch
No response
Alternatives
No response
Additional context
No response
The text was updated successfully, but these errors were encountered: