GeezerVison is showing github.com — cursor.request · Issue #255 · w3c/IndexedDB
DescriptionRequests for cursors are different as they can succeed many times. When you call cursor.continue() the success of that operation is provided in the request returned by source.openCursor().
If I want to give an object store to another piece of code, I can just pass the object store. However, if I want to pass the cursor elsewhere, I also have to pass the related request.
Should cursor.request return https://w3c.github.io/IndexedDB/#cursor-request? Reacted by Joshua Bell, Andrew Sutherland, Soviut, Tim Weißenfels, Ståle Raknes, Forge, Vinícius Lemes, erhise, Claude Kirke, Oleksandr Nikitin and 3 more Activityinexorabletash commented on Mar 4, 2019 I've run into wanting this for prollyfills. SGTM.
Should be easy to spec/test/ship. jakearchibald commented on Mar 5, 2019 ContributorAuthor More actions Yeah, I ran into this trying to make wrap() and unwrap() functions that 'upgrade' IDB objects. Works for everything except cursors. jakearchibald commented on Mar 6, 2019 ContributorAuthor More actions I'll take this on.
I've been wanting to document how standards are developed for a while, so @surma and I will film ourselves updating the spec and writing tests. Reacted by Billie Hilton, Trent Monahan, Theo, Danilo Fuchs, Pauline Nemchak, dpeter99, Petri-Oosthuizen and David Brito inexorabletash commented on Mar 6, 2019 aliams commented on Mar 6, 2019 No objections here. It sounds like we're just exposing a property that is already initialized as part of the cursor and should live as long as the cursor. Reacted by Jake Archibald asutherland commented on Mar 6, 2019 Did my thumbs-up not say it all? ;) This sounds good from a Gecko/Firefox perspective. We have the request available and Jake's magical upgrade use-case sounds both reasonable and essential to help move IDB forward from an API that only an implementer could love to the favorite API of developers everywhere! Reacted by Jake Archibald and Pauline Nemchak Reacted by Jake Archibald, Joshua Bell, Mark Tomlin, Danilo Fuchs, Pauline Nemchak and Joel Mason beidson commented on Mar 6, 2019 Reacted by Jake Archibald inexorabletash commented on Mar 6, 2019 Thumbs everywhere! Thanks all!
Thanks @jakearchibald and @surma - looking forward to the PRs. Blink change should be trivial once WPTs are in. inexorabletash commented on Mar 18, 2019 asutherland commented on Mar 19, 2019 Soviut commented on Apr 16, 2019 Reacted by Timo Mühlbach, Kristoffer Lindström, Trent Monahan, Vinícius Lemes, Nicola Dal Maso, Brendan McGuire, Belan, Dennis Kugelmann, Jonas Kuske, Theo and 6 more Reacted by Marceline Szmadzinski, Ståle Raknes, Grzegorz Bielski, Michael Richins, Martino di Filippo, Kristoffer Lindström, Forge, Trent Monahan, Mark Tomlin, Belan and 12 more inexorabletash commented on May 3, 2019 FYI, @Yannic's change just landed in the Chromium repo so this should be available for testing in the next Chrome Canary build. Reacted by Yannic, Trent Monahan, Theo, Jake Archibald and Pauline Nemchak julienw commented on Apr 27, 2020 Reacted by Jake Archibald, Joshua Bell and Pauline Nemchak inexorabletash commented on Apr 27, 2020 asutherland commented on Apr 27, 2020 Reacted by Jake Archibald
MetadataMetadataDevelopmentNo branches or pull requestsIssue actions
|