-
Notifications
You must be signed in to change notification settings - Fork 120
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
Pie/sunburst chart - discussion about small slices #381
Comments
Essentially, all of the slices that cannot display a label would have a single overlay hover state encompassing all slices. It would highlight that entire section and then onClick would display a tooltip of all the slices and their values. |
@cchaos exactly what I was envisioning! Thanks for mocking up the visual. I think for the sake of filtering/selection of a specific slice we should allow the tooltip to be interactable, which would be new, such that the user can hover and click on any of the "other" values and on hover the selected slice would be exploded or highlighted. This is crude example with the limited tools on my mac 😞 (using @cchaos's as a base 😅) |
Looks good! Below are a few consequences:
Note: it may easily be the case that there are numerous slices which are still easy to hover over, but won't have static text labels associated with them due to their small size, ie. no room inside or outside to render text (I'm just trying to make it explicit that the "other" group is mostly a hover-convenience, as vanishingly small slices can't be hovered over. While it may be a point we all agree on, it looks like more than a coincidence that the cutoff on @cchaos's mock just happens to be past Netherlands, which is the last item with a static text label)
I'd like to mention that there are other options for those slices which are too small for static text, but large enough for hovering. For example, if other settings allow it, render their linked text on hover even at the expense of temporarily removing text from nearby slices. In short, let's continue to distinguish among
I'll think a bit more—feel free to discuss ideas, or question anything I said or implied. |
I agree with everything you said and great clarity on the issues we could run into. RE: 4 I think the EUI popover type of component is what we need, but not using EUI 😞. Secondly, is it typical to sort sunbursts? I would say that we avoid allowing sorting in a sunburst. I like the 3 designations you made between the slice "types" |
@nickofthyme In point 4 I didn't mean sorting the sunburst slices themselves, but simply, the rows in a tabular, scrollable popup (which is a consequence of the proposal, as there can be an infinite number of items in the tooltip, so it's gotta be scrollable, and also, it's a multicolumn table, because there's name and value / % as a minimum; user may want to see alphabetical or value sorting) |
FWIW I'd encourage us to look at the same scenario but use a hierarchical treemap layout. In treemap the consumer can visually compare the size of the cells, label only the biggest cells and hierarchy can be established via border size. Example: https://finviz.com/map.ashx |
Update with the #493 merge:
<Partition
/* ... */
config={{ partitionLayout: PartitionLayout.treemap }}
>
Residual burnout in the middle after adding 2A but before 2B: No burnout after 2A and 2B (this shows the tapering off even for larger slices, so its mechanism can be seen, but is no longer currently perceptible): Again, these don't directly address the grouping of slices into an "other" category, but is related to the discussion about numerous, therefore vanishingly small slices. |
Hi @cchaos, I'm looking forward to your input and suggestions on this, textual and image / drawing / interaction alike 🙇
This is a quick example for a sunburst diagram that has vanishingly small slices, potentially subpixel territory even at the outermost ring:
cc @nickofthyme @snide @miukimiu @markov00
The text was updated successfully, but these errors were encountered: