-
Notifications
You must be signed in to change notification settings - Fork 119
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
Allow tooltip option for max number of items #872
Comments
What we want to do in the case where the highlighted/hovered points are more than the |
I would say show the full list. Another option could be to only show the closest values to the cursor, but this would require this issue 👉 #835. To your other point, I agree, a more advanced interaction with the tooltip like scrolling or keyboard shortcuts would be a great improvement. How we do this is hard and would be nice to get the design teams' input |
Closing this as this is now available in the |
Yup good call, I think the latest changes cover this request. |
Is your feature request related to a problem? Please describe.
In TSVB in kibana there is an option to limit the tooltip items only to the focused items. There should be an option to specify the max number of items in the tooltip.
Describe the solution you'd like
This solution would add a new option to the
TooltipProps
to enable this feature and optionally set the max limit.Then internally elastic charts uses this
limitCount
prop to either set a default limit (i.e.10
) or use the passed limit value.The way this would work for a limit of
10
, if there are10
series on the tooltip, but only2
are highlighted, the tooltip is shown with all10
series. If another index has12
series on the tooltip, but only2
are highlighted, it will only show the highlighted count of2
.This should not effect the
type
of the tooltip.Describe alternatives you've considered
Using a toggle between
TooltipType.Follow
andTooltipType.VerticalCursor
to achieve this effect.https://github.com/elastic/kibana/blob/7e47e3935ea58cc69f566b8970f8f503e3fcbe36/src/plugins/vis_type_timeseries/public/application/visualizations/views/timeseries/index.js#L139-L143
Additional context
Related to tooltip redesign #615
cc: @markov00 @timroes
The text was updated successfully, but these errors were encountered: