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

Series values not grouped for two series after drilldown (for multiple series with more than 48 values) #6704

Closed
igimenez opened this issue May 9, 2017 · 3 comments

Comments

@igimenez
Copy link

@igimenez igimenez commented May 9, 2017

Expected behaviour

Chart grouping values for different series after drilldown in the right order

Actual behaviour

After drilldown some values are not grouped as they are supposed to

Live demo with steps to reproduce

http://jsfiddle.net/x9usgfe4/2/

  1. Click on Product B (Label, not chart series)

-> The columns apear separated for same markets

  1. if I click and unclick one of the series label (Products 2015 Product B for example) the chart magically fixes itself.
@igimenez igimenez changed the title Series values not grouped for two series after drilldown Series values not grouped for two series after drilldown (for multiple series) May 9, 2017
@igimenez igimenez changed the title Series values not grouped for two series after drilldown (for multiple series) Series values not grouped for two series after drilldown (for multiple series with more than 48 values) May 9, 2017
@TorsteinHonsi
Copy link
Collaborator

@TorsteinHonsi TorsteinHonsi commented May 10, 2017

Internal note:
The xAxis.names array gets extended during drilldown, so at some stage, both the old and the new categories are part of the names. That's why the offset of the second series is wrong.

@igimenez
Copy link
Author

@igimenez igimenez commented May 10, 2017

Thanks @TorsteinHonsi ! Will this be available in the next release?

@pawelfus
Copy link
Contributor

@pawelfus pawelfus commented May 10, 2017

Hi @igimenez
It's on the master branch so it should be included in the next release.

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
3 participants