Skip to content
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

Current price label (price-indicator) does not work with multiple axis #16940

Closed
pjm17971 opened this issue Jan 31, 2022 · 4 comments · Fixed by #18639
Closed

Current price label (price-indicator) does not work with multiple axis #16940

pjm17971 opened this issue Jan 31, 2022 · 4 comments · Fixed by #18639

Comments

@pjm17971
Copy link

Expected behaviour

If you have say four axis, two on the right, two on the left, corresponding to 4 series, then the current price indicator would show correctly on the axis corresponding to that series.

Actual behaviour

The price indicators show on the same axis on the right, and same axis on the left. See this screenshot from our client:

Screen Shot 2022-01-31 at 11 13 15 AM

Live demo with steps to reproduce

Can't create at this time

Product version

"highcharts": "^9.1.1",
"highcharts-react-official": "^3.0.0",

Affected browser(s)

All

@highsoft-bot highsoft-bot added this to To do in Development-Flow Jan 31, 2022
@raf18seb
Copy link
Contributor

raf18seb commented Feb 1, 2022

Hi @pjm17971, thanks for reporting a problem! I agree, a label could be placed directly over an axis. The current behavior is inconsistent.

Demo: https://jsfiddle.net/BlackLabel/tvgzsL3k/

@pjm17971
Copy link
Author

pjm17971 commented Feb 1, 2022

@raf18seb @mfilipiec Yeah, kind of like that. Thanks for the jsfiddle. Here's a fork of that to show what's wrong more closely:
https://jsfiddle.net/sxp5n43L/1/

@pawelfus
Copy link
Contributor

pawelfus commented Mar 9, 2022

Internal note:
It's not directly caused by the Price indicator, but by the logic shared by Price Indicator and Crosshair: https://jsfiddle.net/BlackLabel/tvgzsL3k/1/

@pjm17971
Copy link
Author

This is awesome! Thank you for fixing.

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

Successfully merging a pull request may close this issue.

6 participants