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

Bug with events.load setSize() #15400

Closed
cgahmann opened this issue Mar 25, 2021 · 4 comments · Fixed by #15645
Closed

Bug with events.load setSize() #15400

cgahmann opened this issue Mar 25, 2021 · 4 comments · Fixed by #15645
Assignees

Comments

@cgahmann
Copy link

Expected behaviour

Show me all bars

Actual behaviour

When I calculate the height of the chart dynamically, it shows me only the last bars in the bar chart. The others come sometime later or when I change the height of the page so it has to recalculate the chart.
It works if I take the highchart.js (https://code.highcharts.com/highcharts.js) instead of highstocks.js (https://code.highcharts.com/stock/highstock.js)

Live demo with steps to reproduce

http://jsfiddle.net/wmys9xkc/

Product version

Latest stable Highstock

Affected browser(s)

Google Chrome, Firefox in MacOS

@highsoft-bot highsoft-bot added this to To do in Development-Flow Mar 25, 2021
@khlieng
Copy link
Member

khlieng commented Mar 25, 2021

This was a clipping bug in stock that has been fixed on master (#15200): http://jsfiddle.net/9fzqx0ne/

@khlieng khlieng closed this as completed Mar 25, 2021
Development-Flow automation moved this from To do to Done Mar 25, 2021
@cgahmann
Copy link
Author

cgahmann commented May 4, 2021

@khlieng I guess it looks like the fix is out again? http://jsfiddle.net/9fzqx0ne/

@khlieng
Copy link
Member

khlieng commented May 4, 2021

Hmm, seems so, you can work around it for now by running the setSize in a rAF or zero setTimeout: http://jsfiddle.net/n1bgah78/

@khlieng
Copy link
Member

khlieng commented May 4, 2021

Its a regression since #15516.

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.

3 participants