error in //Adjust for border radius code

Bug #528600 reported by patricia on 2010-02-26
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
jquery (Ubuntu)
Undecided
Unassigned
Nominated for Lucid by solaxso

Bug Description

In rev 25 on line 1098 I am getting :
"newPosition is undefined" in Firefox and
"undefined is null or not an object" in IE.

using jquery 1.4.2

when i comment out this code, it goes away, and the tool tips seem to be positioned fine.

    // Adjust for border radius
    if(self.options.style.border.radius > 0) {
     if(my.x === 'left') { newPosition.left -= self.options.style.border.radius; }
     else if(my.x === 'right') { newPosition.left += self.options.style.border.radius; }

     if(my.y === 'top') { newPosition.top -= self.options.style.border.radius; }
     else if(my.y === 'bottom') { newPosition.top += self.options.style.border.radius; }
    }

Hongli Lai (hongli) wrote :

I countered this bug too. It also occurs with jQuery 1.4.1. Here's a patch which fixes the problem; it appears that the author renamed 'newPosition' to 'position' and forgot to update that piece of code.

Hongli Lai (hongli) wrote :

Here's a test case.

Fixed in rev26 of the 1.x branch.

Changed in qtip:
assignee: nobody → Craig Thompson (craig.craigsworks)
importance: Undecided → Critical
milestone: none → 1.0
status: New → Fix Committed
solaxso (peter-lohs) on 2010-07-11
Changed in qtip:
status: Fix Committed → Fix Released
Philip Muškovac (yofel) wrote :

Thank you for taking the time to report this bug and helping to make Ubuntu better. This bug did not have a package associated with it, which is important for ensuring that it gets looked at by the proper developers. You can learn more about finding the right package at https://wiki.ubuntu.com/Bugs/FindRightPackage. I have classified this bug as a bug in jquery.

When reporting bugs in the future please use apport, either via the appropriate application's "Help -> Report a Problem" menu or using 'ubuntu-bug' and the name of the package affected. You can learn more about this functionality at https://wiki.ubuntu.com/ReportingBugs.

affects: ubuntu → jquery (Ubuntu)
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers

Bug attachments