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

[vi-mode] Supports _ (void) and + (system clipboard) named registers for copying and pasting. #4220

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

springcomp
Copy link
Contributor

@springcomp springcomp commented Oct 12, 2024

PR Summary

This PR builds upon the existing support for Vi Registers and now supports two new named registers:

  • Vi Register _ (underscore) supports copying text into the "black hole" register. That text is lost forever and is not copied to the internal builtin clipboard.
  • Vi Register + (plus sign) or " (double-quote) supports copying text to, and pasting text from the operating system host’s system clipboard. Copied text does not end up in the internal builtin clipboard which stays unaffected.

This PR is a follow up on design discussions that happened in #3769.

PR Checklist

  • PR has a meaningful title
    • Use the present tense and imperative mood when describing your changes
  • Summarized changes
  • Make sure you've added one or more new tests
  • Make sure you've tested these changes in terminals that PowerShell is commonly used in (i.e. conhost.exe, Windows Terminal, Visual Studio Code Integrated Terminal, etc.)
  • User-facing changes
    • Not Applicable
    • OR
    • Documentation needed at PowerShell-Docs
      • Doc Issue filed:
Microsoft Reviewers: Open in CodeFlow

@springcomp springcomp changed the title [vi-mode] Supports _ and + named registers. [vi-mode] Supports _ (void) and + (system clipboard) named registers for copying and pasting. Oct 12, 2024
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.

1 participant