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 rendering of pie legends when arrays e.g. pattern or line.width having 4 items or more #6628

Merged
merged 2 commits into from
Jun 5, 2023

Conversation

archmoj
Copy link
Contributor

@archmoj archmoj commented Jun 5, 2023

Follow up of #3891 and #3891 (comment).

cc: @thierryVergult

@archmoj archmoj added the bug something broken label Jun 5, 2023
@archmoj archmoj merged commit f32c7ea into master Jun 5, 2023
@archmoj archmoj deleted the patterns-4-items-more branch June 5, 2023 20:00
var objOut = {};
if(typeof obj2 !== 'object') obj2 = {};
var arrayLen = 3;
var arrayLen = isPie = 3;
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure what happened with my editor here!
This logic is not quite right as it changes the input argument. This may slow the process.
I'll fix it and release a patch shortly.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed by #6632.

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

Successfully merging this pull request may close these issues.

1 participant