| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
|
PatternFly-React preview: https://patternfly-react-pr-2497.surge.sh |
Sorry, something went wrong.
|
This looks good to me and should resolve my default background issue in #2476 (so I'd be able to use the same theme). The responsive use case you've added here should help. I'd argue we should try to add those examples for test purposes for each Chart in the future w/ the way these svgs are scaling. I think what's potentially still different about my use case in 35 is I'm expected to render around 30 x-axis labels responsively (days of the month). When I compress this chart for mobile, I'm seeing some weirdness with the Voronoi tooltip hover, where if bars become close, the first bar hovered is still showing it's tooltip when I move to hover a second bar. I'm not seeing this same buggy behavior happen when I pass the default ChartTooltip to the ChartBar labelComponent like the example here: I will continue to experiment here, but it's just something to test w/ VoronoiContainer in the future. Not sure if you would see the same thing here if trying to render several x-axis labels. |
Sorry, something went wrong.
There was a problem hiding this comment.
Looking good. So if I undestand this correctly I will add voronoiDimension to whichever axis I want to?
Sorry, something went wrong.
There was a problem hiding this comment.
Always happy to see cleaner code with functionality intact :)
Sorry, something went wrong.
|
Your changes have been released in:
Thanks for your contribution! 🎉 |
Sorry, something went wrong.
|
Thank you for the reviews! @karelhala Yes, please use voronoiDimension for your dependent axis. Or, try the approach used by the stack chart example. @priley86 Although, a voronoi container appears to work with bar charts, it may not be best when wrapped by a stack chart? I've also encountered some (non-mobile) jumpy hovers with the stack example. You could try working with a basic container, instead? I've also modified the stack chart example to show an alternative way to apply tooltips without voronoi container -- that fixed the tooltip hover behavior. |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
Adjusted/added pf-core vars to ensure custom tooltip colors are displayed correctly. Previously, the background and label were the same black color, so the text wasn't visible.
We were also applying old, hard coded color properties over the pf-core vars, so the theme had to be cleaned up a bit.
Added the voronoiDimension prop to fix the tooltip hover for our bar chart examples. Then, modified the stack chart example to show how similar tooltips are applied without a voronoi container.
Also modified an area chart example, showing how width can be responsive (per @priley86 's request).
Fixes #2496
Fixes #2482