-
Notifications
You must be signed in to change notification settings - Fork 3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
dom storage #19
Comments
this would be a revision of : http://github.com/Modernizr/Modernizr/issues/closed#issue/3 |
fwiw pilgrim refers to this as "HTML5 storage": http://diveintohtml5.org/detect.html the spec calls it Web Storage: http://dev.w3.org/html5/webstorage/ I think they should be tested together (as above) and available as |
I agree. It's called Web Storage because it encompasses multiple forms of storage, most notably localStorage and sessionStorage. We should test against both and report on both individually, e.g.: Modernizr.localstorage (note the consistent-to-Modernizr use of all-lowercase attributes) |
definitely keep them separate. chromium currently only implements localstorage: |
sessionStorage and 'x' in window fix for cookies-disabled. closed by 1e0ba91 |
http://github.com/fearphage/Modernizr/commit/fe68e0cb88e05067daeb4163bc263d37a11ad965
tests[domstorage] = function() {
// tested sessionsStorage this way because otherwise Firefox will throw
return !!(window.localStorage && ('sessionStorage' in window));
};
The text was updated successfully, but these errors were encountered: