From f42d42b61fd00269d9fdefde83ed69f0f8bd7489 Mon Sep 17 00:00:00 2001 From: Eduardo Lundgren Date: Fri, 27 Jun 2008 21:36:14 +0000 Subject: [PATCH] Avoid IE memory leak on enableSelection, disableSelection methods. --- ui/ui.core.js | 12 ++++-------- 1 file changed, 4 insertions(+), 8 deletions(-) diff --git a/ui/ui.core.js b/ui/ui.core.js index ff8dec8b4..88596c446 100644 --- a/ui/ui.core.js +++ b/ui/ui.core.js @@ -46,15 +46,11 @@ $.ui = { try { $('body').get(0).removeChild(tmp.get(0)); } catch(e){} return $.ui.cssCache[name]; }, - disableSelection: function(e) { - e.unselectable = "on"; - e.onselectstart = function() { return false; }; - if (e.style) { e.style.MozUserSelect = "none"; } + disableSelection: function(el) { + $(el).attr('unselectable', 'on').css('MozUserSelect', 'none'); }, - enableSelection: function(e) { - e.unselectable = "off"; - e.onselectstart = function() { return true; }; - if (e.style) { e.style.MozUserSelect = ""; } + enableSelection: function(el) { + $(el).attr('unselectable', 'off').css('MozUserSelect', ''); }, hasScroll: function(e, a) { var scroll = /top/.test(a||"top") ? 'scrollTop' : 'scrollLeft', has = false;