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

[ResponseOps][Rules] Rule type selection modal content cut off on small screens due to lack of vertical scrolling #195234

Conversation

georgianaonoleata1904
Copy link
Contributor

@georgianaonoleata1904 georgianaonoleata1904 commented Oct 7, 2024

Summary

Closes #184048

  • the rule type selection modal is now scrollable on smaller windows
Screen.Recording.2024-10-07.at.15.06.20.mov

@georgianaonoleata1904 georgianaonoleata1904 self-assigned this Oct 7, 2024
@georgianaonoleata1904 georgianaonoleata1904 added release_note:skip Skip the PR/issue when compiling release notes Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) v9.0.0 backport:prev-minor Backport to (9.0) the previous minor version (i.e. one version back from main) v8.16.0 labels Oct 7, 2024
@georgianaonoleata1904 georgianaonoleata1904 marked this pull request as ready for review October 7, 2024 12:32
@georgianaonoleata1904 georgianaonoleata1904 requested a review from a team as a code owner October 7, 2024 12:32
@elasticmachine
Copy link
Contributor

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

@cnasikas cnasikas added the bug Fixes for quality problems that affect the customer experience label Oct 7, 2024
@adcoelho
Copy link
Contributor

adcoelho commented Oct 7, 2024

I merged main again to retrigger the CI, the checks step was failing with a weird error. I don't think there was anything wrong with your PR 🤔

Copy link
Contributor

@adcoelho adcoelho left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested and works as expected 👍

@kibana-ci
Copy link
Collaborator

💚 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
observability 467.6KB 467.7KB +42.0B
triggersActionsUi 1.6MB 1.6MB +42.0B
total +84.0B

History

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

cc @georgianaonoleata1904

@georgianaonoleata1904 georgianaonoleata1904 merged commit 6df6724 into elastic:main Oct 9, 2024
20 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.x

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

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Oct 9, 2024
…ll screens due to lack of vertical scrolling (elastic#195234)

## Summary

Closes elastic#184048

- the rule type selection modal is now scrollable on smaller windows

https://github.com/user-attachments/assets/47082b35-02a7-4b67-9a88-ee4200908bef

Co-authored-by: Antonio <[email protected]>
(cherry picked from commit 6df6724)
@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 Oct 9, 2024
…on small screens due to lack of vertical scrolling (#195234) (#195543)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[ResponseOps][Rules] Rule type selection modal content cut off on
small screens due to lack of vertical scrolling
(#195234)](#195234)

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

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

<!--BACKPORT [{"author":{"name":"Georgiana-Andreea
Onoleață","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-10-09T08:22:57Z","message":"[ResponseOps][Rules]
Rule type selection modal content cut off on small screens due to lack
of vertical scrolling (#195234)\n\n## Summary\r\n\r\nCloses
https://github.com/elastic/kibana/issues/184048\r\n\r\n- the rule type
selection modal is now scrollable on smaller windows
\r\n\r\n\r\nhttps://github.com/user-attachments/assets/47082b35-02a7-4b67-9a88-ee4200908bef\r\n\r\nCo-authored-by:
Antonio
<[email protected]>","sha":"6df672421d6df47cb3e8f86ab9448f021dabc871","branchLabelMapping":{"^v9.0.0$":"main","^v8.16.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["bug","release_note:skip","Team:ResponseOps","v9.0.0","backport:prev-minor","v8.16.0"],"title":"[ResponseOps][Rules]
Rule type selection modal content cut off on small screens due to lack
of vertical
scrolling","number":195234,"url":"https://github.com/elastic/kibana/pull/195234","mergeCommit":{"message":"[ResponseOps][Rules]
Rule type selection modal content cut off on small screens due to lack
of vertical scrolling (#195234)\n\n## Summary\r\n\r\nCloses
https://github.com/elastic/kibana/issues/184048\r\n\r\n- the rule type
selection modal is now scrollable on smaller windows
\r\n\r\n\r\nhttps://github.com/user-attachments/assets/47082b35-02a7-4b67-9a88-ee4200908bef\r\n\r\nCo-authored-by:
Antonio
<[email protected]>","sha":"6df672421d6df47cb3e8f86ab9448f021dabc871"}},"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/195234","number":195234,"mergeCommit":{"message":"[ResponseOps][Rules]
Rule type selection modal content cut off on small screens due to lack
of vertical scrolling (#195234)\n\n## Summary\r\n\r\nCloses
https://github.com/elastic/kibana/issues/184048\r\n\r\n- the rule type
selection modal is now scrollable on smaller windows
\r\n\r\n\r\nhttps://github.com/user-attachments/assets/47082b35-02a7-4b67-9a88-ee4200908bef\r\n\r\nCo-authored-by:
Antonio
<[email protected]>","sha":"6df672421d6df47cb3e8f86ab9448f021dabc871"}},{"branch":"8.x","label":"v8.16.0","branchLabelMappingKey":"^v8.16.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Georgiana-Andreea Onoleață <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:prev-minor Backport to (9.0) the previous minor version (i.e. one version back from main) bug Fixes for quality problems that affect the customer experience release_note:skip Skip the PR/issue when compiling release notes Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) v8.16.0 v9.0.0
Projects
None yet
7 participants