terminal icon indicating copy to clipboard operation
terminal copied to clipboard

Skip whitespace removal for multiline pastes

Open serd2011 opened this issue 2 years ago • 2 comments

Summary of the Pull Request

Skips whitespace removal if pasted string is multiline

PR Checklist

  • [x] Closes #12387
  • [x] CLA signed. If not, go over here and sign the CLA
  • [x] Tests added/passed
  • [ ] Documentation updated. If checked, please file a pull request on our docs repo and link it here: #xxx
  • [ ] Schema updated.
  • [ ] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Validation Steps Performed

Tests passed The last command in multiline paste now executes

serd2011 avatar Aug 07 '22 03:08 serd2011

I don't really see why tabs should be treated differently. Am I missing something?

serd2011 avatar Aug 07 '22 03:08 serd2011

(I've fixed a small grammatical error in the PR title.)

lhecker avatar Aug 08 '22 14:08 lhecker

Hello @zadjii-msft!

Because this pull request has the AutoMerge label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (@msftbot) and give me an instruction to get started! Learn more here.

ghost avatar Aug 17 '22 15:08 ghost

:tada:Windows Terminal Preview v1.16.252 has been released which incorporates this pull request.:tada:

Handy links:

ghost avatar Sep 13 '22 17:09 ghost