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.

Sign up for GitHub

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

Jump to bottom

sunburst text does not fit inside for 0-valued sectors #4569

Closed
emmanuelle opened this issue Feb 11, 2020 · 5 comments · Fixed by #4580
Closed

sunburst text does not fit inside for 0-valued sectors #4569

emmanuelle opened this issue Feb 11, 2020 · 5 comments · Fixed by #4580
Assignees
@archmoj
Labels
bug something broken

Comments

@emmanuelle
Copy link
Contributor

See https://codepen.io/emmanuelle-plotly/pen/JjdYPBN?editable=true, for the sector with 0 value the text label is still displayed even when setting uniformtext parameters. Changing the value to a very small number (e.g. 1.e-7) fixes the issue. Raised by https://community.plot.ly/t/help-with-sunburst-using-plotly-express-and-path/34823/
@emmanuelle
Copy link
Contributor Author

I'm not sure whether this is fixed by #4519

@archmoj
Copy link
Contributor

archmoj commented Feb 11, 2020

I'm not sure whether this is fixed by #4519

@emmanuelle No #4519 is not related/helpful in this regard.

@archmoj
Copy link
Contributor

archmoj commented Feb 13, 2020

Minimal demo

@archmoj
Copy link
Contributor

archmoj commented Feb 13, 2020

This in not related uniformtext.
Basically the text should not be displayed over the sector.
Demo without uniformtext.

@archmoj archmoj changed the title Uniformtext in sunburst does not work correctly for 0-valued sectors sunburst text does not fit inside for 0-valued sectors Feb 13, 2020
@archmoj
Copy link
Contributor

archmoj commented Feb 13, 2020

Not a regression.

@archmoj archmoj added the bug something broken label Feb 13, 2020
@archmoj archmoj self-assigned this Feb 13, 2020
@archmoj archmoj mentioned this issue Feb 13, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@archmoj archmoj

Labels
bug something broken
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Ensure text fits inside sunburst sectors with zero values plotly/plotly.js
2 participants
@emmanuelle @archmoj