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

[RAM] Integration of EUI Data Grid's Custom Body Renderer for Alerts Table #159003

Closed
jcger opened this issue Jun 5, 2023 · 4 comments
Closed
Assignees
Labels
Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) v8.9.0

Comments

@jcger
Copy link
Contributor

jcger commented Jun 5, 2023

We have been experiencing intermittent issues with our UI components that utilize the EUI Data Grid, most notably our alerts table. These components occasionally display only 1.5 rows and the grid height appears broken. The problem is inconsistent and we currently have no reliable way of reproducing it. Our proposed solution is to implement the EUI Data Grid's Custom Body Render function, which may address this issue across all components using EUI Data Grid.

Issue Details

The sporadic nature of this issue poses a significant challenge. During these instances, the grid's height seems to be miscalculated or incorrectly set, leading to the improper display of data and limiting user interaction. Given that the issue occurs in multiple components that use the EUI Data Grid, we suspect the root cause lies within the grid implementation itself. We propose integrating the EUI Data Grid's Custom Body Renderer as a potential solution. This feature provides control over cell rendering, which could aid in addressing the intermittent layout issues we are encountering.

Reproduction and Verification Challenge

Success will be determined over time; if the issue ceases to be reported, we will consider the solution effective.

@jcger jcger added Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) v8.9.0 labels Jun 5, 2023
@elasticmachine
Copy link
Contributor

Pinging @elastic/response-ops (Team:ResponseOps)

@MakoWish
Copy link

MakoWish commented Jun 13, 2023

Seems like a regression to this #134268

We were previously experiencing it, but it was fixed at some point, and it has now returned after upgrading to 8.8.1 yesterday.

table_too_small

@nicpenning
Copy link

Sounds like it could be. We will see I suppose. Thanks for commenting!

@XavierM
Copy link
Contributor

XavierM commented Jul 12, 2023

this elastic/eui#6895 will fix the problem

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) v8.9.0
Projects
No open projects
Development

No branches or pull requests

5 participants