{"action":"created","issue":{"url":"https://api.github.com/repos/dfahlander/Dexie.js/issues/148","labels_url":"https://api.github.com/repos/dfahlander/Dexie.js/issues/148/labels{/name}","comments_url":"https://api.github.com/repos/dfahlander/Dexie.js/issues/148/comments","events_url":"https://api.github.com/repos/dfahlander/Dexie.js/issues/148/events","html_url":"https://github.com/dfahlander/Dexie.js/issues/148","id":113925319,"number":148,"title":"indexedDBShim and Dexie","user":{"login":"isuda","id":25780,"avatar_url":"https://avatars.githubusercontent.com/u/25780?v=3","gravatar_id":"","url":"https://api.github.com/users/isuda","html_url":"https://github.com/isuda","followers_url":"https://api.github.com/users/isuda/followers","following_url":"https://api.github.com/users/isuda/following{/other_user}","gists_url":"https://api.github.com/users/isuda/gists{/gist_id}","starred_url":"https://api.github.com/users/isuda/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/isuda/subscriptions","organizations_url":"https://api.github.com/users/isuda/orgs","repos_url":"https://api.github.com/users/isuda/repos","events_url":"https://api.github.com/users/isuda/events{/privacy}","received_events_url":"https://api.github.com/users/isuda/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-10-28T21:23:41Z","updated_at":"2015-10-28T22:53:37Z","closed_at":null,"body":"Dexie prefers to use the indexedDBShim if it is loaded, even if native indexedDB is available (and not buggy). \r\nThe default probably should be to use native indexedDB if available, falling back to indexedDBShim. Or, only use the shim if it is actually shimming. \r\n\r\nThis is especially important in cases going forward where WebSQL is not available, but IndexedDB is (WKWebView on iOS9 for example). If the shim were to be included there, everything would break.\r\n\r\nThis would be a huge breaking change if implemented though, as it would break existing apps using WebSQL when indexedDB is available, which is a huge problem.\r\n\r\nThe change is fairly simple: https://github.com/AyogoHealth/Dexie.js/commit/b5f7965c422257aeffeffe2f0c2f272490578b06\r\n\r\nJust wanted to get your thoughts on this."},"comment":{"url":"https://api.github.com/repos/dfahlander/Dexie.js/issues/comments/152021924","html_url":"https://github.com/dfahlander/Dexie.js/issues/148#issuecomment-152021924","issue_url":"https://api.github.com/repos/dfahlander/Dexie.js/issues/148","id":152021924,"user":{"login":"isuda","id":25780,"avatar_url":"https://avatars.githubusercontent.com/u/25780?v=3","gravatar_id":"","url":"https://api.github.com/users/isuda","html_url":"https://github.com/isuda","followers_url":"https://api.github.com/users/isuda/followers","following_url":"https://api.github.com/users/isuda/following{/other_user}","gists_url":"https://api.github.com/users/isuda/gists{/gist_id}","starred_url":"https://api.github.com/users/isuda/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/isuda/subscriptions","organizations_url":"https://api.github.com/users/isuda/orgs","repos_url":"https://api.github.com/users/isuda/repos","events_url":"https://api.github.com/users/isuda/events{/privacy}","received_events_url":"https://api.github.com/users/isuda/received_events","type":"User","site_admin":false},"created_at":"2015-10-28T22:53:37Z","updated_at":"2015-10-28T22:53:37Z","body":"I guess my use case is that I want to use the shim's default behaviour, which is to use the shim if needed.\r\n\r\nDexie however, will force use the shim if its loaded, instead of just defaulting to the shim's behaviour.\r\nI can understand forcing use of the shim to maintain consistency across all browsers though, but I also don't want to have to conditionally load the shim at runtime either."}}