Skip to content

J
jquery
Commit ad8dc12a authored by Dave Methvin's avatar Dave Methvin
Browse files
Options

Fix #11757. IE 6/7 can't clean up circular DOM-JS refs. 

Why? Because it's dumb. Verified with the case in the ticket.
parent d5658e97
Hide whitespace changes
Inline Side-by-side
Showing with 3 additions and 1 deletion
src/support.js
View file @ ad8dc12a
...@@ -10,6 +10,7 @@ jQuery.support = (function() { ...@@ -10,6 +10,7 @@ jQuery.support = (function() {
eventName, eventName,
i, i,
isSupported, isSupported,
clickFn,
div = document.createElement("div"); div = document.createElement("div");
// Preliminary tests // Preliminary tests
...@@ -112,12 +113,13 @@ jQuery.support = (function() { ...@@ -112,12 +113,13 @@ jQuery.support = (function() {
} }
if ( !div.addEventListener && div.attachEvent && div.fireEvent ) { if ( !div.addEventListener && div.attachEvent && div.fireEvent ) {
div.attachEvent( "onclick", function() { div.attachEvent( "onclick", clickFn = function() {
// Cloning a node shouldn't copy over any // Cloning a node shouldn't copy over any
// bound event handlers (IE does this) // bound event handlers (IE does this)
support.noCloneEvent = false; support.noCloneEvent = false;
}); });
div.cloneNode( true ).fireEvent("onclick"); div.cloneNode( true ).fireEvent("onclick");
div.detachEvent( "onclick", clickFn );
} }
// Check if a radio maintains its value // Check if a radio maintains its value
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment