This is a crossposting with (http://groups.google.com/group/jquery-plugins), however to inform the contributors and users of jquery_update of this bug seems to me important enough to allow duplication of this report.

I had problems using the Thickbox 3.1 in Drupal5 which were due to
compat-1.0.js.

Since it is listed in var e: (see:
http://dev.jquery.com/browser/trunk/plugins/compat-1.0/jquery.compat-1.0...)
the event type 'unload' is being replaced by unbind("load",fn) which
makes IMHO not much sense. The event type 'unload' is valid and
should not be replaced for backwards compatibility! Thus I suggest to
change var e by deleting the load event from the string string like:

 var e = ("blur,focus, resize,scroll,unload,click,dblclick," +

"mousedown,mouseup,mousemove,mouseover,mouseout,change,reset,select,"
+
           "submit,keydown,keypress,keyup,error").split(",");

After this step my problems with thickbox were solved!

A patch even if the modification is quite simple is attached to this posting.

cheers
Andreas

CommentFileSizeAuthor
compat-1.0_fix-unload.patch845 bytesAndreas Kohlbecker
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

heddn’s picture

Issue summary: View changes
Status: Needs review » Closed (won't fix)

Marking this as won't fixed. 5.x is no longer supported/maintained. If this is still an issue in 7.x, then please re-open.