-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
[Visualize] [Lens] Inspector 'filter by' creates a standard filter instead of filtering time range for a time interval #116752
Comments
Pinging @elastic/kibana-vis-editors (Team:VisEditors) |
As described here #122113 (comment) , this is blocked on #118851 as we need some additional meta data on the data table to make it work correctly in all cases. |
Actually, this meta data is already available for all cases. If |
Just realized this is bigger than VisEditors because this part of the inspector is maintained as part of the data plugin: kibana/src/plugins/data/public/utils/table_inspector_view/components/data_table.tsx Line 83 in 9d662b7
@elastic/kibana-app-services Could you take a look at that? |
Pinging @elastic/kibana-presentation (Team:Presentation) |
Pinging @elastic/kibana-data-discovery (Team:DataDiscovery) |
Reassigning this to the Data Discovery team because it looks like the change will need to happen in the Data plugin |
When creating a data view, you select a time field. In my opinion, the type of filter we create should be based on whether or not the field is the selected time field for the data view. Here's the two scenarios:
I believe this is how it has historically worked, and makes the most sense. |
I agree with you Lukas ++ |
Kibana version:
7.16, master
Not sure if it's a wanted behavior, so feel free to close it if there's a reason for it.
2. Click on 'Inspect' and choose 'filter for value' in any of the intervals:
The filter was added:
Works the same across Lens and Visualize.
What we would like instead probably, is to narrow the timerange in a timepicker, just like it behaves when clicking in a table.
Oct-29-2021.13-05-45.mp4
The text was updated successfully, but these errors were encountered: