Posted by btopro on July 28, 2009 at 2:44pm
| Project: | jTooltips |
| Version: | 6.x-1.x-dev |
| Component: | Code |
| Category: | bug report |
| Priority: | critical |
| Assigned: | Unassigned |
| Status: | needs review |
Issue Summary
http://docs.jquery.com/Release:jQuery_1.3#Changes
"The '@' in [@attr] has been removed. Deprecated since 1.2 this old syntax no longer works. Simply remove the @ to upgrade."
To make jtooltips / your Drupal site work correctly with jquery_update and this module activated, look for S(\'[@d]\') in the file jtooltips/js/jquery.tooltip.pack.js and take out the @ . The module will work as intended once you take out the @. This is critical if you're using jQuery 1.3.
Comments
#1
Thanks.
Since Drupal 6.x ships jQuery 1.2.x, and this syntax is already deprecated in 1.2.x, the change should really be made in the module.
I just tested using jQuery 1.2 and the changed file, and things still work.
Unfortunately, the latest version from bassistance.de does not seem to be a drop-in replacement. It didn't show my existing tooltips anymore.
Anyway, here's the edited file (without the '@') for who'd rather download instead of use an editor.
#2
module needs to be updated for jquery update
thanks for tip
#3
Is anyone planning to update this module? It seems like a simple fix, unless this modification causes problems in Jquery 1.2, or something. I just spent an hour trying to figure out why the tooltips weren't working, forgetting that I have to hack the JS file for Jquery 1.3. Thanks!