From dae257b04f0533bcc737a86c598d4f61cdca7bb2 Mon Sep 17 00:00:00 2001 From: David Greenspan Date: Wed, 12 Feb 2014 14:06:47 -0800 Subject: [PATCH] Kill malfunctioning shim for localStorage in IE 7 jQuery removed $.browser, so this code threw an exception any time localStorage wasn't available. Moreover, IE 8 supports localStorage (as far as I'm aware), and we don't care about having something fancy just for IE 7. --- packages/localstorage/localstorage.js | 30 --------------------------- 1 file changed, 30 deletions(-) diff --git a/packages/localstorage/localstorage.js b/packages/localstorage/localstorage.js index ccaefcca08..fe6ff10f17 100644 --- a/packages/localstorage/localstorage.js +++ b/packages/localstorage/localstorage.js @@ -29,36 +29,6 @@ if (window.localStorage) { } } -// XXX eliminate dependency on jQuery, detect browsers ourselves -// Else, if we are on IE, which support userData -if (!Meteor._localStorage && $.browser.msie) { - var userdata = document.createElement('span'); // could be anything - userdata.style.behavior = 'url("#default#userData")'; - userdata.id = 'localstorage-helper'; - userdata.style.display = 'none'; - document.getElementsByTagName("head")[0].appendChild(userdata); - - var userdataKey = 'localStorage'; - userdata.load(userdataKey); - - Meteor._localStorage = { - setItem: function (key, val) { - userdata.setAttribute(key, val); - userdata.save(userdataKey); - }, - - removeItem: function (key) { - userdata.removeAttribute(key); - userdata.save(userdataKey); - }, - - getItem: function (key) { - userdata.load(userdataKey); - return userdata.getAttribute(key); - } - }; -} - if (!Meteor._localStorage) { Meteor._debug( "You are running a browser with no localStorage or userData "