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

Fix keyboard-layout being changed to US on RDP connection #268

Merged
merged 2 commits into from
Nov 1, 2024

Conversation

tristanRW
Copy link
Contributor

Added the IgnoreRemoteKeyboardLayout Registry Key to prevent automatic changes to the keyboard layout.

As mentioned in this article the rdp server adds and activates the US keyboard-layout if the keyboard-layout of the RDP client does not match the RDP servers.
Since our client is on linux it is very likely that this is the case.
This changes makes the RDP-session ignore the clients keyboard-layout and keeps the one that is configured on the server (= the one set up during the setup of the windows VM).

Added the IgnoreRemoteKeyboardLayout Registry Key to prevent automatic changes to the keyboard layout.

As mentioned in (this article)[https://poweradm.com/ignoreremotekeyboardlayout-windows-rdp/] the rdp server adds and activates the US keyboard-layout if the keyboard-layout of the client does not match the servers. Since our client is on linux there are many cases where this is not given, so it is better to let the user choose a layout on windows.
@CLAassistant
Copy link

CLAassistant commented Sep 23, 2024

CLA assistant check
All committers have signed the CLA.

@LDprg
Copy link
Member

LDprg commented Oct 1, 2024

@oskardotglobal @KernelGhost What do you think? I think this could solve a lot of this weird keyboard issues.

Copy link
Member

@oskardotglobal oskardotglobal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@oskardotglobal oskardotglobal merged commit b49979e into winapps-org:main Nov 1, 2024
3 checks passed
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.

4 participants