-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[Lens] Auto height setting for table #115582
Comments
Pinging @elastic/kibana-vis-editors (Team:VisEditors) |
@flash1293 , we have the following task elastic/eui#5080 which provide a control inside grid which allow to switch between auto-height and usual state. I am not sure that we need add |
@VladLasitsa I think it makes sense to put it into the Lens toolbar for now because:
We absolutely should store this in the saved object, this is the most important part. Even if we decide to show the data grid toolbar at some point, we still want to store this setting in the saved object. |
Regarding this topic, I think we need to ask ourselves whether or not it would valuable for consumers/viewers of the visualization (i.e. not the person creating the visualization in Lens) to have access to this row height control. For example, is there a use case for folks viewing a Lens visualization in a dashboard to change this row height option and have that change stored client-side? If there is such a use case, then perhaps we should talk about introducing the data grid toolbar. If there is no such use case, then I agree with @flash1293 on placing this new option in the Lens app toolbar. As table visualizations don't currently have any toolbar buttons beyond visualization type, we'll need to add a new button called "Appearance" and using the |
This is a good point, but IMHO it’s a rare case to change this setting on a dashboard (at least too rare to introduce the toolbar for it which would take away quite a bit of vertical space, especially if there are multiple tables on a dense dashboard). Also, there’s another issue we would like to work on soon (allowing to paginate the table) which could go into the same popover menu - it seems like the most consistent approach to me. We could of course introduce a “show toolbar” switch, but that seems like overcomplicating the issue - I think it makes sense to go with the setting in Lens editor for now and see whether users request more control. |
Cc @ghudgins what do you think about this? |
I think we have a number of data grid things we might want to expose outside of the editor but for the sake of this issue we don't need to solve for those edit-from-view cases here. I also agree, for this case, it's definitely more rare than changing sorting or resizing a column. |
Aggregation based table has a setting to scale the row height of the table by the content. We should allow the user to specify this in Lens as well.
The text was updated successfully, but these errors were encountered: