-
Notifications
You must be signed in to change notification settings - Fork 245
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
indexed-db adapter error in source #196
Comments
Yes please, that would be very nice! |
Yes, please update this. I spent about an hour scratching my head as to why Lawnchair wasn't working in Firefox! I don't think any of the source files on the website are up to date. |
I see what you are talking about on developer.mozilla.org. However, the following line of code checks whether var READ_WRITE = (getIDBTransaction() && 'READ_WRITE' in getIDBTransaction()) ? getIDBTransaction().READ_WRITE : 'readwrite'; I don't use IndexedDB so, all of you who do use it are saying this line does not work? |
The thing is, I haven't used Firefox in over ten years and I don't use IndexedDB at work so I can test this in short order. The code Brian has is what Mozilla recommends and it looks good to me; however, I have seen browsers stub in enough stuff to make it look like it has a feature when it does not (e.g. Safari acting like it has toLocaleString for Number and Date). So, someone with this problem please go to the console in Firefox and tell us what the value of |
I think there are two separate issues in here. Lawnchair and the indexed-db adapter both work for me in Firefox 36 (current latest). There's no problem with the library in my mind. The issue is that the code on the webpage is not the same or as current as the code in Github. The old code on the website caused errors in Firefox that were resolved just by using the Github code. I was just asking if the site owner/operator could update the code on brian.io or just remove it and link directly to Github. |
@mikegioia , @MarkMYoung - yes indeed, according to what I have scanned there has been quite a bit of work done on github source, that has not yet made it through to the "official" releases on the lawnchair downloads page. Speaking under correction, this is not just indexed-db related code. |
the indexed db adapter on the lawnchair home page downloads section does not work (uses deprecated constants, has fixed test name for store).
The source in github works.
You should update thev version on the lawnchair downloads....
cheers
MT
The text was updated successfully, but these errors were encountered: