search

PolicyStat / ckeditor-spell-check-plugin-js-dev-edge

User interface enhancements and beta features for http://ckeditor-spellcheck.nanospell.com . This repo is for nanospell developers and trusted partners to modify and fork plugin.js
Other
0 stars 0 forks source link

Use store.js to deal with missing localstorage, if available #24

Closed caffodian closed 8 years ago

caffodian commented 8 years ago

It looks like store.js will handle the permissions problem for us. It has a similar snippet to what Modernizr does. Also update the non-storejs feature detection to do the modernizr technique of attempting to use a localStorage API from inside a try/catch.

caffodian commented 8 years ago

Argh. Incognito mode isn't a good way to disable localstorage for testing either. Continued search for how to actually trigger this security exception in IE...

caffodian commented 8 years ago

Fixes #21

caffodian commented 8 years ago

https://github.com/marcuswestin/store.js/

caffodian commented 8 years ago

I think https://github.com/marcuswestin/store.js/blob/master/src/store.js#L156 catches the security exception correctly.