[8.17] Backport rowHeightOption: auto EuiDataGrid fix

[weronikaolejniczak]

Summary

Fix for elastic/eui#8245

Backporting af189da to 8.17 Kibana version. It was introduced on elastic/eui#8251.

The npm backport release: https://www.npmjs.com/package/@elastic/eui/v/97.3.1-backport.0

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.17 are:

1. Go to Security > Alerts.
2. In the data grid, set the view from "Grid view" to "Event rendered view".
3. Choose the settings icon, in the picker choose "Auto fit" in Row height.

The data grid's height changes to 0. It's reproducible on first load.

[weronikaolejniczak]

@elasticmachine merge upstream

[github-actions]

A documentation preview will be available soon.

• 🔨 Buildkite builds
• 📚 HTML diff
• 📙 Preview page

Request a new doc build by commenting

• Rebuild this PR: run docs-build
• Rebuild this PR and all Elastic docs: run docs-build rebuild

_{run docs-build is much faster than run docs-build rebuild. A rebuild should only be needed in rare situations.}

_{If your PR continues to fail for an unknown reason, the doc build pipeline may be broken. Elastic employees can check the pipeline status here.}

[weronikaolejniczak]

All the above-mentioned reviewers were added automatically ☝🏻 but it's more than expected, we only need @elastic/kibana-operations approve.

@logeekal @kqualters-elastic I'd appreciate your QA, mates 🙏🏻 it seems you are the most well-informed regarding this issue. If I can help in any way, please let me know!

[logeekal]

@logeekal @kqualters-elastic I'd appreciate your QA, mates 🙏🏻 it seems you are the most well-informed regarding this issue. If I can help in any way, please let me know!

Thanks @weronikaolejniczak for the tag. I will desk test it later today and reply.

[kqualters-elastic]

possible issue with the fix this is backporting, blocking this until we talk through it

[weronikaolejniczak]

@kqualters-elastic I will update the PR with the suggestion from @jbudz just in case we proceed with the release.

[logeekal]

So I did some Desk Testing and it looks like error still exists. 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.

See below demo.

Screen.Recording.2025-02-20.at.11.03.59.mov

[elasticmachine]

💚 Build Succeeded

• Buildkite Build
• Commit: 4be69f0

Metrics [docs]

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id	before	after	diff
kbnUiSharedDeps-npmDll	5.8MB	5.8MB	+12.0B

History

• 💛 Build #277348 was flaky bcd7703

[weronikaolejniczak]

Closing the backport PRs due to the above-mentioned issues. Going back to the drawing board with this issue.