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

Wrong Hover/Highlight(Selection) on Charts #6687

Closed
dragoshzava opened this Issue May 5, 2017 · 15 comments

Comments

Projects
None yet
7 participants
@dragoshzava

dragoshzava commented May 5, 2017

Description

This happens when hovering over last serie, highcharts selects wrong points.

Expected behaviour

Highcharts should select an entire column with all points.

Actual behaviour

Highcharts selects points from different columns

Live demo with steps to reproduce

http://jsfiddle.net/Resems/m23omt3r/1/

Image attached, hovering last Column, lowest points, selects nearby Column, so the end result is awkward, selection is : 3 point from 7 jan, and other points from 8 jan.

image

@PedroSolintel

This comment has been minimized.

Show comment
Hide comment
@PedroSolintel

PedroSolintel May 5, 2017

Same error happens here, with an additional bug: not only the hovering goes wrong, but the highlighting too. See http://jsfiddle.net/c0fbL8vb/

How to reproduce the highlighting bug:

  1. Put the mouse over 'Jan' category.
  2. Jun's orange column will highlight.

How to reproduce the wrong hovering bug:

  1. Slowly move your mouse back and forth over the base of the columns.
  2. You'll notice strange behaviors on every category except for Dec.

PedroSolintel commented May 5, 2017

Same error happens here, with an additional bug: not only the hovering goes wrong, but the highlighting too. See http://jsfiddle.net/c0fbL8vb/

How to reproduce the highlighting bug:

  1. Put the mouse over 'Jan' category.
  2. Jun's orange column will highlight.

How to reproduce the wrong hovering bug:

  1. Slowly move your mouse back and forth over the base of the columns.
  2. You'll notice strange behaviors on every category except for Dec.

@dragoshzava dragoshzava changed the title from Wrong Hover on Stacked Chart to Wrong Hover and Selection on Charts May 5, 2017

@dragoshzava dragoshzava changed the title from Wrong Hover and Selection on Charts to Wrong Hover/Highlight(Selection) on Charts May 5, 2017

@pawelfus

This comment has been minimized.

Show comment
Hide comment
@pawelfus

pawelfus May 8, 2017

Contributor

Thank you for reporting this issue. It's a regression, in 5.0.10 examples work fine:

Regression from 13cfbe1 commit.

Contributor

pawelfus commented May 8, 2017

Thank you for reporting this issue. It's a regression, in 5.0.10 examples work fine:

Regression from 13cfbe1 commit.

@pawelfus pawelfus added the Regression label May 8, 2017

@dragoshzava

This comment has been minimized.

Show comment
Hide comment
@dragoshzava

dragoshzava May 8, 2017

Hey, unfortunately with 5.0.10 i could still repro
http://jsfiddle.net/m23omt3r/3/
image
in this attached picture, the hover is over living room column, but the info is displayed from kitchen.
you can repro by moving the mouse from under the living room column.

But with 5.0.9 works, attached fiddle.
http://jsfiddle.net/m23omt3r/4/

dragoshzava commented May 8, 2017

Hey, unfortunately with 5.0.10 i could still repro
http://jsfiddle.net/m23omt3r/3/
image
in this attached picture, the hover is over living room column, but the info is displayed from kitchen.
you can repro by moving the mouse from under the living room column.

But with 5.0.9 works, attached fiddle.
http://jsfiddle.net/m23omt3r/4/

@PedroSolintel

This comment has been minimized.

Show comment
Hide comment
@PedroSolintel

PedroSolintel May 8, 2017

At first I thought that the bug persisted, because I regressed my highcharts.js to v5.0.10 and was still facing the same issues. It turned out that I was supposed to update every module as well (modules were a little bit outdated). Changed everything to version 5.0.10 and then everything started to work fine.

So for other people who might have the same problem: you should upgrade (or downgrade) every module as well, otherwise the bug will remain there.

Thanks for the help!

PedroSolintel commented May 8, 2017

At first I thought that the bug persisted, because I regressed my highcharts.js to v5.0.10 and was still facing the same issues. It turned out that I was supposed to update every module as well (modules were a little bit outdated). Changed everything to version 5.0.10 and then everything started to work fine.

So for other people who might have the same problem: you should upgrade (or downgrade) every module as well, otherwise the bug will remain there.

Thanks for the help!

@TorsteinHonsi

This comment has been minimized.

Show comment
Hide comment
@TorsteinHonsi

TorsteinHonsi May 10, 2017

Collaborator

@jon-a-nygaard This needs priority.

Collaborator

TorsteinHonsi commented May 10, 2017

@jon-a-nygaard This needs priority.

@jon-a-nygaard

This comment has been minimized.

Show comment
Hide comment
@jon-a-nygaard

jon-a-nygaard May 10, 2017

Collaborator

@TorsteinHonsi Yes, currently working on it. Not yet found the cause of problem.

Collaborator

jon-a-nygaard commented May 10, 2017

@TorsteinHonsi Yes, currently working on it. Not yet found the cause of problem.

@jon-a-nygaard

This comment has been minimized.

Show comment
Hide comment
@jon-a-nygaard

jon-a-nygaard May 10, 2017

Collaborator

Internal Note: Simplified demo

Collaborator

jon-a-nygaard commented May 10, 2017

Internal Note: Simplified demo

@jon-a-nygaard

This comment has been minimized.

Show comment
Hide comment
@jon-a-nygaard

jon-a-nygaard May 11, 2017

Collaborator

Potential workaround
Here's an example of a potential workaround. Unfortunately it is not very compact, so I will see if there an better alternative.

Update
Slightly improved workaround

Collaborator

jon-a-nygaard commented May 11, 2017

Potential workaround
Here's an example of a potential workaround. Unfortunately it is not very compact, so I will see if there an better alternative.

Update
Slightly improved workaround

@MiroLiska

This comment has been minimized.

Show comment
Hide comment
@MiroLiska

MiroLiska May 22, 2017

Something is completely wrong now. I must hover with the mouse directly over the data point. Before, I could hover with the mouse anywhere over the data point, the tooltip would come. See
http://jsfiddle.net/cjsnxj4x/2/
wrongtooltip

versus http://jsfiddle.net/cjsnxj4x/3/ (version 4.2.3)

MiroLiska commented May 22, 2017

Something is completely wrong now. I must hover with the mouse directly over the data point. Before, I could hover with the mouse anywhere over the data point, the tooltip would come. See
http://jsfiddle.net/cjsnxj4x/2/
wrongtooltip

versus http://jsfiddle.net/cjsnxj4x/3/ (version 4.2.3)

@MiroLiska

This comment has been minimized.

Show comment
Hide comment
@MiroLiska

MiroLiska May 22, 2017

Another bug found - with scatter charts using shared tooltips. See http://jsfiddle.net/773sz6Lv/1/ - move the mouse over the blue part of the point 1269 - the shared tooltip from market value does not appear.
ig1

And now move the mouse up - the shared tooltip will eventually appear:
ig2

MiroLiska commented May 22, 2017

Another bug found - with scatter charts using shared tooltips. See http://jsfiddle.net/773sz6Lv/1/ - move the mouse over the blue part of the point 1269 - the shared tooltip from market value does not appear.
ig1

And now move the mouse up - the shared tooltip will eventually appear:
ig2

@MiroLiska

This comment has been minimized.

Show comment
Hide comment
@MiroLiska

MiroLiska May 24, 2017

Just tested the issue above with freshly released 5.0.12 - it still persists :(
http://jsfiddle.net/773sz6Lv/2/

MiroLiska commented May 24, 2017

Just tested the issue above with freshly released 5.0.12 - it still persists :(
http://jsfiddle.net/773sz6Lv/2/

@sebastianbochan

This comment has been minimized.

Show comment
Hide comment
@sebastianbochan

sebastianbochan May 24, 2017

Contributor

@jon-a-nygaard Could you verify it?

Contributor

sebastianbochan commented May 24, 2017

@jon-a-nygaard Could you verify it?

@jon-a-nygaard

This comment has been minimized.

Show comment
Hide comment
@jon-a-nygaard

jon-a-nygaard May 26, 2017

Collaborator

Not certain what excactly is causing that issue, currently looking into it.

Collaborator

jon-a-nygaard commented May 26, 2017

Not certain what excactly is causing that issue, currently looking into it.

@jon-a-nygaard

This comment has been minimized.

Show comment
Hide comment
@jon-a-nygaard

jon-a-nygaard May 26, 2017

Collaborator

The last issue is a similar but a different one from the original one in this topic. I will likely create a new issue to seperate these.

Explanation

Caused by some series having the closest point to the mouse pointer on a different x value than the hovered point. Resulting in a filter removing these points, and too few points are passed to the tooltip.

Potential Workaround

http://jsfiddle.net/jon_a_nygaard/773sz6Lv/3/

Collaborator

jon-a-nygaard commented May 26, 2017

The last issue is a similar but a different one from the original one in this topic. I will likely create a new issue to seperate these.

Explanation

Caused by some series having the closest point to the mouse pointer on a different x value than the hovered point. Resulting in a filter removing these points, and too few points are passed to the tooltip.

Potential Workaround

http://jsfiddle.net/jon_a_nygaard/773sz6Lv/3/

@MiroLiska

This comment has been minimized.

Show comment
Hide comment
@MiroLiska

MiroLiska May 29, 2017

Hi. The issue is not a show-stopper, take your time.

MiroLiska commented May 29, 2017

Hi. The issue is not a show-stopper, take your time.

jon-a-nygaard added a commit that referenced this issue Jun 14, 2017

Fixed #6764, fixed #6791, fixed #6808, addition #6687. Series with no…
…SharedTooltip was unaccessible when tooltip is shared.

TorsteinHonsi added a commit that referenced this issue Jun 15, 2017

Fixed #6764, fixed #6791, fixed #6808, addition #6687. Series with no…
…SharedTooltip was unaccessible when tooltip is shared.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment