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

[v15] Improve copy/paste behavior in Connect terminal #45266

Merged
merged 1 commit into from
Aug 8, 2024

Conversation

gzdunek
Copy link
Contributor

@gzdunek gzdunek commented Aug 8, 2024

Backport #44871 to branch/v15

changelog: Improved copy and paste behavior in the terminal in Teleport Connect. On Windows and Linux, Ctrl+Shift+C/V now copies and pastes text (these shortcuts can be changed with keymap.terminalCopy/keymap.terminalPaste). A mouse right click (terminal.rightClick) can copy/paste text too (enabled by default on Windows).

* Allow reading clipboard content in renderer process

* Make keyboard shortcuts to copy/paste text in terminal configurable, set reasonable defaults for Windows and Linux

* Allow copying text in terminal with selection and a right click

* Remove check for `selection.trim()`, check if there's something selected in callsites, rename `copy` to `copySelection`

* Show copy/paste keyboard shortcuts in the context menu

* Remove redundant `to`

* Add `Copy in Terminal` and `Paste in Terminal` shortcuts to `KeyboardShortcutsPanel`

* Set `rightClickSelectsWord: true` only if `terminal.rightClick` is `menu`

* Add docs

* Patched jsdom `global.matchMedia` should return `matches: false`, not `true`

* Small docs correction

(cherry picked from commit ee1a5ba)
Copy link

github-actions bot commented Aug 8, 2024

🤖 Vercel preview here: https://docs-lur4s7pjj-goteleport.vercel.app/docs/ver/preview

@gzdunek gzdunek added this pull request to the merge queue Aug 8, 2024
Merged via the queue into branch/v15 with commit 1189f94 Aug 8, 2024
35 checks passed
@gzdunek gzdunek deleted the gzdunek/backport-44871/v15 branch August 8, 2024 15:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants