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

Stack labels' verticalAlign doesn't work for logarithmic yAxis. #18501

Closed
ddragula opened this issue Feb 15, 2023 · 2 comments · Fixed by #18534
Closed

Stack labels' verticalAlign doesn't work for logarithmic yAxis. #18501

ddragula opened this issue Feb 15, 2023 · 2 comments · Fixed by #18534

Comments

@ddragula
Copy link
Member

Expected behaviour

Setting stackLabels.verticalAlign in the logarithmic axis to any value works as in the regular axis.

Actual behaviour

Setting stackLabels.verticalAlign in the logarithmic axis to bottom or middle makes the labels invisible.

Live demo with steps to reproduce

https://jsfiddle.net/BlackLabel/fntvzh31/

Product version

Highcharts v10.3.3

@ddragula ddragula changed the title StackLabels' verticalAlign doesn't work for logarithmic yAxis. Stack labels' verticalAlign doesn't work for logarithmic yAxis. Feb 15, 2023
@NathanManning
Copy link

Please make sure it works with the original data layout as well.

broken: https://jsfiddle.net/o0L9wmhu/
working: https://jsfiddle.net/m9touxh8/1/

@hubertkozik hubertkozik self-assigned this Feb 17, 2023
@hubertkozik
Copy link
Member

Hi guys! There is a problem with calculating the 0 value to pixels on logarithmic axes - I will fix in HC Core. Now, I have a workaround for that issue, which overrides an HC method. You can check the demo below (the plugin is inside the IIFE function). A similar solution will be used in HC Core.

Workaround demos: https://jsfiddle.net/BlackLabel/w8ck6osf/
https://jsfiddle.net/BlackLabel/m5zpf0wh/

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

Successfully merging a pull request may close this issue.

4 participants