Skip to content

Conversation

migsc
Copy link
Contributor

@migsc migsc commented Apr 29, 2016

Pie chart wasn't randomizing colors when Randomize Data button was clicked. The code to do this was commented out but incorrect. Fixed it and added it back it in.

Figure someone learning the API would like to see the color change effect. Also, this makes it consistent with other chart samples like bar that do the color change on randomize.

@etimberg
Copy link
Member

Thanks @mchateloin
Will merge when CI finishes

@coveralls
Copy link

Coverage Status

Coverage remained the same at 77.909% when pulling 3067249 on mchateloin:pie-demo-randomize-colors into adc430f on chartjs:master.

@etimberg etimberg merged commit 99c2d67 into chartjs:master Apr 29, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants