Closed (fixed)
Project:
jQuery Update
Version:
5.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Anonymous (not verified)
Created:
21 May 2007 at 08:20 UTC
Updated:
23 Jul 2018 at 22:18 UTC
Jump to comment: Most recent
Comments
Comment #1
jjeff commentedThis is the Thickbox module for Drupal? If so, it's probably using an old version of the Thickbox scripts in order to be compatible with Drupal-5's JQuery 1.0.1.
My guess is that you'll want to make sure that you're using the latest Thickbox scripts. My quick Google search got me here: http://jquery.com/demo/thickbox/
You *might* be able to simply download the new thickbox.js and replace the one included with module.
Good luck!
Comment #2
Anonymous (not verified) commentedYeah, I am using the latest version of thickbox. I also tried using some of the previous versions we have installed on other sites, no dice. It looks like the issue is with the compatibility script, I just don't know exactly how to debug it. No errors are thrown in Firebug and I can't exactly trace the execution of the code.
I'm gonna keep investigating, but would love to hear if anyone else is having similar issues.
M
Comment #3
Anonymous (not verified) commentedYeah, after spending some time with this issue some things are starting to make a little sense.
First off, using the following to test:
Under this configuration, Thickbox 3.0 dies. Turning off jquery_update brings it back. Have tried the same with a version of Thickbox 2.0 and problems are identical. The issue appears to be in compat-1.0.js, I have put up an empty version of that file and Thickbox will work. I am going to float the issue on the juery lists and see what people have to say and take a look at exactly what compat is doing. My guess is selectors behave differently between 1.0.1 and 1.1.2 and something is not being addressed, by limitation or design of the compatibility script.
M
Comment #4
Anonymous (not verified) commentedFound the problem, jquery 1.1.2 with the compat-1.0.js patch does not like the show() function at all. Thickbox 3.0 makes a call to it around line 53, which causes TB to die without error. Going to look at the compat patch when I get time and try making a patch there.
M
Comment #5
jjeff commentedWow! Nice. I know how hard it can be to track down JS bugs.
Hmmm... I wonder if there's a way to coordinate this with the Thickbox developer and/or John Resig if the problem is in the compat-1.0.js file...
There's a very active JQuery mailing list that might be a good place to coordinate some help.
Good luck!
Comment #6
Duplika commentedSuscribing... I'd love to see Thickbox working with jQuery update installed.
Comment #7
TBarregren commentedHave anyone found a solution yet?
Comment #8
guardian commentedsubscribing
patched jquery_update to 1.1.3 from http://drupal.org/node/168875 but it does not work well :(
Comment #9
inforeto commentedsubscribing
Comment #10
wim leersSince we've upgraded to jQuery 1.2.3 (http://drupal.org/node/156221), I'm assuming this is fixed. If not, feel free to reopen!
Comment #11
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.