-
-
Notifications
You must be signed in to change notification settings - Fork 391
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
Add German language support for Virtual Keyboard #9608
base: master
Are you sure you want to change the base?
Conversation
Hi @roman-yagodin , all letters are generated by the engine. You can take a look at ui_font.cpp file, |
Thanks, I will look into it. |
Hi @roman-yagodin , I assumed that this pull request is ready for review so I added reviewers here. |
Hi @zenseii , would you mind to review this pull request if you have time these days? |
@ihhub. I'll properly review this and other PRs today as I have a bit more time. |
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.
Great job, @roman-yagodin. I left a small comment for improvement with some hopefully useful advice.
Hi @roman-yagodin , if you would be able to update the pull request by today then we can merge it to the upcoming release. |
Hi @zenseii , would you be able to help with the change you requested here? |
@roman-yagodin, I made the suggested commit and solved the merge conflicts. |
OK, thanks @zenseii ! I did some experiments and simply decreasing the sprite width will result in the symbol not being properly centered on the button (a little to the right than I've originally intended): In order to fix this, we need to move entire symbol by -1px along X axis. This will move first drawing point to Another option would be just to leave |
@roman-yagodin, you can use the same format for offsetting buttons on the x-axis and y-axis as we do in the for loop in the top of the function. We do this for the A symbol for example. |
Actually scratch that last idea. That won't work. It will only move the letter in relation to other letters. |
Relates to #8826
The button for "ß" (Eszett) placed in the 3rd row - should be right of "0" in a numeric row on real keyboard.
More importantly, button for "ß" is missing its glyph - where are no such glyph in the font?..
Where in code/resources should I look to address that?