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

Inverted polar chart points tracking issue #5237

Closed
Izothep opened this Issue Apr 22, 2016 · 4 comments

Comments

Projects
None yet
3 participants
@Izothep

Izothep commented Apr 22, 2016

Hi,

It looks like when using inverted polar chart, wrong data points are hovered when mouse is over the plotArea. I think it may be the problem with tracking points.

Here you can find live example of this issue:
http://jsfiddle.net/vp95hjcc/3/

To reproduce problem hover on points - for example near '270' label.

Here I have made gif showing this problem:

animation

Kind regards.

@TorsteinHonsi

This comment has been minimized.

Show comment
Hide comment
@TorsteinHonsi

TorsteinHonsi Apr 25, 2016

Collaborator

I'm not sure I have the fantasy to imagine what an inverted polar chart should look like :)

In cartesian charts, inverted means that the X axis and the Y axis swap places. That doesn't happen in this polar chart. A consequence of inverting a cartesian chart is that the X axis is reversed by default, and this does happen in the polar chart.

Also, it seems that the tooltip behaves as expected if we don't set chart.inverted, but set xAxis.reversed instead: http://jsfiddle.net/highcharts/vp95hjcc/4/. Does this solve your problem?

In that case, the bug here is that a) the docs should state clearly that inverted doesn't apply to polar charts, and b) setting inverted should have no consequences on the axis.

Collaborator

TorsteinHonsi commented Apr 25, 2016

I'm not sure I have the fantasy to imagine what an inverted polar chart should look like :)

In cartesian charts, inverted means that the X axis and the Y axis swap places. That doesn't happen in this polar chart. A consequence of inverting a cartesian chart is that the X axis is reversed by default, and this does happen in the polar chart.

Also, it seems that the tooltip behaves as expected if we don't set chart.inverted, but set xAxis.reversed instead: http://jsfiddle.net/highcharts/vp95hjcc/4/. Does this solve your problem?

In that case, the bug here is that a) the docs should state clearly that inverted doesn't apply to polar charts, and b) setting inverted should have no consequences on the axis.

@Izothep

This comment has been minimized.

Show comment
Hide comment
@Izothep

Izothep Apr 26, 2016

Hi,

Thank you for your help, I had the same problem with imagine how this chart should look like. :)
It looks like setting xAxis.reversed solve this small issue.

I also agree that the main bug here is that inverted should have no consequences on the axis and it should be written in docs.

Have a nice day!

Izothep commented Apr 26, 2016

Hi,

Thank you for your help, I had the same problem with imagine how this chart should look like. :)
It looks like setting xAxis.reversed solve this small issue.

I also agree that the main bug here is that inverted should have no consequences on the axis and it should be written in docs.

Have a nice day!

@toshkent82

This comment has been minimized.

Show comment
Hide comment
@toshkent82

toshkent82 Apr 26, 2016

Thank you guys. It was originally my problem. Now, I like Highcharts even more with such support :)
I just add pane.startAngle:90 option to your advice and the polar diagram looks like I really need.

Good luck!

toshkent82 commented Apr 26, 2016

Thank you guys. It was originally my problem. Now, I like Highcharts even more with such support :)
I just add pane.startAngle:90 option to your advice and the polar diagram looks like I really need.

Good luck!

@TorsteinHonsi TorsteinHonsi added Bug Docs and removed Undecided labels Apr 27, 2016

@TorsteinHonsi

This comment has been minimized.

Show comment
Hide comment
@TorsteinHonsi
Collaborator

TorsteinHonsi commented Apr 27, 2016

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