Skip to content

GUACAMOLE-2237: Pasting multiline text via browser menu produces sing…#1201

Draft
bbennett-ks wants to merge 1 commit into
apache:patchfrom
bbennett-ks:GUACAMOLE-2237-Multiline-Paste-improvements
Draft

GUACAMOLE-2237: Pasting multiline text via browser menu produces sing…#1201
bbennett-ks wants to merge 1 commit into
apache:patchfrom
bbennett-ks:GUACAMOLE-2237-Multiline-Paste-improvements

Conversation

@bbennett-ks

Copy link
Copy Markdown
Contributor

Inline comments explain the core issue and the fix.

@bbennett-ks
bbennett-ks force-pushed the GUACAMOLE-2237-Multiline-Paste-improvements branch from 6553592 to 7543d73 Compare April 13, 2026 15:56
@bbennett-ks

Copy link
Copy Markdown
Contributor Author

I've included the fixes for GUACAMOLE-2260 in this PR:

  • The Keyboard.js fix causes the "extra blank lines" part of the issue.
  • The other part of the 2260 issue is in the same code block.

Comment thread guacamole-common-js/src/main/webapp/modules/Keyboard.js Outdated
Comment thread guacamole/src/main/frontend/src/app/textInput/directives/guacTextInput.js Outdated
@bbennett-ks
bbennett-ks force-pushed the GUACAMOLE-2237-Multiline-Paste-improvements branch 2 times, most recently from 2bd1f7e to 41fcb87 Compare April 16, 2026 03:24
@bbennett-ks
bbennett-ks force-pushed the GUACAMOLE-2237-Multiline-Paste-improvements branch from 41fcb87 to 11effb7 Compare May 27, 2026 04:42
@mike-jumper
mike-jumper marked this pull request as draft July 14, 2026 03:37
@mike-jumper
mike-jumper changed the base branch from staging/1.6.1 to patch July 14, 2026 03:38
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.

2 participants