Skip to content

GUACAMOLE-925: Add Russian keymap support "ru-ru-qwerty"#1044

Closed
VAGurko wants to merge 3 commits intoapache:mainfrom
VAGurko:main
Closed

GUACAMOLE-925: Add Russian keymap support "ru-ru-qwerty"#1044
VAGurko wants to merge 3 commits intoapache:mainfrom
VAGurko:main

Conversation

@VAGurko
Copy link
Copy Markdown

@VAGurko VAGurko commented Jan 4, 2025

The parameter "ru-ru-qwerty" has been added to the "guacamole-ext/src/main/resources/org/apache/guacamole/protocols/rdp.json" file to support the Russian keyboard layout.
The parameter "FIELD_OPTION_SERVER_LAYOUT_RU_RU_QWERTY" : "Russian (Qwerty)" has been added to the "guacamole/src/main/frontend/src/translations/en.json" file to support the Russian keyboard layout.

I added the "Ё" character to support the Russian keyboard layout and corrected the missing comma in the configuration file. Everything is working correctly in the current build of the application.

The parameter "ru-ru-qwerty" has been added to the "guacamole-ext/src/main/resources/org/apache/guacamole/protocols/rdp.json" file to support the Russian keyboard layout.
The parameter "FIELD_OPTION_SERVER_LAYOUT_RU_RU_QWERTY" : "Russian (Qwerty)" has been added to the "guacamole/src/main/frontend/src/translations/en.json" file to support the Russian keyboard layout.
…camole interface "секуны", it needs to be fixed to "секунды".

In the file "guacamole/src/main/frontend/src/translations/ru.json" the typo of "секуны" has been corrected to "секунды".
@mike-jumper mike-jumper changed the base branch from main to patch January 14, 2025 23:18
@mike-jumper mike-jumper changed the base branch from patch to main January 14, 2025 23:19
Copy link
Copy Markdown
Contributor

@mike-jumper mike-jumper left a comment

Choose a reason for hiding this comment

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

Please separate commit b445aa7 (from GUACAMOLE-2013) into a separate PR. Since GUACAMOLE-2013 is just a typo fix, that PR would be best against patch instead of main.

Otherwise, LGTM (pending merge of apache/guacamole-server#570).

@VAGurko
Copy link
Copy Markdown
Author

VAGurko commented Jan 15, 2025

Yes, I was in a hurry and made a mistake, I'll go fix it.

@VAGurko VAGurko closed this Jan 15, 2025
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