Skip to content

Remove global storage #58

Merged
merged 4 commits into from Dec 19, 2012

3 participants

@rmg
rmg commented Dec 18, 2012

Pretty straight forward. Might want to bump to 1.4.x if you're following SemVer.. not sure if anyone would care.

@rmg
rmg commented Dec 18, 2012

Should close #44, I just forgot to mention it in a commit so GH would do it automagically..

@whitehat101

Without globalStorage, Firefox 2.0, 3.0 are no longer supported. The (un)supported browsers in the readme should be updated.

https://developer.mozilla.org/en-US/docs/DOM/Storage#Browser_compatibility

Someone might appreciate a branch or a cdn hosted version of the last globalStorage-supporting version. That said, none of my clients use these browsers.

@marcuswestin
Owner

Looks great!

Good call @whitehat101, I'll add a note about FF 2/3 in readme.

Will merge & bump/tag version. Thanks @rmg!

Cheers,
Marcus

@marcuswestin marcuswestin merged commit 0f1582e into marcuswestin:master Dec 19, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.