-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[Console] Fix actions menu scrolling issue #200018
[Console] Fix actions menu scrolling issue #200018
Conversation
Pinging @elastic/kibana-management (Team:Kibana Management) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would be nice to see a unit test for the code that determines the line number where the action buttons are placed BUT I leave it up to you to decide if you want to this. Otherwise the code look good and works well.
/ci |
@elasticmachine merge upstream |
@elasticmachine merge upstream |
Starting backport for target branches: 8.x |
💚 Build Succeeded
Metrics [docs]Async chunks
History
cc @sabarasaba |
(cherry picked from commit 0ed8230)
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
# Backport This will backport the following commits from `main` to `8.x`: - [[Console] Fix actions menu scrolling issue (#200018)](#200018) <!--- Backport version: 9.4.3 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Ignacio Rivas","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-11-17T11:44:07Z","message":"[Console] Fix actions menu scrolling issue (#200018)","sha":"0ed82309e22a7fc8ac067c1dbf2c378735f86855","branchLabelMapping":{"^v9.0.0$":"main","^v8.17.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Feature:Console","Team:Kibana Management","release_note:skip","v9.0.0","backport:prev-minor"],"title":"[Console] Fix actions menu scrolling issue","number":200018,"url":"https://github.com/elastic/kibana/pull/200018","mergeCommit":{"message":"[Console] Fix actions menu scrolling issue (#200018)","sha":"0ed82309e22a7fc8ac067c1dbf2c378735f86855"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/200018","number":200018,"mergeCommit":{"message":"[Console] Fix actions menu scrolling issue (#200018)","sha":"0ed82309e22a7fc8ac067c1dbf2c378735f86855"}}]}] BACKPORT--> Co-authored-by: Ignacio Rivas <[email protected]>
Fixes: #197965
Summary
Make sure that the offset never goes into negative so that if a request is long enough, the actions menu remains anchored at the top of the editor.