Skip to content

add closeOnResult for editor's find widget#299865

Merged
meganrogge merged 2 commits intomainfrom
merogge/find-close
Mar 6, 2026
Merged

add closeOnResult for editor's find widget#299865
meganrogge merged 2 commits intomainfrom
merogge/find-close

Conversation

@meganrogge
Copy link
Collaborator

fixes #264818

Screenshot 2026-03-06 at 3 21 47 PM

Copilot AI review requested due to automatic review settings March 6, 2026 20:22
@meganrogge meganrogge self-assigned this Mar 6, 2026
@meganrogge meganrogge added this to the 1.112.0 milestone Mar 6, 2026
@meganrogge meganrogge enabled auto-merge (squash) March 6, 2026 20:23
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a new editor find setting to automatically close the Find widget after an explicit navigation command successfully lands on a match, aligning Ctrl+F → Enter workflows more closely with “jump to match then keep editing”.

Changes:

  • Introduces editor.find.closeOnResult to editor find options (config schema + Monaco typings).
  • Implements close-on-result behavior in matchFindAction for next/previous match navigation.
  • Updates accessibility help and adds unit tests covering the new setting behavior.

Reviewed changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
src/vs/workbench/contrib/codeEditor/browser/editorFindAccessibilityHelp.ts Documents the new editor.find.closeOnResult setting in Find accessibility help.
src/vs/monaco.d.ts Adds closeOnResult to monaco.editor.IEditorFindOptions typings.
src/vs/editor/contrib/find/test/browser/findController.test.ts Adds tests validating close behavior on match / no match / setting disabled.
src/vs/editor/contrib/find/browser/findController.ts Implements the close-on-result behavior during next/previous match actions.
src/vs/editor/common/config/editorOptions.ts Adds editor.find.closeOnResult setting with default + validation + schema description.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@meganrogge meganrogge merged commit 22933ce into main Mar 6, 2026
20 checks passed
@meganrogge meganrogge deleted the merogge/find-close branch March 6, 2026 21:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Feature Request: Option to auto Close Find Dialog After Search

3 participants