Skip to content

/ highcharts

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

Jump to bottom

Unable to render a chart in a child of a detached node #7024

Closed
natalan opened this issue Jul 31, 2017 · 1 comment
Closed

Unable to render a chart in a child of a detached node #7024

natalan opened this issue Jul 31, 2017 · 1 comment
Labels
Type: Bug Type: Regression

Comments

@natalan
Copy link

@natalan natalan commented Jul 31, 2017

@TorsteinHonsi there is another regression since 5800b1b:

chart.renderTo node is getting removed from a parent container and never put back to the original location after "pre-rendering" and removal here

It was working as expected in 5.0.12 and broken in 5.0.13 and 5.0.14

Live demo with steps to reproduce

http://jsfiddle.net/45mt97xs/
@sebastianbochan
Copy link
Contributor

@sebastianbochan sebastianbochan commented Aug 1, 2017

Hi @natalan
Thank you for the reporting.
@TorsteinHonsi TorsteinHonsi mentioned this issue Aug 28, 2017
Closed
oluwasayo pushed a commit to vaadin/vaadin-charts that referenced this issue Jun 25, 2018
Sayo Oladeji
Remove workaround for highcharts/highcharts#7024
Unverified
No user is associated with the committer email.
Learn about signing commits
Loading status checks…
0ac25d0
oluwasayo pushed a commit to vaadin/vaadin-charts that referenced this issue Jun 25, 2018
Sayo Oladeji
Remove workaround for highcharts/highcharts#7024
Unverified
No user is associated with the committer email.
Learn about signing commits
04576e0
oluwasayo added a commit to vaadin/vaadin-charts that referenced this issue Jun 25, 2018
@oluwasayo
Upgrade to Highcharts 6 (#381)
Verified
This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.
GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits
Loading status checks…
3ee23cd 
* Upgrade to Highcharts 6

* Fix 3D by performing 3D setup after next render after initialization completes. Add observers after initialization.

* 3D Stacking workaround. highcharts/highcharts#8417

* Move 'stacking' from vaadin-chart-series to vaadin-chart as a result of highcharts/highcharts#8475

* Remove test cases for inferred X in series data as a result of highcharts/highcharts#8462

* Remove workaround for highcharts/highcharts#7024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Type: Bug Type: Regression
Projects
None yet
Milestone
No milestone
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
@natalan @sebastianbochan @KacperMadej