/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

Drilldown chart not rendered in styled mode #7523

Closed
oluwasayo opened this Issue Dec 13, 2017 · 1 comment

Comments

Assignees

@KacperMadej KacperMadej

Labels
Bug
Projects
None yet
Milestone
No milestone
2 participants
@oluwasayo @KacperMadej
@oluwasayo

oluwasayo commented Dec 13, 2017 •
edited

Expected behaviour

Columns should display properly at every drilldown level
https://imgur.com/a/jWHqW

Actual behaviour

Top-level drilldown columns not rendered. Drilling down into the innermost level (by clicking on the axis labels) displays but not the top levels even when drilling back up.
https://imgur.com/a/HxfZo

Live demo with steps to reproduce

http://jsfiddle.net/oluwasayo/w158g3pm/

Affected browser(s)

Chrome
Firefox
Safari

oluwasayo added a commit to oluwasayo/highcharts that referenced this issue Dec 13, 2017

@oluwasayo
highcharts#7523 Properly process drilldown color in styled mode 
Fixes highcharts#7523
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
3f33d46

This was referenced Dec 13, 2017

Closed
Merged
@KacperMadej

This comment has been minimized.

Sign in to view
Contributor

KacperMadej commented Dec 13, 2017

Hi @oluwasayo

Thank you for reporting about the problem.
You are mixing styled mode with classic. Use one or the another, not both.

Classic mode (working fine): http://jsfiddle.net/BlackLabel/w158g3pm/1/
Styled mode (bug visible - error in console): http://jsfiddle.net/BlackLabel/w158g3pm/2/

@KacperMadej KacperMadej added the Bug label Dec 13, 2017

@KacperMadej KacperMadej self-assigned this Dec 13, 2017

oluwasayo added a commit to vaadin/vaadin-charts that referenced this issue Dec 14, 2017

@oluwasayo
Drilldown workaround for highcharts/highcharts#7523
Verified
This commit was signed with a verified signature.
@oluwasayo oluwasayo Sayo Oladeji
GPG key ID: 9F3FE22065B24740 Learn about signing commits
64a3a47

oluwasayo added a commit to vaadin/vaadin-charts that referenced this issue Dec 14, 2017

@oluwasayo
CHARTS-692: Event handling enhancement (#252) 
* Series visibility toggle support

* Safe navigation workaround for selection and click events in vaadin-charts-flow because of vaadin/flow#3102

* Drilldown workaround because of highcharts/highcharts#7523
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
714192c

@TorsteinHonsi TorsteinHonsi closed this in 21f3846 Dec 18, 2017

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