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

feat: expose all hierarchical information to the partition tooltip #1843

Merged

Conversation

markov00
Copy link
Member

@markov00 markov00 commented Oct 11, 2022

Summary

This PR exposes all the layers of a partition chart in the tooltip.
Only linear partition charts (flamegraph and icicle graph) are excluded from this behaviour, due to their possible deep hierarchy.

Screenshot 2022-10-12 at 15 23 55

Screenshot 2022-10-12 at 14 52 17

Details

The percentValueGetter argument type was updated to be able to pass the various ArrayNode, ShapeTreeNode etc that have the same concept but all of them slightly different parameters.

Issues

fix #623

Checklist

  • The proper chart type label has been added (e.g. :xy, :partition)
  • The proper feature labels have been added (e.g. :interactions, :axis)
  • All related issues have been linked (i.e. closes #123, fixes #123)
  • Unit tests have been added or updated to match the most common scenarios
  • The proper documentation and/or storybook story has been added or updated
  • Visual changes have been tested with all available themes including dark, light, eui-dark & eui-light

@markov00 markov00 added :tooltip Related to hover tooltip :partition Partition/PieChart/Donut/Sunburst/Treemap chart related labels Oct 11, 2022
@markov00
Copy link
Member Author

buildkite update screenshots

@markov00 markov00 marked this pull request as ready for review October 12, 2022 08:44
@markov00 markov00 requested a review from monfera October 12, 2022 08:44
@markov00 markov00 merged commit 136b7ae into elastic:master Oct 12, 2022
@markov00 markov00 deleted the 2022_10_11-hierarchical_partition_tooltip branch October 12, 2022 16:21
nickofthyme pushed a commit that referenced this pull request Nov 3, 2022
# [50.1.0](v50.0.2...v50.1.0) (2022-11-03)

### Bug Fixes

* **deps:** update dependency @elastic/eui to v70 ([#1850](#1850)) ([d05994e](d05994e))

### Features

* **partition:** show hierarchical information in the tooltip ([#1843](#1843)) ([136b7ae](136b7ae))
* **xy:** quarterly time and zoom pan numerical raster as well as other improvs ([#1856](#1856)) ([75d826a](75d826a))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:partition Partition/PieChart/Donut/Sunburst/Treemap chart related :tooltip Related to hover tooltip
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Piechart tooltip shows field names
2 participants