[8.16] Backport rowHeightOption: auto EuiDataGrid fix#211601
[8.16] Backport rowHeightOption: auto EuiDataGrid fix#211601weronikaolejniczak wants to merge 2 commits intoelastic:8.16from
Conversation
kqualters-elastic
left a comment
There was a problem hiding this comment.
we might have another issue with the auto row height, blocking this until we talk through it
logeekal
left a comment
There was a problem hiding this comment.
Testing it in following browser. Looks good from my side but I will wait to see what @kqualters-elastic has encountered
- Firefox
- Chrome
- Edge ( Chromium Based )
- Brave ( Chromium Based )
- Safari
|
@kqualters-elastic I will update the PR with the suggestion from @jbudz just in case we proceed with the release. |
logeekal
left a comment
There was a problem hiding this comment.
I think I know what Kevin might have encountered. But let's wait for more details. As I mentioned in 8.17 backport, I am seeing the same error in this backport as well.
The error surfaces when the height of the complete table and row are both auto.
In security Solution, whenever the number of items in Alert table is below 20, we set the table height as auto which results in this wierd error.
Screen.Recording.2025-02-20.at.11.43.14.mov
💛 Build succeeded, but was flaky
Failed CI StepsTest Failures
Metrics [docs]Page load bundle
|
|
Closing the backport PRs due to the above-mentioned issues. Going back to the drawing board with this issue. |
Summary
Fix for elastic/eui#8245
Backporting af189da to
8.16Kibana version. It was introduced on elastic/eui#8251.The npm backport release: https://www.npmjs.com/package/@elastic/eui/v/97.0.0-backport.3
Checklist
The issue being backported, elastic/eui#8245, is intermittent. You cannot consistently reproduce it on OSX, apparently it's reproducible on Windows. I'd appreciate a manual test from the reviewers 🙏🏻
The steps to reproduce in
8.16are:The data grid's height changes to 0. It's reproducible on first load.