Skip to content
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

Non-working link with “Add to Home Screen” on iOS #335

Closed
j-conrad opened this issue Apr 28, 2018 · 14 comments
Closed

Non-working link with “Add to Home Screen” on iOS #335

j-conrad opened this issue Apr 28, 2018 · 14 comments
Labels
enhancement work that enhances an existing feature

Comments

@j-conrad
Copy link
Contributor

In Safari (iOS 11), while in a hub, using the “Add To Home Screen” function creates a shortcut that resides with other apps as expected. However, it creates an (A-frame?) icon and subsequently opening that app appears to open a webview instead of opening up an actual bookmark to that hub, which means the hub space doesn't open.

Note: using “Add To Home Screen” on the hubs.mozilla.com landing page makes a usable app button.
It would be useful to be able to make a series of app icons to easily return to previously-created/visited hub spaces.

Platform: iOS 11 / Safari / iPhone 6s

@cvan
Copy link
Contributor

cvan commented Apr 28, 2018

thanks for filing! this is being introduced in PR #317 (in progress, under reviewg). agreed that it's important - will fix it soon 👍

@robertlong robertlong added the enhancement work that enhances an existing feature label Oct 19, 2018
@gfodor
Copy link
Contributor

gfodor commented May 31, 2019

this is (finally) getting looked at now via #1380. sorry it took so long :(

@misslivirose
Copy link
Contributor

Since 1380 was merged, @j-conrad, can you verify that this is no longer an issue?

@j-conrad
Copy link
Contributor Author

This is still not working. Just tested on Safari with iOS 12.4.
Same result as described in the original issue— app icon gets created, but only for hubs.mozilla.com, not the actual room you created. Icon is also still the A-Frame logo.

@misslivirose
Copy link
Contributor

Thanks for verifying!

@gfodor
Copy link
Contributor

gfodor commented Jul 23, 2019 via email

@gfodor
Copy link
Contributor

gfodor commented Jul 23, 2019 via email

@misslivirose
Copy link
Contributor

Great, that makes sense. So the bug (in this case) would be if the A-Frame logo is still what shows up for iOS users when they save it to the home screen, if I'm understanding correctly. The rest is by design with the favorites feature and manifest.

@gfodor
Copy link
Contributor

gfodor commented Jul 23, 2019 via email

@j-conrad
Copy link
Contributor Author

j-conrad commented Jul 24, 2019

Ok then. The A-frame icon only shows up if I attempt to do ‘Add to Home Screen’ with a Hubs room. It does not occur if I do it with the main landing page. Although it’d be cool if it used some logo other than a blank white rounded square. Looks like it attempts to grab a screenshot of the page or something.
— scratch that— It seems to work ok now. I get a little mini image of the page now. Must’ve been a holdover from the last time I did it.

@gfodor
Copy link
Contributor

gfodor commented Jul 24, 2019 via email

@cvan
Copy link
Contributor

cvan commented Jul 24, 2019

hiya, folks! opened PR #1557 to address.

some context around the icon / manifest weirdness:

  • Apple shipped support in Safari for iOS 11.1 for parsing web-app manifests via <link rel="manifest"> but (for some reason) ignores the icons defined in the manifest. A <meta name="apple-touch-icon"> tag appears to be still required for the time being. (I assume that'll change in an upcoming release.)
  • Having the Hubs scene page define its own <meta name="apple-touch-icon"> will fix the issue of A-Frame's injecting its own (which it does if it's missing, along with a few other Apple-proprietary <meta> tags.

@joshmarinacci
Copy link
Contributor

I think this is fixed now, yes?

@misslivirose
Copy link
Contributor

Confirming that the A-Frame logo no longer appears for a Hubs room. It's now the duck logo :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement work that enhances an existing feature
Projects
None yet
Development

No branches or pull requests

6 participants