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

After removing an annotation the chart throws errors #7677

Closed
morganfree opened this issue Jan 16, 2018 · 0 comments
Closed

After removing an annotation the chart throws errors #7677

morganfree opened this issue Jan 16, 2018 · 0 comments

Comments

@morganfree
Copy link
Contributor

@morganfree morganfree commented Jan 16, 2018

Actual behaviour

After chart.removeAnnotation() the chart throws the error:
Uncaught TypeError: Array.prototype.forEach called on null or undefined

Live demo with steps to reproduce

http://jsfiddle.net/5L7ta5fb/

  1. Run the fiddle
  2. Open the browser console

Product version

All products 6+

An annotation is not destroyed correctly - chart.labelsCollectors still runs the removed annotation's label collector - the collector refers to removed props.

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

Successfully merging a pull request may close this issue.

None yet
1 participant