[wp-trac] [WordPress Trac] #19189: thickbox.js .trigger("unload") causes conflicts

[WordPress Trac]

#19189: thickbox.js .trigger("unload") causes conflicts
----------------------------+---------------------
 Reporter:  ocean90         |       Owner:  azaozz
     Type:  defect (bug)    |      Status:  closed
 Priority:  normal          |   Milestone:  3.3
Component:  Administration  |     Version:  3.3
 Severity:  major           |  Resolution:  fixed
 Keywords:  has-patch       |
----------------------------+---------------------

Comment (by aesqe):

 Could this please be changed into a custom .trigger() event? Because it
 looks like .triggerHandler("unload") does nothing at all in 3.3-beta4 with
 jQuery 1.7.1.

 Also, .triggerHandler() doesn't work with .live(), so what's the point of
 having it there, anyway?

 These both worked in 3.2.1:
 {{{
 $("#TB_overlay, #TB_window").live("unload", function(e){
         console.debug(this.id + " live unload");
 });

 $(window).bind("unload", function(e){
         console.debug("window unload");
 });
 }}}

 Before 3.3-beta4 and jQuery 1.7.1,
 $("#TB_window").triggerHandler("unload") was registered with
 window.unload, now it's not :/

 Any thoughts?

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/19189#comment:20>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software