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

Test with Safari Technology Preview #929

Closed
raucao opened this issue Apr 12, 2016 · 12 comments
Closed

Test with Safari Technology Preview #929

raucao opened this issue Apr 12, 2016 · 12 comments

Comments

@raucao
Copy link
Member

raucao commented Apr 12, 2016

Apple claims they considerably improved IndexedDB, so we can probably use it in Safari as well soon:

https://webkit.org/blog/6017/introducing-safari-technology-preview/

@raucao raucao added the ready label Apr 12, 2016
@raucao raucao changed the title Test with Safari Preview Test with Safari Technology Preview Apr 12, 2016
@gregkare
Copy link
Member

Safari 10.0 is supposed to support 100% of the recommended IndexDB standard: https://developer.apple.com/library/prerelease/content/releasenotes/General/WhatsNewInSafari/Articles/Safari_10_0.html

@raucao
Copy link
Member Author

raucao commented Jun 14, 2016

Is that already released, or if not, when is it expected?

@galfert
Copy link
Member

galfert commented Jun 14, 2016

I think it's supposed to be released with the next macOS version (Sierra) in the fall.

@raucao
Copy link
Member Author

raucao commented Jun 14, 2016

LOL macOS

@raucao
Copy link
Member Author

raucao commented Jun 14, 2016

Cool. So we just need to change it sometime before fall in rs.js.

@galfert
Copy link
Member

galfert commented Jun 14, 2016

I'm downloading the private beta and will install it on my old Macbook to check it out.

@galfert
Copy link
Member

galfert commented Jul 8, 2016

I tested Webmarks with the latest Safari Technology Preview Release 8 (Safari 9.1.2) and it all worked flawlessly using IndexedDB. 🎉

@galfert galfert closed this as completed Jul 8, 2016
@raucao
Copy link
Member Author

raucao commented Jul 8, 2016

Sounds great. However, I thought we'd have to remove code that makes Safari not use IndexedDB?

@galfert
Copy link
Member

galfert commented Jul 9, 2016

No, we only have a specific Browser/device check for Android <= 4.3. For everything else we try to create an actual database to determine if IndexedDB is supported.

@raucao
Copy link
Member Author

raucao commented Jul 9, 2016

Nice!

Did you also test how it behaves in private mode?

@galfert
Copy link
Member

galfert commented Jul 11, 2016

Behaves the same as the stable Safari in private mode.

@raucao
Copy link
Member Author

raucao commented Jul 11, 2016

dislike

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

No branches or pull requests

3 participants