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

Some feedback #2

Open
filipbech opened this issue Jun 21, 2016 · 2 comments
Open

Some feedback #2

filipbech opened this issue Jun 21, 2016 · 2 comments

Comments

@filipbech
Copy link

Thanks for the work. This could turn out very useful for many people...

I like the approach, and would love for this to be developed further. A couple of comments/questions though.
Also, please let me know if you want more code-style/QA feedback - Im happy to help.

  • iPod? You only initiate if platform is iPad or iPhone (ln 152), but later you check to see if its iPod (ln 121)
  • best-icon? why not just add them all and let the browser do the magic. If safari changes its logic this would no longer follow the platform as intented
  • I think web-app manifest urls are calculated from the base-url, which XHR is not. Minor thing, but base-urls are very common when routing clientside

I would love to see the start_url thing implemented like Paul Kinlands issue...

@firtman
Copy link
Owner

firtman commented Jun 23, 2016

Thanks @filipbech

  • Regarding iPod miss, I'll check it.
  • For the best icon, there is an explanation: there is a bug today on iOS where when you add the page to the homescreen, a screenshot is taken instead of the icon; it happens randomly, and I'm talking only when the network is still up. Is the network is down when the user presses "Add to Home Screen", then the icon will never appear, therefore the best approach today is to set the icon in base64 to guarantee it's going to be used by Safari. It's part of my ToDo list, but that's the reason why I didn't let Safari just pick the icon.
  • I'll do a test suite for the base-url thing; sounds like something to be solved.

Regarding start_url like Paul says, look at my answer. I think I'm probably missing something there.

@filipbech
Copy link
Author

thanks for getting back to me... great explanation of the ios icon-bug... i didn't know that!
You can close this issue as it was just for getting in touch...
let me know if there is anything I can do to help!

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

No branches or pull requests

2 participants