-
Notifications
You must be signed in to change notification settings - Fork 156
fix(web)!: replace current clipboard logic with auto and manual clipboard modes #935
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
Conversation
…lipboard modes This commit adds: * auto clipboard mode. When it's enabled, clipboard will be automatically monitored for changes and updates from the server will be automatically saved to the local clipboard (it's the old logic, nothing has changed) * Manual clipboard mode. One calls dedicated functions to interact with the clipboard.
|
Hi @CBenoit! Could you review when you get a chance? |
Coverage Report 🤖 ⚙️Past: New: Diff: +0.01% [this comment will be updated automatically] |
…modes This commit refactors the clipboard part of the Devolutions Gateway Standalone to follow the new API(see Devolutions/IronRDP#935) of the _iron-remote-desktop_. The changes: * Introduce _autoclipboard_ mode, which the user can enable/disable using the connection form. However, it's available only for browsers based on _Blink_ engine. For others, _autoclipboard_ mode is always false. * Add two new buttons to the toolbar that are available when autoclipboard is false: _Save Clipboard_ and _Send Clipboard_. * Add notification for manual clipboard actions. For example, when the user clicks on the _Save Clipboard_ button, the error or success notification will appear informing the user about the operation status.
CBenoit
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Nothing blocking, but feel free to open a follow up PR! Thanks!
…modes This commit refactors the clipboard part of the Devolutions Gateway Standalone to follow the new API(see Devolutions/IronRDP#935) of the _iron-remote-desktop_. The changes: * Introduce _autoclipboard_ mode, which the user can enable/disable using the connection form. However, it's available only for browsers based on _Blink_ engine. For others, _autoclipboard_ mode is always false. * Add two new buttons to the toolbar that are available when autoclipboard is false: _Save Clipboard_ and _Send Clipboard_. * Add notification for manual clipboard actions. For example, when the user clicks on the _Save Clipboard_ button, the error or success notification will appear informing the user about the operation status.
This PR adds:
When it's enabled, the clipboard will be automatically monitored for changes, and updates from the server will be automatically saved to the local clipboard (this is the old logic; nothing has changed).
One calls dedicated functions to interact with the clipboard.