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

errorbar not visible with default 1 month zoom #20097

Closed
gregory-clement opened this issue Nov 7, 2023 · 2 comments · Fixed by #20197
Closed

errorbar not visible with default 1 month zoom #20097

gregory-clement opened this issue Nov 7, 2023 · 2 comments · Fixed by #20197

Comments

@gregory-clement
Copy link

gregory-clement commented Nov 7, 2023

Expected behaviour

The errorbar should be displayed when the page is loaded

Actual behaviour

The errorbar is not displayed when the page is loaded. After selecting "All" zoom level, errorbar is correctly displayed. Then, if the "1 month" zoom is selected, the errorbar is still displayed.

If I delete 1 (or more) line of data, the bug disappears.

Live demo with steps to reproduce

https://jsfiddle.net/gregoryclement/eashqu81/30/

Product version

Highstock JS v11.2.0 (2023-10-30)

Affected browser(s)

Edge, Chrome, Firefox

@highsoft-bot highsoft-bot added this to To do in Development-Flow via automation Nov 7, 2023
@bm64
Copy link
Member

bm64 commented Nov 7, 2023

Hello @gregory-clement

Looks like it used to work correctly in v11.1.0 and has been broken since v11.2.0: https://jsfiddle.net/BlackLabel/heq6c7kf/

@pawelfus
Copy link
Contributor

pawelfus commented Nov 10, 2023

Bisected to commit 52a6ede0a5

Recreated without Stock: https://jsfiddle.net/BlackLabel/zn5v0b9x/

FYI @bre1470 - it seems cropped data can have empty elements in array, which were easily skipped with forEach(), but for of will reach them

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.

5 participants