search

trezor / trezor-suite

Trezor Suite Monorepo
https://trezor.io/trezor-suite
Other
708 stars 247 forks source link

Label community translations #7624

Closed hynek-jina closed 1 year ago

hynek-jina commented 1 year ago

Add new community translations

Final design: Figma

alex-jerechinsky commented 1 year ago

@hynek-jina blablableble is actually @mroz22. Some translation credits were overwritten during a file structure migration. Italian translation was entirely done by @GiorgioHerbie

hrudolf commented 1 year ago

The transactions label for this issue must be a typo, it should be translations if I'm correct.

matejkriz commented 1 year ago

@Hermez-cz This comment should be addressed https://github.com/trezor/trezor-suite/pull/9151 and new community languages added.

bosomt commented 1 year ago

QA OK

one question is this final text that will be used for Trezor Suite regular users ?

The Trezor community translated Czech, and we are thankful for their contributions. Ensure language consistency by referring to our official languages for verification.

I would expect only info about language and not about translation process, especially when this is first thing that user see when navigating to Settings

Image

Info:

Hannsek commented 1 year ago

Yes, plus it should be somehow collapsible. @Hermez-cz

matejkriz commented 1 year ago

Yes, plus it should be somehow collapsible. @Hermez-cz

What exactly would you expect to be collapsible?

Hannsek commented 1 year ago

The text is imo too long to be there visible all the time.

matejkriz commented 1 year ago

The text is imo too long to be there visible all the time.

Agree! Let's shorten the text then 😉 !

sime commented 1 year ago

The text is imo too long to be there visible all the time.

Agree! Let's shorten the text then 😉 !

As previously mentioned by @komret

Will task @cavigneron (and @Hermez-cz) with this.

Hermez-cz commented 1 year ago

Final & agreed solution:

bosomt commented 1 year ago

QA OK

Info: