r727 - trunk/ui
r727 - trunk/ui
Author: scott.gonzalez
Date: Sat Sep 20 06:55:35 2008
New Revision: 727
Modified:
trunk/ui/ui.core.js
Log:
Core: Added comment about why we use triggerHandler instead of trigger in
$.fn.remove.
Modified: trunk/ui/ui.core.js
==============================================================================
--- trunk/ui/ui.core.js (original)
+++ trunk/ui/ui.core.js Sat Sep 20 06:55:35 2008
@@ -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");
});