Skip to content

[8.14] [Bug][Investigations] - Fix eql tab scrolling (#181763)#181874

Merged
michaelolo24 merged 2 commits intoelastic:8.14from
michaelolo24:backport/8.14/pr-181763
Apr 26, 2024
Merged

[8.14] [Bug][Investigations] - Fix eql tab scrolling (#181763)#181874
michaelolo24 merged 2 commits intoelastic:8.14from
michaelolo24:backport/8.14/pr-181763

Conversation

@michaelolo24
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 8.14:

Questions ?

Please refer to the Backport tool documentation

## Summary

This issue only affects 8.14.

Some wrappers were added unnecessarily to the correlations tab in
Timeline preventing users from scrolling in this PR:
elastic#179832

This PR fixes it by placing the structure more in line with how it was
prior here:
https://github.com/elastic/kibana/pull/179832/files#diff-a11ba69be2253bc1c3183eb6589e3f30b9ec6e77353364208e906526712562fe

The fix:

https://github.com/elastic/kibana/assets/17211684/fc288ab8-3cbf-4f4d-8303-4fb5e970dfcd
(cherry picked from commit 11eb39b)

# Conflicts:
#	x-pack/plugins/security_solution/public/timelines/components/timeline/tabs/eql/header/index.tsx
#	x-pack/plugins/security_solution/public/timelines/components/timeline/tabs/eql/index.tsx
@michaelolo24 michaelolo24 added the backport This PR is a backport of another PR label Apr 26, 2024
@michaelolo24 michaelolo24 enabled auto-merge (squash) April 26, 2024 15:21
Copy link
Copy Markdown
Contributor

@PhilippeOberti PhilippeOberti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

desk tested and code LGTM!

@kibana-ci
Copy link
Copy Markdown

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] Jest Tests #5 / useSourceIndicesFields Hook should provide warning message for adding an index without any fields

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
securitySolution 17.3MB 17.3MB -105.0B

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants