-
Notifications
You must be signed in to change notification settings - Fork 313
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
Switch to ironDB? #539
Comments
There's no guarantee that ironDB will solve this issue. It also tries to save the DB to a cookie, which is impractical for our case, since a cookie can only store 4093 bytes. I think the next best solution would be to add popups (with an option to disable) that remind the users to backup their styles. |
If we decide to switch to something, we can switch to chrome.storage by default and offer an option to use IndexedDB for those who wants speed. The slowness of chrome.storage shouldn't be a big issue: it's observable only when all styles are retrieved from the storage at startup so instead of say 40ms it would be 150ms for a 6MB worth of styles. Could be worse though on a slower computer. But then we use a cache. Updates of individual styles are fast in both chrome.storage and IndexedDB. |
There are many users got data loss bug due to browser bugs (#278, #528) that we can't fix them in the extension. @vsemozhetbyt has suggested an ironDB library which can push data into multiple browser storage just like what tophf tried to do.
I'm not sure if it is a good idea though. It is basically duplicating the data.
The text was updated successfully, but these errors were encountered: