You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@guacamole.apache.org by GitBox <gi...@apache.org> on 2021/02/22 05:37:38 UTC

[GitHub] [guacamole-client] mike-jumper commented on pull request #505: GUACAMOLE-989: Implement keyboard lock menu item.

mike-jumper commented on pull request #505:
URL: https://github.com/apache/guacamole-client/pull/505#issuecomment-783097870


   So, I've been plowing through the PR backlog and came across these changes. Pulling them to test things out locally, I see the new menu option:
   
   ![image](https://user-images.githubusercontent.com/4632905/108666555-38ac6e00-748c-11eb-94ad-d201d7bf4def.png)
   
   but clicking "Lock keyboard" seems to have no effect, at least when testing from my Chromebook. The Guacamole menu closes as a result of clicking the menu item, but that's it. Is there something I'm missing?
   
   Regarding the functionality itself:
   
   * We should probable use a different icon.
   * What if the user wishes to unlock the keyboard? Should this be a checkbox?
   * Would locating this elsewhere within Guacamole menu be better?


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org