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

UI i18n: percentage formatting #13880

Merged
merged 4 commits into from May 17, 2024
Merged

Conversation

naltatis
Copy link
Member

@naltatis naltatis commented May 12, 2024

fixes #8510, fixes #13836

overview for all existing languages:

Bildschirmfoto 2024-05-12 um 16 14 42

English

Bildschirmfoto 2024-05-12 um 16 03 58

German

Bildschirmfoto 2024-05-12 um 16 19 08

Arabic

Bildschirmfoto 2024-05-12 um 16 02 16 Bildschirmfoto 2024-05-12 um 16 02 12 Bildschirmfoto 2024-05-12 um 16 02 08 Bildschirmfoto 2024-05-12 um 16 02 01 Bildschirmfoto 2024-05-12 um 16 01 39

Turkish

Bildschirmfoto 2024-05-12 um 16 02 37 Bildschirmfoto 2024-05-12 um 16 02 30 Bildschirmfoto 2024-05-12 um 16 02 27 Bildschirmfoto 2024-05-12 um 16 02 55

@naltatis naltatis added the ux User experience/ interface label May 12, 2024
@naltatis naltatis changed the title UI i18n: ipercentage formatting UI i18n: percentage formatting May 12, 2024
@andig
Copy link
Member

andig commented May 15, 2024

fixes #8510, #13836

Das verlinkt nicht ;)

@andig
Copy link
Member

andig commented May 15, 2024

@naltatis 13% aber 135 g? Gibt es Sprachen, wo nur das %, nicht aber sonstige Einheiten, eine Sonderbehandlung bekommt?

@naltatis
Copy link
Member Author

In diesem PR hab ich mich erstmal nur um % gekümmert. Bei den anderen Einheiten können wir gerne separat noch mal gucken. Aber in türkischen muss scheinbar wirklich nur das % nach vorne.

@naltatis
Copy link
Member Author

Das verlinkt nicht ;)

Bei mir schon.

@andig
Copy link
Member

andig commented May 16, 2024

Was ich sagen wollte: es taucht nicht als PR der die Issues schliesst in den issues auf. Siehe Syntax für multiple issues: https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue:

Multiple issues: Use full syntax for each issue

IMG_0711

@naltatis
Copy link
Member Author

Was ich sagen wollte: es taucht nicht als PR der die Issues schliesst in den issues auf.

Ah verstehe. Habs angepasst.

@andig
Copy link
Member

andig commented May 17, 2024

Gibt noch einen Konflikt bei TR

@naltatis naltatis merged commit df5d6ec into master May 17, 2024
7 checks passed
@naltatis naltatis deleted the feature/percentage_i18n_formatting branch May 17, 2024 08:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ux User experience/ interface
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Translations: Improve language-specific formatting Zahl und Prozentsymbol nach DIN 5008 formatieren
2 participants