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

Review IT #3505

Merged
merged 3 commits into from
May 11, 2023
Merged

Review IT #3505

merged 3 commits into from
May 11, 2023

Conversation

Pancronos
Copy link
Contributor

Signed-off-by: Pancronos [email protected]

Fixes #

Summary of changes:

@gagarinlg
Copy link
Member

did you test this on ColoLCD and B&W targets, to check if the translations fit on the screen?

@Pancronos
Copy link
Contributor Author

did you test this on ColoLCD and B&W targets, to check if the translations fit on the screen?

Nope, couldn't test but i tried to keep same lenght as english, in any case shorter.

@robustini
Copy link
Contributor

robustini commented May 9, 2023

It's okay with me, nice job, thanks.
Maybe this would be better... :-)

Update V2.9.x by: Marco Robustini [email protected] - Fix V2.9.x by: MatCron [email protected]

@pfeerick
Copy link
Member

pfeerick commented May 10, 2023

Thanks Marco :) I'll see what the conflict is and get this merged soon. And thanks for the review Pancronos, more native language eyes the better :)

@pfeerick pfeerick added this to the 2.9 milestone May 10, 2023
@pfeerick pfeerick merged commit 0ef4199 into EdgeTX:main May 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants