Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tooltip positions are wrong around north in a polar chart #1469

Closed
TorsteinHonsi opened this issue Jan 24, 2013 · 6 comments
Closed

Tooltip positions are wrong around north in a polar chart #1469

TorsteinHonsi opened this issue Jan 24, 2013 · 6 comments

Comments

@TorsteinHonsi
Copy link
Collaborator

  1. Open http://jsfiddle.net/ptd5e/1/
  2. Hover over the line series, then move the mouse from the north to the west point.
  3. The tooltip shifts directly to the left of the north point, while it is expected to shift midway between them, at northwest position.
@TorsteinHonsi
Copy link
Collaborator Author

It's because the line is wrapped around the perimiter of the chart as a consequence of the connectEnds option. In the setTooltipPositions method, the first point should be considered. In order to fix it, the Series.connectEnds property, that is currently set in the extended getSegmentPath method, must be set earlier. Then this property must be checked in the setTooltipPoints method, and the first point must be appended to the points array so that it is taken into account when X positions are computed.

@markmsmith
Copy link

Just adding a +1 in favor of fixing this, since I also encountered this bug today.

@hofo
Copy link

hofo commented May 2, 2013

+1, just ran into this as well.

@markmsmith
Copy link

This is awesome, thanks for fixing it!
Do you know when the next stable release is going to be cut? We'd rather not run on bleeding-edge master.

@TorsteinHonsi
Copy link
Collaborator Author

It will be out in a few weeks. To avoid running the master, you can also
consider applying the patch for this one bug fix. Then you can compile the
source code at closure-compiler.appspot.com/home.

@markmsmith
Copy link

Thanks, I appreciate it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants