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

[Security Solution] Fix Auto-grow behaviour of Filter Page Controls #200152

Merged

Conversation

logeekal
Copy link
Contributor

@logeekal logeekal commented Nov 14, 2024

Summary

PR : #199408 disables auto grow behaviour page filters controls which has been default behavior for some time.

It causes issue on the page filters of Security and probably Observability as well since Page Filter controls do not grow anymore.

Correct in 8.16 Broken in 8.17/9.0.0
image image

@logeekal logeekal added backport Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) Team:Threat Hunting:Investigations Security Solution Investigations Team v8.17.0 labels Nov 14, 2024
@logeekal logeekal requested a review from a team as a code owner November 14, 2024 11:22
@elasticmachine
Copy link
Contributor

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

@elasticmachine
Copy link
Contributor

Pinging @elastic/security-threat-hunting-investigations (Team:Threat Hunting:Investigations)

@logeekal logeekal enabled auto-merge (squash) November 14, 2024 11:26
@logeekal logeekal merged commit a7470dd into elastic:main Nov 14, 2024
31 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.x

https://github.com/elastic/kibana/actions/runs/11837862841

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Nov 14, 2024
…lastic#200152)

## Summary

PR : elastic#199408 disables `auto` grow
behaviour page filters controls which has been default behavior for some
time.

It causes issue on the page filters of Security and probably
Observability as well since Page Filter controls do not grow anymore.

|Correct in 8.16| Broken in 8.17/9.0.0|
|---|---|

|![image](https://github.com/user-attachments/assets/cc120c75-971c-44e6-bd63-65e5193cc0e5)|![image](https://github.com/user-attachments/assets/9a83a75d-70e2-4543-b9db-d0cbee24a404)|

(cherry picked from commit a7470dd)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.x

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Nov 14, 2024
…rols (#200152) (#200173)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[Security Solution] Fix Auto-grow behaviour of Filter Page Controls
(#200152)](#200152)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Jatin
Kathuria","email":"jatin.kathuria@elastic.co"},"sourceCommit":{"committedDate":"2024-11-14T13:09:28Z","message":"[Security
Solution] Fix Auto-grow behaviour of Filter Page Controls
(#200152)\n\n## Summary\r\n\r\nPR :
#199408 disables `auto`
grow\r\nbehaviour page filters controls which has been default behavior
for some\r\ntime.\r\n\r\nIt causes issue on the page filters of Security
and probably\r\nObservability as well since Page Filter controls do not
grow anymore.\r\n\r\n|Correct in 8.16| Broken in
8.17/9.0.0|\r\n|---|---|\r\n\r\n|![image](https://github.com/user-attachments/assets/cc120c75-971c-44e6-bd63-65e5193cc0e5)|![image](https://github.com/user-attachments/assets/9a83a75d-70e2-4543-b9db-d0cbee24a404)|","sha":"a7470dd0473259418b1af748c94cc6f51dbcd41f","branchLabelMapping":{"^v9.0.0$":"main","^v8.17.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["backport","Team:ResponseOps","v9.0.0","Team:Threat
Hunting:Investigations","v8.17.0"],"title":"[Security Solution] Fix
Auto-grow behaviour of Filter Page
Controls","number":200152,"url":"https://github.com/elastic/kibana/pull/200152","mergeCommit":{"message":"[Security
Solution] Fix Auto-grow behaviour of Filter Page Controls
(#200152)\n\n## Summary\r\n\r\nPR :
#199408 disables `auto`
grow\r\nbehaviour page filters controls which has been default behavior
for some\r\ntime.\r\n\r\nIt causes issue on the page filters of Security
and probably\r\nObservability as well since Page Filter controls do not
grow anymore.\r\n\r\n|Correct in 8.16| Broken in
8.17/9.0.0|\r\n|---|---|\r\n\r\n|![image](https://github.com/user-attachments/assets/cc120c75-971c-44e6-bd63-65e5193cc0e5)|![image](https://github.com/user-attachments/assets/9a83a75d-70e2-4543-b9db-d0cbee24a404)|","sha":"a7470dd0473259418b1af748c94cc6f51dbcd41f"}},"sourceBranch":"main","suggestedTargetBranches":["8.x"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/200152","number":200152,"mergeCommit":{"message":"[Security
Solution] Fix Auto-grow behaviour of Filter Page Controls
(#200152)\n\n## Summary\r\n\r\nPR :
#199408 disables `auto`
grow\r\nbehaviour page filters controls which has been default behavior
for some\r\ntime.\r\n\r\nIt causes issue on the page filters of Security
and probably\r\nObservability as well since Page Filter controls do not
grow anymore.\r\n\r\n|Correct in 8.16| Broken in
8.17/9.0.0|\r\n|---|---|\r\n\r\n|![image](https://github.com/user-attachments/assets/cc120c75-971c-44e6-bd63-65e5193cc0e5)|![image](https://github.com/user-attachments/assets/9a83a75d-70e2-4543-b9db-d0cbee24a404)|","sha":"a7470dd0473259418b1af748c94cc6f51dbcd41f"}},{"branch":"8.x","label":"v8.17.0","branchLabelMappingKey":"^v8.17.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Jatin Kathuria <jatin.kathuria@elastic.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) Team:Threat Hunting:Investigations Security Solution Investigations Team v8.17.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants