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

WordPress Trac wp-trac at lists.automattic.com
Sat Nov 12 19:33:44 UTC 2011


#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 azaozz):

 Replying to [comment:17 duck_]:

 Right. The problem is that the "unload" event
 [http://api.jquery.com/unload/ fires only on "window"]. What Thickbox did
 is hacky in the first place, firing "unload" on DIVs. That's why all the
 other scripts that use window.unload have problems when used together with
 Thickbox (btw this should have been a custom event).

 Internally this "unload" is used only for the #TB_window DIV, seems it was
 fired on the other DIVs because they all were selected at the time. Don't
 think any other script uses/expects "unload" on anything else than
 "window".

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


More information about the wp-trac mailing list