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

Add German language support for Virtual Keyboard #9608

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

roman-yagodin
Copy link
Contributor

@roman-yagodin roman-yagodin commented Mar 6, 2025

Relates to #8826

image

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?

@roman-yagodin roman-yagodin marked this pull request as draft March 6, 2025 06:12
@ihhub
Copy link
Owner

ihhub commented Mar 7, 2025

Hi @roman-yagodin , all letters are generated by the engine. You can take a look at ui_font.cpp file, generateCP1252GoodButtonFont() function.

@roman-yagodin
Copy link
Contributor Author

Thanks, I will look into it.

@roman-yagodin
Copy link
Contributor Author

roman-yagodin commented Mar 9, 2025

Screenshot with the brand-new "ß" letter on button:

image

Enlarged:

image

Hope it will be in a row with rest of the style!

@ihhub ihhub marked this pull request as ready for review March 18, 2025 02:37
@ihhub ihhub requested review from zenseii and ihhub March 18, 2025 02:37
@ihhub ihhub added ui UI/GUI related stuff translation Things related to languages and translations improvement New feature, request or improvement labels Mar 18, 2025
@ihhub ihhub added this to the 1.1.7 milestone Mar 18, 2025
@ihhub
Copy link
Owner

ihhub commented Mar 18, 2025

Hi @roman-yagodin , I assumed that this pull request is ready for review so I added reviewers here.

@ihhub
Copy link
Owner

ihhub commented Mar 20, 2025

Hi @zenseii , would you mind to review this pull request if you have time these days?

@zenseii
Copy link
Collaborator

zenseii commented Mar 20, 2025

@ihhub. I'll properly review this and other PRs today as I have a bit more time.

Copy link
Collaborator

@zenseii zenseii left a 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.

@ihhub
Copy link
Owner

ihhub commented Mar 21, 2025

Hi @roman-yagodin , if you would be able to update the pull request by today then we can merge it to the upcoming release.

@ihhub
Copy link
Owner

ihhub commented Mar 21, 2025

Hi @zenseii , would you be able to help with the change you requested here?

@zenseii
Copy link
Collaborator

zenseii commented Mar 21, 2025

@roman-yagodin, I made the suggested commit and solved the merge conflicts.

@roman-yagodin
Copy link
Contributor Author

roman-yagodin commented Mar 21, 2025

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):

w13

In order to fix this, we need to move entire symbol by -1px along X axis. This will move first drawing point to (offset - 1, offset + 9), which is effectively inside 2px offset frame. I'm not sure is it acceptable or not.

Another option would be just to leave 15 + offset * 2 width as in my original commit... 😄

@zenseii
Copy link
Collaborator

zenseii commented Mar 21, 2025

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

@zenseii
Copy link
Collaborator

zenseii commented Mar 21, 2025

Actually scratch that last idea. That won't work. It will only move the letter in relation to other letters.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement New feature, request or improvement translation Things related to languages and translations ui UI/GUI related stuff
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants