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

[SecuritySolution] Fix styling issues for visualizations #198410

Merged
merged 3 commits into from
Nov 1, 2024

Conversation

angorayc
Copy link
Contributor

@angorayc angorayc commented Oct 30, 2024

Summary

Fixes: #198370

Before - The was an unexpected outline for each visualization.

Screenshot 2024-10-30 at 14 50 08

After - Overriding the style to avoid unexpected styling changed.

Screenshot 2024-10-30 at 17 28 38
Screen.Recording.2024-10-30.at.18.26.32.mov

@angorayc angorayc changed the title Remove border for visualizations [SecuritySolution] Fix styling issues for visualizations Oct 30, 2024
@angorayc angorayc marked this pull request as ready for review October 30, 2024 18:43
@angorayc angorayc requested a review from a team as a code owner October 30, 2024 18:43
@elasticmachine
Copy link
Contributor

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

@angorayc angorayc requested a review from machadoum October 30, 2024 18:44
@angorayc
Copy link
Contributor Author

/ci

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

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 21.0MB 21.0MB -255.0B

History

@angorayc angorayc removed the v8.16.0 label Oct 31, 2024
@angorayc angorayc merged commit 494a1df into elastic:main Nov 1, 2024
45 of 46 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.x

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

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Nov 1, 2024
)

## Summary

Fixes: elastic#198370

**Before** - The was an unexpected outline for each visualization.

![Screenshot 2024-10-30 at 14 50
08](https://github.com/user-attachments/assets/84565278-ef4c-438c-8ae6-138588aca3d7)

**After** - Overriding the style to avoid unexpected styling changed.

<img width="2555" alt="Screenshot 2024-10-30 at 17 28 38"
src="https://github.com/user-attachments/assets/3116a798-c859-4543-a732-ebf1ff556c68">

https://github.com/user-attachments/assets/b19ec64e-1fa0-475a-9c14-9c0e237b8f95
(cherry picked from commit 494a1df)
@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 1, 2024
…) (#198704)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[SecuritySolution] Fix styling issues for visualizations
(#198410)](#198410)

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

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

<!--BACKPORT [{"author":{"name":"Angela
Chuang","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-11-01T16:45:16Z","message":"[SecuritySolution]
Fix styling issues for visualizations (#198410)\n\n##
Summary\r\n\r\nFixes:
https://github.com/elastic/kibana/issues/198370\r\n\r\n**Before** - The
was an unexpected outline for each visualization.\r\n\r\n![Screenshot
2024-10-30 at 14
50\r\n08](https://github.com/user-attachments/assets/84565278-ef4c-438c-8ae6-138588aca3d7)\r\n\r\n**After**
- Overriding the style to avoid unexpected styling changed.\r\n\r\n<img
width=\"2555\" alt=\"Screenshot 2024-10-30 at 17 28
38\"\r\nsrc=\"https://github.com/user-attachments/assets/3116a798-c859-4543-a732-ebf1ff556c68\">\r\n\r\n\r\n\r\n\r\nhttps://github.com/user-attachments/assets/b19ec64e-1fa0-475a-9c14-9c0e237b8f95","sha":"494a1dfed36ea7cb7b050e833ecfacb6bda21a85","branchLabelMapping":{"^v9.0.0$":"main","^v8.17.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["backport","v9.0.0","Team:Threat
Hunting:Explore","v8.17.0"],"title":"[SecuritySolution] Fix styling
issues for
visualizations","number":198410,"url":"https://github.com/elastic/kibana/pull/198410","mergeCommit":{"message":"[SecuritySolution]
Fix styling issues for visualizations (#198410)\n\n##
Summary\r\n\r\nFixes:
https://github.com/elastic/kibana/issues/198370\r\n\r\n**Before** - The
was an unexpected outline for each visualization.\r\n\r\n![Screenshot
2024-10-30 at 14
50\r\n08](https://github.com/user-attachments/assets/84565278-ef4c-438c-8ae6-138588aca3d7)\r\n\r\n**After**
- Overriding the style to avoid unexpected styling changed.\r\n\r\n<img
width=\"2555\" alt=\"Screenshot 2024-10-30 at 17 28
38\"\r\nsrc=\"https://github.com/user-attachments/assets/3116a798-c859-4543-a732-ebf1ff556c68\">\r\n\r\n\r\n\r\n\r\nhttps://github.com/user-attachments/assets/b19ec64e-1fa0-475a-9c14-9c0e237b8f95","sha":"494a1dfed36ea7cb7b050e833ecfacb6bda21a85"}},"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/198410","number":198410,"mergeCommit":{"message":"[SecuritySolution]
Fix styling issues for visualizations (#198410)\n\n##
Summary\r\n\r\nFixes:
https://github.com/elastic/kibana/issues/198370\r\n\r\n**Before** - The
was an unexpected outline for each visualization.\r\n\r\n![Screenshot
2024-10-30 at 14
50\r\n08](https://github.com/user-attachments/assets/84565278-ef4c-438c-8ae6-138588aca3d7)\r\n\r\n**After**
- Overriding the style to avoid unexpected styling changed.\r\n\r\n<img
width=\"2555\" alt=\"Screenshot 2024-10-30 at 17 28
38\"\r\nsrc=\"https://github.com/user-attachments/assets/3116a798-c859-4543-a732-ebf1ff556c68\">\r\n\r\n\r\n\r\n\r\nhttps://github.com/user-attachments/assets/b19ec64e-1fa0-475a-9c14-9c0e237b8f95","sha":"494a1dfed36ea7cb7b050e833ecfacb6bda21a85"}},{"branch":"8.x","label":"v8.17.0","branchLabelMappingKey":"^v8.17.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

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

Successfully merging this pull request may close these issues.

[SecuritySolution] Unexpected outline around visualizations
4 participants