diff options
| author | Scott González <scott.gonzalez@gmail.com> | 2008-09-20 13:55:35 +0000 |
|---|---|---|
| committer | Scott González <scott.gonzalez@gmail.com> | 2008-09-20 13:55:35 +0000 |
| commit | 068d2483c973ff888d30d83b99f2b867a7907112 (patch) | |
| tree | bcdff612937d810db9d671e67560a85ca27f606e | |
| parent | 1df15fabb459b7b824ec9a7827dce4ffede78683 (diff) | |
| download | jquery-ui-068d2483c973ff888d30d83b99f2b867a7907112.tar.gz jquery-ui-068d2483c973ff888d30d83b99f2b867a7907112.zip | |
Core: Added comment about why we use triggerHandler instead of trigger in $.fn.remove.
| -rw-r--r-- | ui/ui.core.js | 3 |
1 files changed, 2 insertions, 1 deletions
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"); }); |