Skip to content

[9.2] [Ingest Pipelines] Fix add/edit processor flyout return focus behavior (#239720)#240030

Merged
kibanamachine merged 2 commits intoelastic:9.2from
kibanamachine:backport/9.2/pr-239720
Oct 22, 2025
Merged

[9.2] [Ingest Pipelines] Fix add/edit processor flyout return focus behavior (#239720)#240030
kibanamachine merged 2 commits intoelastic:9.2from
kibanamachine:backport/9.2/pr-239720

Conversation

@kibanamachine
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 9.2:

Questions ?

Please refer to the Backport tool documentation

elastic#239720)

## Summary

Closes elastic#217956, closes elastic#218207

This PR fixes an issue with add/edit processor flyouts not returning
focus to the correct element after closure.

Changes:

- Added a optional `buttonRef` property to the `EditorMode` that allows
to pass the trigger element reference using `onTreeAction` or
`editor.setMode`.
- Added `returnFocus` prop to `AddProcessorForm` and `EditProcessorForm`
to allow focusing of the trigger element after flyout closure.

(cherry picked from commit 6526f44)
@damian-polewski
Copy link
Copy Markdown
Contributor

@elasticmachine merge upstream

@elasticmachine
Copy link
Copy Markdown
Contributor

There are no new commits on the base branch.

@damian-polewski
Copy link
Copy Markdown
Contributor

/ci

@damian-polewski
Copy link
Copy Markdown
Contributor

@elasticmachine merge upstream

@kibanamachine kibanamachine merged commit c9081e1 into elastic:9.2 Oct 22, 2025
12 checks passed
@elasticmachine
Copy link
Copy Markdown
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
ingestPipelines 346.1KB 346.8KB +655.0B

History

cc @damian-polewski

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