Skip to content

Put terminal voice actions in separate group#269596

Merged
meganrogge merged 3 commits intomainfrom
voiceless-crawdad
Oct 2, 2025
Merged

Put terminal voice actions in separate group#269596
meganrogge merged 3 commits intomainfrom
voiceless-crawdad

Conversation

@meganrogge
Copy link
Copy Markdown
Collaborator

fix #269587

Screenshot 2025-10-02 at 3 03 01 PM

Copilot AI review requested due to automatic review settings October 2, 2025 19:07
@meganrogge meganrogge self-assigned this Oct 2, 2025
@meganrogge meganrogge added this to the September 2025 milestone Oct 2, 2025
@meganrogge meganrogge enabled auto-merge (squash) October 2, 2025 19:07
Copy link
Copy Markdown
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

Separates terminal voice (dictation) actions into their own menu group rather than mixing them with general navigation actions in the terminal view title. Voice start/stop actions are now registered alongside their action definitions instead of globally in terminalMenus.

  • Moved Start/Stop Dictation menu contributions from terminalMenus.ts into terminalVoiceActions.ts
  • Introduced new menu group 'voice' for terminal dictation actions
  • Removed old menu entries from the generic terminal menus

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
src/vs/workbench/contrib/terminalContrib/voice/browser/terminalVoiceActions.ts Adds menu contributions for start/stop dictation actions in a new 'voice' group
src/vs/workbench/contrib/terminal/browser/terminalMenus.ts Removes previous dictation menu items from the generic terminal view title menu

…lVoiceActions.ts

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Comment thread src/vs/workbench/contrib/terminalContrib/voice/browser/terminalVoiceActions.ts Outdated
@meganrogge meganrogge merged commit 77a9703 into main Oct 2, 2025
28 checks passed
@meganrogge meganrogge deleted the voiceless-crawdad branch October 2, 2025 20:11
meganrogge added a commit that referenced this pull request Oct 6, 2025
meganrogge added a commit that referenced this pull request Oct 6, 2025
@vs-code-engineering vs-code-engineering Bot locked and limited conversation to collaborators Nov 16, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Terminal menu would benefit from separators

3 participants