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

Crosshairs overlapping #5424

Closed
PaulDalek opened this Issue Jun 22, 2016 · 3 comments

Comments

Projects
None yet
3 participants
@PaulDalek
Contributor

PaulDalek commented Jun 22, 2016

Hi,

Behaviour:

It seems that in line chart, in Highstock the number of yAxis affects the number of times crosshair is drawn, while in Highcharts it doesn’t matter how many yAxis there are, the crosshair is drawn only once. Beside of this, 'multiplied crosshairs' in first case overlaps each other, giving the impression that crosshair’s color is darker, please take a look at this example: http://jsfiddle.net/d_paul/bfL2nzk9/2/. Although it occurs only when crosshair’s width is set to 1. Changing this to higher value makes ‘crosshairs’ don’t overlap and crosshair’s color is displayed correctly.

Best regards.

@pawelfus

This comment has been minimized.

Show comment
Hide comment
@pawelfus

pawelfus Jun 23, 2016

Contributor

Thank you for reporting. You are right, issue is visible only in Highstock, where extra logic is added to support multiple panes.

Internal note:
uniqueAxis here most probably should compare not only object, but also position of the axis. If axis.pos and axis.len are the same, then leave axis out of the scope. Not sure if this will not break something with plotLines/plotBands/gridLines.

Example for generated paths Highcharts vs Highstock: http://jsfiddle.net/yu3encs3/

Contributor

pawelfus commented Jun 23, 2016

Thank you for reporting. You are right, issue is visible only in Highstock, where extra logic is added to support multiple panes.

Internal note:
uniqueAxis here most probably should compare not only object, but also position of the axis. If axis.pos and axis.len are the same, then leave axis out of the scope. Not sure if this will not break something with plotLines/plotBands/gridLines.

Example for generated paths Highcharts vs Highstock: http://jsfiddle.net/yu3encs3/

@jon-a-nygaard

This comment has been minimized.

Show comment
Hide comment
@jon-a-nygaard

jon-a-nygaard Apr 3, 2017

Collaborator

@PaulDalek @pawelfus Tried to locate this issue, but was not able to do so. Does this issue still exist?

Collaborator

jon-a-nygaard commented Apr 3, 2017

@PaulDalek @pawelfus Tried to locate this issue, but was not able to do so. Does this issue still exist?

@pawelfus

This comment has been minimized.

Show comment
Hide comment
@pawelfus

pawelfus Apr 4, 2017

Contributor

Yes, it does, compare:

Contributor

pawelfus commented Apr 4, 2017

Yes, it does, compare:

@jon-a-nygaard jon-a-nygaard self-assigned this Apr 20, 2017

@jon-a-nygaard jon-a-nygaard added this to the 5.0.11 milestone Apr 20, 2017

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