Skip to content

[Security Solution] Fix cell actions in Explore pages#177478

Merged
semd merged 1 commit intoelastic:mainfrom
semd:177410_fix_explore_cell_actions
Feb 21, 2024
Merged

[Security Solution] Fix cell actions in Explore pages#177478
semd merged 1 commit intoelastic:mainfrom
semd:177410_fix_explore_cell_actions

Conversation

@semd
Copy link
Copy Markdown
Contributor

@semd semd commented Feb 21, 2024

Summary

issue: #177410

Adding overflow: hidden style was causing the cell actions to disappear after the upgrade to EUI upgrade to v91.3.1

The overflow: hidden style is no longer needed, it has been removed. So cell actions are displayed

@semd semd added bug Fixes for quality problems that affect the customer experience release_note:skip Skip the PR/issue when compiling release notes Team:Threat Hunting Security Solution Threat Hunting Team Team:Threat Hunting:Explore v8.13.0 labels Feb 21, 2024
@semd semd self-assigned this Feb 21, 2024
@semd semd requested a review from a team as a code owner February 21, 2024 16:30
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/security-threat-hunting-explore (Team:Threat Hunting:Explore)

@elasticmachine
Copy link
Copy Markdown
Contributor

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

@semd semd enabled auto-merge (squash) February 21, 2024 16:48
@semd semd merged commit ab864b4 into elastic:main Feb 21, 2024
@kibana-ci
Copy link
Copy Markdown

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] Serverless Detection Engine - Security Solution Cypress Tests #3 / Sourcerer correctly loads SIEM data view correctly loads SIEM data view

Metrics [docs]

Public APIs missing comments

Total count of every public API that lacks a comment. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats comments for more detailed information.

id before after diff
@kbn/securitysolution-data-table 72 71 -1

Async chunks

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

id before after diff
securitySolution 11.6MB 11.6MB -114.0B
Unknown metric groups

API count

id before after diff
@kbn/securitysolution-data-table 95 94 -1

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

cc @semd

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Feb 21, 2024
## Summary

issue: elastic#177410

Adding `overflow: hidden` style was causing the cell actions to
disappear after the upgrade to [EUI upgrade to
v91.3.1](elastic#173569)

The `overflow: hidden` style is no longer needed, it has been removed.
So cell actions are displayed

(cherry picked from commit ab864b4)
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 All backports created successfully

Status Branch Result
8.13

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 Feb 21, 2024
#177501)

# Backport

This will backport the following commits from `main` to `8.13`:
- [[Security Solution] Fix cell actions in Explore pages
(#177478)](#177478)

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

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

<!--BACKPORT [{"author":{"name":"Sergi
Massaneda","email":"sergi.massaneda@elastic.co"},"sourceCommit":{"committedDate":"2024-02-21T17:52:45Z","message":"[Security
Solution] Fix cell actions in Explore pages (#177478)\n\n##
Summary\r\n\r\nissue:
https://github.com/elastic/kibana/issues/177410\r\n\r\nAdding `overflow:
hidden` style was causing the cell actions to\r\ndisappear after the
upgrade to [EUI upgrade
to\r\nv91.3.1](https://github.com/elastic/kibana/pull/173569)\r\n\r\nThe
`overflow: hidden` style is no longer needed, it has been removed.\r\nSo
cell actions are
displayed","sha":"ab864b4ccc699cc1a986bf099460ad877a42333d","branchLabelMapping":{"^v8.14.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["bug","release_note:skip","Team:Threat
Hunting","Team:Threat
Hunting:Explore","v8.13.0","v8.14.0"],"title":"[Security Solution] Fix
cell actions in Explore
pages","number":177478,"url":"https://github.com/elastic/kibana/pull/177478","mergeCommit":{"message":"[Security
Solution] Fix cell actions in Explore pages (#177478)\n\n##
Summary\r\n\r\nissue:
https://github.com/elastic/kibana/issues/177410\r\n\r\nAdding `overflow:
hidden` style was causing the cell actions to\r\ndisappear after the
upgrade to [EUI upgrade
to\r\nv91.3.1](https://github.com/elastic/kibana/pull/173569)\r\n\r\nThe
`overflow: hidden` style is no longer needed, it has been removed.\r\nSo
cell actions are
displayed","sha":"ab864b4ccc699cc1a986bf099460ad877a42333d"}},"sourceBranch":"main","suggestedTargetBranches":["8.13"],"targetPullRequestStates":[{"branch":"8.13","label":"v8.13.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v8.14.0","branchLabelMappingKey":"^v8.14.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/177478","number":177478,"mergeCommit":{"message":"[Security
Solution] Fix cell actions in Explore pages (#177478)\n\n##
Summary\r\n\r\nissue:
https://github.com/elastic/kibana/issues/177410\r\n\r\nAdding `overflow:
hidden` style was causing the cell actions to\r\ndisappear after the
upgrade to [EUI upgrade
to\r\nv91.3.1](https://github.com/elastic/kibana/pull/173569)\r\n\r\nThe
`overflow: hidden` style is no longer needed, it has been removed.\r\nSo
cell actions are
displayed","sha":"ab864b4ccc699cc1a986bf099460ad877a42333d"}}]}]
BACKPORT-->

Co-authored-by: Sergi Massaneda <sergi.massaneda@elastic.co>
fkanout pushed a commit to fkanout/kibana that referenced this pull request Mar 4, 2024
## Summary

issue: elastic#177410

Adding `overflow: hidden` style was causing the cell actions to
disappear after the upgrade to [EUI upgrade to
v91.3.1](elastic#173569)

The `overflow: hidden` style is no longer needed, it has been removed.
So cell actions are displayed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Fixes for quality problems that affect the customer experience release_note:skip Skip the PR/issue when compiling release notes Team:Threat Hunting Security Solution Threat Hunting Team v8.13.0 v8.14.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants