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

#1416: Improve German translations #1417

Merged

Conversation

GitRon
Copy link
Contributor

@GitRon GitRon commented Jan 31, 2024

closes #1416.

@@ -1,22 +1,22 @@
export default {
search: {
placeholder: 'Geben Sie ein Schlüsselwort ein...',
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is technically correct but sounds very clunky...

l10n/de_DE.ts Outdated Show resolved Hide resolved
next: 'Nächste',
navigate: (page, pages) => `Seite ${page} von ${pages}`,
page: (page) => `Seite ${page}`,
showing: 'Anzeigen',
showing: 'Datensätze',
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can't use the "showing x of y" in German, so I had to get a little creative. In the perfect world, the whole "sentence" would be a translatable.

to: 'zu',
results: 'Ergebnisse',
to: 'bis',
results: 'Ergebnissen',
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wrong case

},
loading: 'Wird geladen...',
noRecordsFound: 'Keine übereinstimmenden Aufzeichnungen gefunden',
noRecordsFound: 'Keine passenden Daten gefunden',
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

less clunky 🙂

@GitRon
Copy link
Contributor Author

GitRon commented Feb 21, 2024

Could anybody please have a look at the changes? Should be a quick one, right? I'm going to use it in production soon and the current translations are a little off 😅

@afshinm afshinm merged commit d9d6407 into grid-js:master Mar 3, 2024
@afshinm afshinm added the bug fix Something isn't working label Mar 3, 2024
@GitRon GitRon deleted the bugfix/#1416-improve-german-translations branch March 4, 2024 07:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug fix Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Improve German translations
2 participants