-
Notifications
You must be signed in to change notification settings - Fork 34
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
What about polyfills? #2
Comments
Hi. Thank you :) I have not tried using SyncedDB with polyfills. I'm using it an Cordova app myself where IndexedDB is supported. SyncedDB does deliberately not use any exotic features of IndexedDB so polyfilling should be possible. localForage is not bad. If localStorage + async is all one needs then it's great. But for anything but the most basic client site storage it falls short. |
@paldepind thanks. |
As @paldepind said if there is a standard and fast wrapper library of IndexedDB accessing, we can choose it. But I'm not sure we should support localStorage (slow, no indexing) or WebSQL (obsolete) here. What do you think? |
Okay let's say we don't support LocalStorage an WebSQL. Reasons: LocalStorage must very slow as a DB use (JSON.parse, JSON.stringify... and not any indexing). And I believe W3C abandoned WebSQL and recommends IndexedDB. |
@piglovesyou sorry, I was lost in a world of messages :) and this was curiosity only. |
I agree. IndexedDB is the most powerful client site database. Thus polyfilling it with weaker solutions will probably have limitations. |
Hi @paldepind, your work is very good.
Have you checked your library with any polyfill?
In a commercial app, I'm using mozilla localForage, an localStorage/IndexedDB/Webkit abstraction, very limited but lightweight and confiable.
The text was updated successfully, but these errors were encountered: