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

Halo visible outside plotting area #3916

Closed
pawelfus opened this Issue Mar 2, 2015 · 12 comments

Comments

Projects
None yet
6 participants
@pawelfus
Contributor

pawelfus commented Mar 2, 2015

Example: http://jsfiddle.net/yaot3ebq/

Steps:

  • zoom-in, that some of the points are outside the plotting area
  • hover one of visible points
  • move mouse to hover points outside the plotting area: tooltip and halo are displayed

helo

@TorsteinHonsi

This comment has been minimized.

Show comment
Hide comment
@TorsteinHonsi

TorsteinHonsi Mar 2, 2015

Collaborator

Simplified non-Highstock demo: http://jsfiddle.net/highcharts/9vxtf5nq/. Hover on left side of plot area, tooltip displays for the first point.

It also happened in Highcharts 4.0: http://jsfiddle.net/highcharts/9vxtf5nq/1/

Collaborator

TorsteinHonsi commented Mar 2, 2015

Simplified non-Highstock demo: http://jsfiddle.net/highcharts/9vxtf5nq/. Hover on left side of plot area, tooltip displays for the first point.

It also happened in Highcharts 4.0: http://jsfiddle.net/highcharts/9vxtf5nq/1/

@pawelfus

This comment has been minimized.

Show comment
Hide comment
@pawelfus

pawelfus Aug 17, 2015

Contributor

Another case, it doesn't have to be outside the plotting area, just misplaced: http://jsfiddle.net/n6buh1o6/3/

Steps:

  • hover one of the points
  • zoom in
  • we get tooltip for a proper point, but halo is misplaced
Contributor

pawelfus commented Aug 17, 2015

Another case, it doesn't have to be outside the plotting area, just misplaced: http://jsfiddle.net/n6buh1o6/3/

Steps:

  • hover one of the points
  • zoom in
  • we get tooltip for a proper point, but halo is misplaced
@sebastianbochan

This comment has been minimized.

Show comment
Hide comment
@sebastianbochan

sebastianbochan Sep 8, 2015

Contributor

Seems that problem also exists in bubble chart.

Steps:

  1. http://www.highcharts.com/demo/bubble
  2. hover on gray point
  3. zoom in
  4. halo is misplaced

https://drive.google.com/file/d/0ByEiCW-yZo19S1ZDOGlVWjllSkU/view

Browser: Chrome 45.0.2454.85

Contributor

sebastianbochan commented Sep 8, 2015

Seems that problem also exists in bubble chart.

Steps:

  1. http://www.highcharts.com/demo/bubble
  2. hover on gray point
  3. zoom in
  4. halo is misplaced

https://drive.google.com/file/d/0ByEiCW-yZo19S1ZDOGlVWjllSkU/view

Browser: Chrome 45.0.2454.85

@mekhatria

This comment has been minimized.

Show comment
Hide comment
@sebastianbochan

This comment has been minimized.

Show comment
Hide comment
@sebastianbochan

sebastianbochan Apr 5, 2016

Contributor

+1

Contributor

sebastianbochan commented Apr 5, 2016

+1

@scottpdawson

This comment has been minimized.

Show comment
Hide comment
@scottpdawson

scottpdawson Apr 5, 2016

I have this problem, too. I traced the change in the highCharts library to code that was introduced in 4.1.6 (2015-06-12) of highcharts-more.js. I did trace a change to line 9575 of this file that, when reverted to 4.1.5, results in the issue going away.

// 4.1.5 CODE - tooltips only appear for visible points //

if (p && defined(p.plotX) && defined(p.plotY)) {
    if ((p.dist.distX < distance) || ((p.dist.distX === distance || p.series.kdDimensions > 1) && p.dist.distR < rdistance)) {
        distance = p.dist.distX;
        rdistance = p.dist.distR;
    }
}

// 4.1.6 CODE - tooltip outside of visible range //

if (p && typeof p.dist === 'number' && p.dist < distance) {
    distance = p.dist;
    kdpoint = p;
}

scottpdawson commented Apr 5, 2016

I have this problem, too. I traced the change in the highCharts library to code that was introduced in 4.1.6 (2015-06-12) of highcharts-more.js. I did trace a change to line 9575 of this file that, when reverted to 4.1.5, results in the issue going away.

// 4.1.5 CODE - tooltips only appear for visible points //

if (p && defined(p.plotX) && defined(p.plotY)) {
    if ((p.dist.distX < distance) || ((p.dist.distX === distance || p.series.kdDimensions > 1) && p.dist.distR < rdistance)) {
        distance = p.dist.distX;
        rdistance = p.dist.distR;
    }
}

// 4.1.6 CODE - tooltip outside of visible range //

if (p && typeof p.dist === 'number' && p.dist < distance) {
    distance = p.dist;
    kdpoint = p;
}

@TorsteinHonsi

This comment has been minimized.

Show comment
Hide comment
@TorsteinHonsi

TorsteinHonsi Apr 6, 2016

Collaborator

It's like #5085 (fixed in dev branch), except this is on the Y axis.

Collaborator

TorsteinHonsi commented Apr 6, 2016

It's like #5085 (fixed in dev branch), except this is on the Y axis.

@scottpdawson

This comment has been minimized.

Show comment
Hide comment
@scottpdawson

scottpdawson Apr 21, 2016

Do you know when this fix will be released?

scottpdawson commented Apr 21, 2016

Do you know when this fix will be released?

@TorsteinHonsi

This comment has been minimized.

Show comment
Hide comment
@TorsteinHonsi

TorsteinHonsi Apr 22, 2016

Collaborator

It was fixed with version 4.2.4, see http://jsfiddle.net/highcharts/9vxtf5nq/

Collaborator

TorsteinHonsi commented Apr 22, 2016

It was fixed with version 4.2.4, see http://jsfiddle.net/highcharts/9vxtf5nq/

@dotnetjalps

This comment has been minimized.

Show comment
Hide comment
@dotnetjalps

dotnetjalps Oct 28, 2016

This is still happening in highcharts 5 also. When you zoom to x its going outside.

bug.

You should reopen this issue. I am using highchart 5.0 version.

dotnetjalps commented Oct 28, 2016

This is still happening in highcharts 5 also. When you zoom to x its going outside.

bug.

You should reopen this issue. I am using highchart 5.0 version.

@TorsteinHonsi

This comment has been minimized.

Show comment
Hide comment
@TorsteinHonsi

TorsteinHonsi Oct 28, 2016

Collaborator

Live demo please? With our latest demo it doen't happen.

Collaborator

TorsteinHonsi commented Oct 28, 2016

Live demo please? With our latest demo it doen't happen.

@dotnetjalps

This comment has been minimized.

Show comment
Hide comment
@dotnetjalps

dotnetjalps Oct 28, 2016

I will create a JSfiddle for that.

dotnetjalps commented Oct 28, 2016

I will create a JSfiddle for that.

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