Bugfix: ESC didn't clear selection (except CMD) (#647)

* Bugfix: ESC didn't clear selection (except CMD)

* Bugfix: ESC didn't clear selection - moved TriggerSelection() to ClearSelection()
This commit is contained in:
Carlos Zamora 2019-05-10 11:16:59 -07:00 committed by GitHub
parent 99555ef9e9
commit 644cd3ec6c
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 2 additions and 1 deletions

View file

@ -960,7 +960,6 @@ namespace winrt::Microsoft::Terminal::TerminalControl::implementation
const auto copiedData = _terminal->RetrieveSelectedTextFromBuffer(trimTrailingWhitespace);
_terminal->ClearSelection();
_renderer->TriggerSelection();
// send data up for clipboard
_clipboardCopyHandlers(copiedData);

View file

@ -560,6 +560,8 @@ void Terminal::ClearSelection() noexcept
_endSelectionPosition = {0, 0};
_selectionAnchor_YOffset = 0;
_endSelectionPosition_YOffset = 0;
_buffer->GetRenderTarget().TriggerSelection();
}
// Method Description: