jmuehlner commented on code in PR #796:
URL: https://github.com/apache/guacamole-client/pull/796#discussion_r1114842252
##########
guacamole/src/main/frontend/src/app/clipboard/directives/guacClipboard.js:
##########
@@ -82,7 +100,7 @@ angular.module('clipboard').directive('guacClipboard',
['$injector',
// If the clipboard data is a string, render it as text
if (typeof data.data === 'string')
- element.value = data.data;
+ element.value = (shouldDisplay) ? data.data : '';
Review Comment:
It looks like this just shows an empty clipboard until the user clicks on
the clipboard element, and then the data pops in, yeah? It'd probably be lot
more obvious to users if there was some sort of styling to indicate an inactive
clipboard, and some indication to the user about what to do if they want to see
the clipboard.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
