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

No pointer cursor on drill down line charts #6695

Closed
kzoon opened this Issue May 8, 2017 · 7 comments

Comments

Projects
None yet
2 participants
@kzoon

kzoon commented May 8, 2017

Expected behaviour

The cursor should change into a 'hand' when you hover over a data point/series that can drill down. In line charts that doesn't happen (in column charts, pie charts, etc. it does)

Actual behaviour

The cursor doesn't change when hovering a drill down line

Live demo with steps to reproduce

http://jsfiddle.net/kzoon/nnyeho5j/3/

Affected browser(s)

@pawelfus pawelfus added the Bug label May 8, 2017

@pawelfus

This comment has been minimized.

Show comment
Hide comment
@pawelfus

pawelfus May 8, 2017

Contributor

Thank you for the report.

Workaround:
Set styles manually:

.highcharts-point-hover, .highcharts-halo {
  cursor: pointer;
}
Contributor

pawelfus commented May 8, 2017

Thank you for the report.

Workaround:
Set styles manually:

.highcharts-point-hover, .highcharts-halo {
  cursor: pointer;
}
@kzoon

This comment has been minimized.

Show comment
Hide comment
@kzoon

kzoon May 9, 2017

Bubble chart has comparable issue: See http://jsfiddle.net/6oq0pr6L/: it does drill down, but the cursor is not pointer.

(this fiddle uses 5.0.10 explicitly, because of a regression in the latest version, see https://forum.highcharts.com/post134961.html#p134961)

kzoon commented May 9, 2017

Bubble chart has comparable issue: See http://jsfiddle.net/6oq0pr6L/: it does drill down, but the cursor is not pointer.

(this fiddle uses 5.0.10 explicitly, because of a regression in the latest version, see https://forum.highcharts.com/post134961.html#p134961)

@kzoon

This comment has been minimized.

Show comment
Hide comment
@kzoon

kzoon May 19, 2017

Treemaps with drill down also have no pointer cursor.
See http://jsfiddle.net/kzoon/ksb9k1yd/ (copied from pie drill down http://jsfiddle.net/gh/get/library/pure/highcharts/highcharts/tree/master/samples/highcharts/demo/pie-drilldown/)

Notice that this concerns regular drill down on tree map, not allowDrillToNode. http://jsfiddle.net/kzoon/ksb9k1yd/ drills down exactly how it should. Only the cursor is wrong.

kzoon commented May 19, 2017

Treemaps with drill down also have no pointer cursor.
See http://jsfiddle.net/kzoon/ksb9k1yd/ (copied from pie drill down http://jsfiddle.net/gh/get/library/pure/highcharts/highcharts/tree/master/samples/highcharts/demo/pie-drilldown/)

Notice that this concerns regular drill down on tree map, not allowDrillToNode. http://jsfiddle.net/kzoon/ksb9k1yd/ drills down exactly how it should. Only the cursor is wrong.

@kzoon

This comment has been minimized.

Show comment
Hide comment
@kzoon

kzoon Jun 6, 2017

I plan to release a new version of my software in 2 weeks time. This version includes drill down. Any chance that this bug is fixed by then?
To recap: drill down works fine on line charts, bubble charts and treemaps, but the cursor doesn't indicate that the user can drill down (by showing a 'hand').

kzoon commented Jun 6, 2017

I plan to release a new version of my software in 2 weeks time. This version includes drill down. Any chance that this bug is fixed by then?
To recap: drill down works fine on line charts, bubble charts and treemaps, but the cursor doesn't indicate that the user can drill down (by showing a 'hand').

@pawelfus

This comment has been minimized.

Show comment
Hide comment
@pawelfus

pawelfus Jun 7, 2017

Contributor

Version 5.0.12 was released about two weeks ago. We don't have ETA for the next release so it's unlikely to be released within two weeks.

Contributor

pawelfus commented Jun 7, 2017

Version 5.0.12 was released about two weeks ago. We don't have ETA for the next release so it's unlikely to be released within two weeks.

@kzoon

This comment has been minimized.

Show comment
Hide comment
@kzoon

kzoon Jun 7, 2017

I am happy to integrate fixes from github into my release

kzoon commented Jun 7, 2017

I am happy to integrate fixes from github into my release

@kzoon

This comment has been minimized.

Show comment
Hide comment
@kzoon

kzoon Jun 14, 2017

It looks like the fix solves the issue for bubble (http://jsfiddle.net/kzoon/6oq0pr6L/6/) and tree map (http://jsfiddle.net/kzoon/ksb9k1yd/2/),
but not (completely) for line charts. The line between points does drill down, but doesn't have a 'hand' cursor:
http://jsfiddle.net/kzoon/nnyeho5j/8/

kzoon commented Jun 14, 2017

It looks like the fix solves the issue for bubble (http://jsfiddle.net/kzoon/6oq0pr6L/6/) and tree map (http://jsfiddle.net/kzoon/ksb9k1yd/2/),
but not (completely) for line charts. The line between points does drill down, but doesn't have a 'hand' cursor:
http://jsfiddle.net/kzoon/nnyeho5j/8/

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