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][Notes] - fix column width on events table on host, user and network pages #193460

Merged
merged 1 commit into from
Sep 20, 2024

Conversation

PhilippeOberti
Copy link
Contributor

@PhilippeOberti PhilippeOberti commented Sep 19, 2024

Summary

This PR fixes a small bug on the events table on the Explore host, user and network pages.
As can be seen in the screenshot below, the actions column width is not correct when the securitySolutionNotesEnabled feature flag is enabled in main (we're missing the session view icon)
Screenshot 2024-09-19 at 12 39 54 PM

This branch increases the width to make room for the notes button when the securitySolutionNotesEnabled feature flag is enabled
Screenshot 2024-09-19 at 12 37 08 PM

And this is the normal width when the feature flag is not enabled
Screenshot 2024-09-19 at 12 38 05 PM

#193088

@elasticmachine
Copy link
Contributor

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

@kibana-ci
Copy link
Collaborator

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] Jest Tests #2 / Configuration button renders the tooltip correctly when hovering the button

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 20.4MB 20.4MB +60.0B

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

@PhilippeOberti PhilippeOberti merged commit 2a98d1a into elastic:main Sep 20, 2024
47 checks passed
@PhilippeOberti PhilippeOberti deleted the fix-notes-events-table branch September 20, 2024 13:38
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Sep 20, 2024
@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 Sep 20, 2024
…n host, user and network pages (#193460) (#193580)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[Security Solution][Notes] - fix column width on events table on
host, user and network pages
(#193460)](#193460)

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

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

<!--BACKPORT [{"author":{"name":"Philippe
Oberti","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-09-20T13:38:05Z","message":"[Security
Solution][Notes] - fix column width on events table on host, user and
network pages
(#193460)","sha":"2a98d1af37ca8aca570b3c44728087b6271c3444","branchLabelMapping":{"^v9.0.0$":"main","^v8.16.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["backport","v9.0.0","Team:Threat
Hunting:Investigations","v8.16.0"],"title":"[Security Solution][Notes] -
fix column width on events table on host, user and network
pages","number":193460,"url":"https://github.com/elastic/kibana/pull/193460","mergeCommit":{"message":"[Security
Solution][Notes] - fix column width on events table on host, user and
network pages
(#193460)","sha":"2a98d1af37ca8aca570b3c44728087b6271c3444"}},"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/193460","number":193460,"mergeCommit":{"message":"[Security
Solution][Notes] - fix column width on events table on host, user and
network pages
(#193460)","sha":"2a98d1af37ca8aca570b3c44728087b6271c3444"}},{"branch":"8.x","label":"v8.16.0","branchLabelMappingKey":"^v8.16.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Philippe Oberti <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants