EdgeTX / edgetx

EdgeTX is the cutting edge open source firmware for your R/C radio
https://edgetx.org
GNU General Public License v2.0
1.59k stars 337 forks source link

Curve types not translated in Inputs or Mixes #3309

Closed ulfhedlund closed 1 year ago

ulfhedlund commented 1 year ago

Is there an existing issue for this problem?

What part of EdgeTX is the focus of this bug?

Transmitter firmware

Current Behavior

The curve types are hardcoded to "Diff", "Expo", "Func" and "Cstm" and can not be translated. It's the same for B&W and color radios.

Expected Behavior

The curve types are not hardcoded and can be translated.

Steps To Reproduce

Go Model Settings - Mixes or Inputs Edit any line Go down to Curve and click in the drop down list to the right.

Version

2.8.0

Transmitter

FrSky X9 Lite / Lite S, Radiomaster TX16S / TX16SMK2

Anything else?

@ParkerEde DE @Eldenroot CZ @robustini IT @zyren CN / TW @ajjjjjjjj PL @ToshihiroMakuuchi JP @HThuren DA

If you can provide translations for the following I'll include them in a PR.

define TR_VCURVETYPE "Diff","Expo","Func","Cstm"

HThuren commented 1 year ago

DA Those are very techincal, so

define TR_VCURVETYPE "Diff","Expo","Funk","Cstm"

zyren commented 1 year ago

CN

define TR_VCURVETYPE "单边","指数","函数","自定义"



TW

define TR_VCURVETYPE "單邊","指數","函數","自定義"


robustini commented 1 year ago

IT

define TR_VCURVETYPE "Diff","Espo","Funz","Pers"

ToshihiroMakuuchi commented 1 year ago

JP

define TR_VCURVETYPE "Diff","Expo","Func","Cstm"

Since it is difficult to express in Japanese, it is fine as it is.

ulfhedlund commented 1 year ago

@HThuren:

Yes, they are quite technical but could nevertheless be translated. I have used "Egen" instead of "Cstm" in Swedish. That may work for Danish as well, perhaps.

ParkerEde commented 1 year ago

DE

define TR_VCURVETYPE "Diff","Expo","Funk","Ind."

Pat6874 commented 1 year ago

FR

define TR_VCURVETYPE "Diff","Expo","Fonct","Perso."

HThuren commented 1 year ago

DA

define TR_VCURVETYPE "Diff","Expo","Funk","Egen"

ulfhedlund commented 1 year ago

@Pat6874: could you have a look at PR #3281 and provide FR translation for that as well?

Eldenroot commented 1 year ago

We cannot use more symbols? Quite difficult for CZ...

Anyway:

CZ:

define TR_VCURVETYPE "Diff","Expo","Funk","Vlas"

ulfhedlund commented 1 year ago

For color radios you could, I suppose, but not for B&W radios with small screens. Den 6 mars 2023 20:10 skrev Eldenroot @.***>: We cannot use more symbols? Quite difficult for CZ...

—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

HThuren commented 1 year ago

For color radios you could, I suppose, but not for B&W radios with small screens. Den 6 mars 2023 20:10 skrev Eldenroot @.>: We cannot use more symbols? Quite difficult for CZ... —Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.>

you can use TR2(x, y), where y will be used on larger screens

Eldenroot commented 1 year ago

@HThuren maybe we should add this to all languages? :)

ulfhedlund commented 1 year ago

@HThuren is right and this is used in many places. But there isn't actually that much more space available for color radios in this case, so we'll keep it the same for all radios.

ulfhedlund commented 1 year ago

Only PL language left now. @ajjjjjjjj - do you have any translations to provide before I make a PR?

pfeerick commented 1 year ago

@Pat6874 Would you be interested in doing other translations for FR? If so, I can add you to the list to be pinged when there is a new translation needed.

Pat6874 commented 1 year ago

Ok, it's possible. Patrick

Le mer. 8 mars 2023, 02:53, Peter Feerick @.***> a écrit :

Closed #3309 https://github.com/EdgeTX/edgetx/issues/3309 as completed via #3319 https://github.com/EdgeTX/edgetx/pull/3319.

— Reply to this email directly, view it on GitHub https://github.com/EdgeTX/edgetx/issues/3309#event-8692207237, or unsubscribe https://github.com/notifications/unsubscribe-auth/A6HE7XPQUPD4ALNPUN7K7ZTW27RBPANCNFSM6AAAAAAVQMUQ64 . You are receiving this because you were mentioned.Message ID: @.***>