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

add pattern to sunburst #6622

Merged
merged 4 commits into from Jun 2, 2023
Merged

Conversation

thierryVergult
Copy link
Contributor

follow up on #6601 and #6619 where pattern was added to pie & funnelarea.

1st version, setting the attributes & defaults in the extended schema, and rework styleOne, in case a marker point is set
issue : check if fgcolor is an array
@archmoj
Copy link
Contributor

archmoj commented Jun 1, 2023

Wow, this would be another really nice feature.
Thank you!

Please collect & update the baseline from here.

I just noticed a bug i.e. by clicking the sunburst, patterns disappear.

@thierryVergult
Copy link
Contributor Author

thierryVergult commented Jun 1, 2023 via email

@archmoj
Copy link
Contributor

archmoj commented Jun 1, 2023

It is fine to pick the baselines from CircleCI.
To generate them locally (just in case you have a PR with multiple mocks), please see
https://github.com/plotly/plotly.js/blob/master/CONTRIBUTING.md#draft-new-baselines

@archmoj
Copy link
Contributor

archmoj commented Jun 2, 2023

Nicely done.
💃

@archmoj archmoj merged commit d31839e into plotly:master Jun 2, 2023
1 check passed
@thierryVergult thierryVergult deleted the sunburstPattern branch June 5, 2023 15:34
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.

None yet

2 participants