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

chart.update() doesn't enable the navigator under certain conditions #7067

Closed
ransoing opened this issue Aug 16, 2017 · 4 comments
Closed

chart.update() doesn't enable the navigator under certain conditions #7067

ransoing opened this issue Aug 16, 2017 · 4 comments

Comments

@ransoing
Copy link

Expected behaviour

Calling chart.update() to toggle navigator.enabled should always work properly.

Actual behaviour

If a plot is initialized with both scrollbar.enabled and navigator.enabled set to false, then the navigator will not appear when chart.update( {navigator: {enabled:true}} ) is called.

If the plot is initialized with either scrollbar.enabled or navigator.enabled set to true, then it will work properly.

Live demo with steps to reproduce

Press the "Toggle Navigators" button. Note how the second plot does not show a navigator.
http://jsfiddle.net/x40me94t/1/

@KacperMadej
Copy link

Hi,

Thank you for reporting about the bug.

Internalnote: Navigator is added only on beforeRender event and cannot be added through chart.update.

@sebastianbochan
Copy link
Contributor

Internal note
It seems that options are not "updated" in update function. However when we edit it, it is still wrong.

Demo:

@KacperMadej
Copy link

Workaround:
Set options for navigator manually, fire event that adds navigator/scrollbar, redraw the chart to see the changes.
Demo: http://jsfiddle.net/x40me94t/2/

@ghost
Copy link

ghost commented May 29, 2018

Internal note
Requires unit test (scheduled for test/move_issues_tests_hs4.0.1)

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

No branches or pull requests

4 participants