Skip to content
Permalink
Browse files

Core: Added comment about why we use triggerHandler instead of trigge…

…r in $.fn.remove.
  • Loading branch information...
scottgonzalez committed Sep 20, 2008
1 parent 1df15fa commit 068d2483c973ff888d30d83b99f2b867a7907112
Showing with 2 additions and 1 deletion.
  1. +2 −1 ui/ui.core.js
@@ -13,7 +13,8 @@

var _remove = $.fn.remove;
$.fn.remove = function() {
// TODO: add comment about why we can't use .trigger()
// Safari has a native remove event which actually removes DOM elements,
// so we have to use triggerHandler instead of trigger (#3037).
$("*", this).add(this).each(function() {
$(this).triggerHandler("remove");
});

0 comments on commit 068d248

Please sign in to comment.
You can’t perform that action at this time.