Skip to content

pie chart hover label hidden #3654

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

Closed
destradafilm opened this issue Mar 19, 2019 · 4 comments · Fixed by #3681
Closed

pie chart hover label hidden #3654

destradafilm opened this issue Mar 19, 2019 · 4 comments · Fixed by #3681
Assignees
Labels
bug something broken

Comments

@destradafilm
Copy link

destradafilm commented Mar 19, 2019

long hoverlabels get hidden outside of plot boundries instead of making sure they fit within the plot container's range

the label for this slice in the pie chart is '1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16' but you can only see from 7 onward.

Screen Shot 2019-03-19 at 4 26 25 PM

Maybe the label placement can be different and if the label is wider than the chart, it can wrap text?

@etpinard
Copy link
Contributor

Thanks for posting!

What are your graph's width, height and margins here?

Or better yet, can you share a reproducible codepen showing the problem? Thank you!

@destradafilm
Copy link
Author

Hi @etpinard

Here is a codepen with the example. Thanks for taking a look

https://codepen.io/destrada/pen/BbbGgJ

@etpinard etpinard added the bug something broken label Mar 25, 2019
@etpinard
Copy link
Contributor

etpinard commented Mar 25, 2019

I'll submit a patch that constrains the hover labels to fit inside the graph's bounding box, but just so you know @destradafilm your long green label still won't fit:

Peek 2019-03-25 16-49

that will have to wait for #1964

@destradafilm
Copy link
Author

it's been almost 5 years since I opened this case - I came across the same bug again. Here is a codepen
https://codepen.io/destrada/pen/NWmxrVe

screenshot of the label being clipped out of visible chart area
Screen Shot 2024-03-07 at 3 28 25 PM

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 a pull request may close this issue.

2 participants