Skip to content

[9.3] [ResponseOps][Rules] Fix alert detail flyout (#247055)#247092

Merged
kibanamachine merged 1 commit intoelastic:9.3from
kibanamachine:backport/9.3/pr-247055
Dec 19, 2025
Merged

[9.3] [ResponseOps][Rules] Fix alert detail flyout (#247055)#247092
kibanamachine merged 1 commit intoelastic:9.3from
kibanamachine:backport/9.3/pr-247055

Conversation

@kibanamachine
Copy link
Contributor

Backport

This will backport the following commits from main to 9.3:

Questions ?

Please refer to the Backport tool documentation

## Summary

Closes elastic#247030

<details>
  <summary>Screenshot after fix</summary>
<img width="1261" height="1115" alt="Screenshot 2025-12-19 at 12 26 27"
src="https://github.com/user-attachments/assets/59d22e6a-69c1-47ac-b7e6-5e1b7441d791"
/>

</details>

The `reason` field can be a very long text with long unbreakable
segments in it (URLs, comma separated values). I added a wrapping class
that takes care of that via css, it effectively adds:

```
word-break: break-word;
overflow-wrap: break-word;
```

These CSS properties have zero visible effect on content that doesn't
need to break - short text, numbers, badges, links all render exactly
the same. The CSS only kicks in when there's actually a long unbreakable
string.
The overhead of one extra `<div>` per row is negligible.

Example of broken `reason` field content:
```
Document count is 1 in the last 5m for 2025-12-19T10:32:24.229Z,Mozilla/4.0 (compatible; MSIE 6.0; Windows NT 5.1; SV1; .NET CLR 1.1.4322),Mozilla/4.0 (compatible; MSIE 6.0; Windows NT 5.1; SV1; .NET CLR 1.1.4322),680,16099067,680,78.218.15.245,sample_web_logs,,,POINT (-82.52886139 42.91095778),US,US,US:US,www.elastic.co,www.elastic.co,kibana_sample_data_logs,kibana_sample_data_logs,78.218.15.245,win 7,win 7,18253611008,78.218.15.245 - - [2018-08-03T10:32:24.229Z] "GET /site-search HTTP/1.1" 200 680 "-" "Mozilla/4.0 (compatible; MSIE 6.0; Windows NT 5.1; SV1; .NET CLR 1.1.4322)",78.218.15.245 - - [2018-08-03T10:32:24.229Z] "GET /site-search HTTP/1.1" 200 680 "-" "Mozilla/4.0 (compatible; MSIE 6.0; Windows NT 5.1; SV1; .NET CLR 1.1.4322)",http://twitter.com/success/nikolai-budarin,/site-search,/site-search,200,200,success,info,info,success,2025-12-19T10:32:24.229Z,https://www.elastic.co/solutions/site-search,https://www.elastic.co/solutions/site-search,2025-12-19T10:32:24.229Z. Alert when greater than 0.
```

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
(cherry picked from commit 2b31b97)
@kibanamachine kibanamachine requested a review from a team as a code owner December 19, 2025 13:44
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Dec 19, 2025
@kibanamachine kibanamachine enabled auto-merge (squash) December 19, 2025 13:44
@kibanamachine kibanamachine merged commit 5ab9ee7 into elastic:9.3 Dec 19, 2025
17 checks passed
@elasticmachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

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
cases 1.4MB 1.4MB +49.0B
embeddableAlertsTable 1017.0KB 1017.1KB +49.0B
ml 5.6MB 5.6MB +49.0B
observability 1.7MB 1.7MB +49.0B
securitySolution 10.8MB 10.8MB +49.0B
triggersActionsUi 1.5MB 1.5MB +49.0B
total +294.0B

cc @jcger

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