From 068d2483c973ff888d30d83b99f2b867a7907112 Mon Sep 17 00:00:00 2001 From: Scott González Date: Sat, 20 Sep 2008 13:55:35 +0000 Subject: Core: Added comment about why we use triggerHandler instead of trigger in $.fn.remove. --- ui/ui.core.js | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/ui/ui.core.js b/ui/ui.core.js index 50140d73e..413d5ee1b 100644 --- a/ui/ui.core.js +++ b/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"); }); -- cgit v1.2.3