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

Marker on hover with shared tooltip appears inconsistently #5766

Closed
morganfree opened this Issue Oct 3, 2016 · 3 comments

Comments

Projects
None yet
3 participants
@morganfree
Contributor

morganfree commented Oct 3, 2016

Expected behaviour

Hovering over point causes showing the marker - even if the data is dense.
http://jsfiddle.net/2tjq74te/6/

Actual behaviour

With shared tooltip the marker is not always displayed over hovered point.

Live demo with steps to reproduce

http://jsfiddle.net/2tjq74te/7/

@pawelfus pawelfus self-assigned this Oct 3, 2016

@pawelfus pawelfus added the Regression label Oct 3, 2016

@pawelfus

This comment has been minimized.

Show comment
Hide comment
@pawelfus

pawelfus Oct 3, 2016

Contributor

Internal note:
Fails since 27f81b8

Contributor

pawelfus commented Oct 3, 2016

Internal note:
Fails since 27f81b8

@idrm

This comment has been minimized.

Show comment
Hide comment
@idrm

idrm Oct 13, 2016

The affected marker is always the one that would normally capture the hover state were the tooltip not shared (tested in version 5.0.0).

The other trigger for this behavior appears to be a datetime x-axis. The markers appear as they should when, for example, the x-axis uses categories instead.

idrm commented Oct 13, 2016

The affected marker is always the one that would normally capture the hover state were the tooltip not shared (tested in version 5.0.0).

The other trigger for this behavior appears to be a datetime x-axis. The markers appear as they should when, for example, the x-axis uses categories instead.

pawelfus added a commit that referenced this issue Oct 13, 2016

@pawelfus

This comment has been minimized.

Show comment
Hide comment
@pawelfus

pawelfus Oct 13, 2016

Contributor

Hi @idrm

Thank you for your feedback. Actually, datetime axis was not important, but number of points: http://jsfiddle.net/dk6x2phu/

@TorsteinHonsi - could you review the commit above before releasing 5.0.1? It also cleans up runPointActions method..

Contributor

pawelfus commented Oct 13, 2016

Hi @idrm

Thank you for your feedback. Actually, datetime axis was not important, but number of points: http://jsfiddle.net/dk6x2phu/

@TorsteinHonsi - could you review the commit above before releasing 5.0.1? It also cleans up runPointActions method..

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