queryIndexedDB icon indicating copy to clipboard operation
queryIndexedDB copied to clipboard

Support webkit

Open ebidel opened this issue 13 years ago • 4 comments

Because this lib is awesome and should work everywhere possible :)

ebidel avatar Mar 10 '12 02:03 ebidel

I wonder if @jensarps idbwrapper is an appropriate dependency here

paulirish avatar Mar 10 '12 17:03 paulirish

@paulirish link?

The only Gecko-specific APIs that I'm aware of is index.getAll(). This could easily be changed in my code, preferably using feature detection and a helper that will provide the same functionality in non-Gecko environments. Anyways, I'll happily accept patches. :)

philikon avatar Mar 12 '12 23:03 philikon

IDBWrapper: https://github.com/jensarps/IDBWrapper

MDN has a nice example of implementing getAll(): https://developer.mozilla.org/en/IndexedDB/Using_IndexedDB#Using_a_cursor (2nd example in that cursor section).

ebidel avatar Mar 12 '12 23:03 ebidel

Yep, faking/working around a missing getAll() isn't a problem. I just want to ensure we use the fast path in Gecko.

Thx for the link to idbwrapper. I'll take a closer look at it, but for me this is not an acceptable dependency, for various reasons, but in particular because we're planning on using queryIndexedDB in Gecko itself for dogfooding some of our own databases.

philikon avatar Mar 13 '12 00:03 philikon