Skip to content

Conversation

etimberg
Copy link
Member

@etimberg etimberg commented Jan 9, 2016

Fixes #1862

@etimberg etimberg added this to the Version 2.0 milestone Jan 9, 2016
etimberg added a commit that referenced this pull request Jan 9, 2016
Fix stacked combo bar charts & add test
@etimberg etimberg merged commit 6b28790 into v2.0-dev Jan 9, 2016
@etimberg etimberg deleted the fix/1862 branch January 9, 2016 15:01
@cdwilhelm
Copy link

@etimberg thank you. when does Chart.js get updated in v2.0-dev branch?

@etimberg
Copy link
Member Author

etimberg commented Jan 9, 2016

@cdwilhelm I just pushed up copies of the built files

@cdwilhelm
Copy link

@etimberg thank you, that indeed solve the problem I reported. Thank you for the quick turn around.
I updated my chart to have multiple lines as y-axes and removed the stack. with that change the x-axes loses data. Is this another bug or do I need to change some params? I didn't see anything that may have helped me.

@etimberg
Copy link
Member Author

@cdwilhelm can you create a JSfiddle or post your config?

@cdwilhelm
Copy link

@etimberg
Copy link
Member Author

@cdwilhelm the X axis is still marked as stacked. Setting it to false is what you'd want.

@cdwilhelm
Copy link

@etimberg yes, that is true. I'm looking to have x-axes stacked, but not the y-axes. my use case will have many bar datasets and it will be better represented as a stacked bar then individual bars.

When the x-axes has stacked: true, the bars are not represented as described in the datasets. for example Jan should should show 1 from each dataset (red and green bar) the others are not shown correctly either, each are missing data or not correctly calculated.

@etimberg
Copy link
Member Author

@cdwilhelm Ah, ok. I see what you're looking for. When only the x axis is stacked, the bars draw on top of each other which is why you only see one. When the y axis is stacked, the bars display correctly but then the lines also stack. I can think of a couple of work-arounds, but none seem ideal. Can you please file a separate issue for this?

@cdwilhelm
Copy link

@etimberg I sure will, thanks for the evaluation and all the help.

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

Successfully merging this pull request may close these issues.

2 participants