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

[Discover] Remove legacy/classic table #167582

Open
davismcphee opened this issue Sep 28, 2023 · 5 comments
Open

[Discover] Remove legacy/classic table #167582

davismcphee opened this issue Sep 28, 2023 · 5 comments
Assignees
Labels
Breaking Change Feature:Discover Discover Application impact:high Addressing this issue will have a high level of impact on the quality/strength of our product. loe:medium Medium Level of Effort Team:DataDiscovery Discover App Team (Document Explorer, Saved Search, Surrounding documents, Data, DataViews) technical debt Improvement of the software architecture and operational architecture

Comments

@davismcphee
Copy link
Contributor

davismcphee commented Sep 28, 2023

In version 9.0, we are removing the legacy/classic table in Discover.

Image

The following needs to be done:

  • Remove the doc_table:legacy setting from the Discover app and all associated code related to the legacy table view.
  • Remove the truncate:maxHeight setting since it is only relevant to the legacy table.
  • Remove all the code + tests depending on these settings
@davismcphee davismcphee added Meta Feature:Discover Discover Application Team:DataDiscovery Discover App Team (Document Explorer, Saved Search, Surrounding documents, Data, DataViews) labels Sep 28, 2023
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-data-discovery (Team:DataDiscovery)

@davismcphee davismcphee added loe:medium Medium Level of Effort impact:low Addressing this issue will have a low level of impact on the quality/strength of our product. labels Sep 28, 2023
@kertal
Copy link
Member

kertal commented Sep 29, 2023

The legacy table doesn't use virtualization when rendering table rows, instead rendering them all at once, which has the side effect of supporting local filtering through built in find functionality in browsers. Unified Data Table is based on EuiDataGrid, which uses virtualization and prevents relying on browser find functionality for local filtering. In order to support this we'd need to implement in-app filtering in Unified Data Table.

About the legacy in-app filtering... it depends, because it renders the list in chunks when scrolling. So when you scroll down you can use the browser to find in all results displayed. Apart from that detail, yes, we should implement searching in the results #70799

@davismcphee
Copy link
Contributor Author

@kertal thanks for the clarification! Updated the bullet to include that info.

@kertal kertal added epic and removed Meta labels Nov 30, 2023
@jughosta
Copy link
Contributor

jughosta commented Aug 9, 2024

And we could remove truncate:maxHeight

@kertal kertal added technical debt Improvement of the software architecture and operational architecture and removed epic labels Sep 25, 2024
@kertal kertal changed the title [Discover] Remove legacy table [Discover] Remove legacy/classic table Oct 16, 2024
@kertal
Copy link
Member

kertal commented Oct 16, 2024

I rewrote this issue to describe the task that needs to be done, we just need to decide when it's the right time, to omit backports of other issues that follow this merge, won't run into too much conflicts

@kertal kertal added Breaking Change impact:high Addressing this issue will have a high level of impact on the quality/strength of our product. and removed impact:low Addressing this issue will have a low level of impact on the quality/strength of our product. labels Oct 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Breaking Change Feature:Discover Discover Application impact:high Addressing this issue will have a high level of impact on the quality/strength of our product. loe:medium Medium Level of Effort Team:DataDiscovery Discover App Team (Document Explorer, Saved Search, Surrounding documents, Data, DataViews) technical debt Improvement of the software architecture and operational architecture
Projects
None yet
Development

No branches or pull requests

4 participants