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

Fix: applyStack() returned the sum of all values for hidden dataset indices, which causes incorrect animations when showing/hiding stacked datasets. #11938

Merged

Conversation

DeyLak
Copy link
Contributor

@DeyLak DeyLak commented Oct 24, 2024

…ndices, which causes incorrect animations when showing/hiding stacked datasets. This fixes #11198.

This is a clone of this with added test, as I promised there.
I didn't find an easy way to test applyStack directly, so I decided that it would be better to test the behaviour and not the implementation and added a test for bar chart rendering.

…ndices, which resulted in wrong show animations
@etimberg etimberg merged commit 6dd448b into chartjs:master Oct 24, 2024
8 checks passed
@DeyLak
Copy link
Contributor Author

DeyLak commented Oct 28, 2024

@etimberg I would really appreciate it, if you make a release with this change. Is it possible?

@etimberg
Copy link
Member

@DeyLak sure, will do a release for this.

@LeeLenaleee LeeLenaleee added this to the Version 4.5.0 milestone Oct 28, 2024
@LeeLenaleee LeeLenaleee changed the title Fix: applyStack() returned the sum of all values for hidden dataset i… Fix: applyStack() returned the sum of all values for hidden dataset indices, which causes incorrect animations when showing/hiding stacked datasets. Oct 28, 2024
@LeeLenaleee
Copy link
Collaborator

This is now released

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.

Stacked Bar Chart show/hide animations have wrong target values
4 participants