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

Column shake in EuiDataGrid in the security alert view #7019

Open
aarju opened this issue Aug 1, 2023 · 5 comments
Open

Column shake in EuiDataGrid in the security alert view #7019

aarju opened this issue Aug 1, 2023 · 5 comments
Labels
bug data grid help wanted The EUI team is looking for community members to pick up and implement this issue ⚠️ needs validation For bugs that need confirmation as to whether they're reproducible

Comments

@aarju
Copy link

aarju commented Aug 1, 2023

Describe the bug
After adding a column to the view in the security alert table if the column was set to a certain width it causes the table to constantly jump around displaying the scroll bars then not displaying them. See the below video for the bug in action. Resizing the column or the browser window stops the activity.

20230801-1120-57.8505939.mp4

Environment and versions

  • EUI version:
  • React version:
  • Kibana version (if applicable): 8.9.0
  • Browser: Chrome 115
  • Operating System: Windows

To Reproduce
I'm not really sure how to reproduce this. When adding a column it depends on the length of the values within the field in relation to the size of the browser window. When I tried to open the developer tools it changes the width of the browser display and causes the bug to stop.

Expected behavior
The screen shouldn't shake

Additional context
This is part of the Elastic Infosec team's SIEM cluster.

@aarju aarju added bug ⚠️ needs validation For bugs that need confirmation as to whether they're reproducible labels Aug 1, 2023
@1Copenut
Copy link
Contributor

1Copenut commented Aug 1, 2023

Thank you for the bug report @aarju. I'll reach out privately for a cloud URL to test.

@cee-chen
Copy link
Contributor

cee-chen commented Aug 1, 2023

@aarju can you confirm if this is happening in Kibana main or just 8.9? Kevin Qualters recently contributed an EuiDataGrid fix that should stabilize height/width recalculations (#6895). This fix is not in 8.9 but will be in 8.10, and we could potentially backport it to 8.9 if it does indeed fix the issue

@aarju
Copy link
Author

aarju commented Aug 1, 2023

This is in 8.9

@cee-chen
Copy link
Contributor

cee-chen commented Aug 2, 2023

Just wanted to recap the conversation we had in Slack:

Unfortunately we need the issue to be reproducible in EUI (either in a CodeSandbox or on our production docs site) to materially help figure out a fix. Our team is small, and we don't have the resources to chase down bugs that are both specific to a consumer's usage of a component and hard to reproduce.

That being said we also always welcome contributions to EUI, so if y'all manage to track this down on your own, please do send a PR our way! (or again, another option is waiting to see if 8.10 fixes the issue)

I'll leave this issue open for now in case @kqualters-elastic or another Elastic dev is able to work on this more/wants to pick this up, but EUI will not be able to work on this issue ourselves in its current state and this issue will auto-close in a year if no other activity is reported.

@cee-chen cee-chen added data grid help wanted The EUI team is looking for community members to pick up and implement this issue labels Aug 2, 2023
@github-actions
Copy link

github-actions bot commented Aug 2, 2023

👋 Thank you for your suggestion or request! While the EUI team agrees that it's valid, it's unlikely that we will prioritize this issue on our roadmap. We'll leave the issue open if you or anyone else in the community wants to implement it by contributing to EUI. If not, this issue will auto close in one year.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug data grid help wanted The EUI team is looking for community members to pick up and implement this issue ⚠️ needs validation For bugs that need confirmation as to whether they're reproducible
Projects
None yet
Development

No branches or pull requests

3 participants