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

German Translation Improvements #890

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

kp-leben
Copy link
Contributor

id: 11462375

id: 11462375

Signed-off-by: kp-leben <[email protected]>
@Jeiii20
Copy link
Collaborator

Jeiii20 commented Oct 25, 2024

Please provide a reason for these changes and explain with enough evidence why the previous texts are wrong.

This comment is written as it follows on the Tips topic:

"Adding comments to your PR that explains why subtle changes were made can help. If you are revising something that is already translated, an explanation is required."

@kp-leben
Copy link
Contributor Author

I changed the cardinal points before the place... technically a direct translation of the english string, because its the correct order.

The other translation are also directly translated, as for seasonal the word could be either the word it was or saisonal because it's shorter.

To = Nach, An (An = to a person /// Nach = after / to smth)

Namensfarbe =it missed the 's' in the conjungation

Gesendete Nachrichten is correct, i changed it because of the context so it makes more sense.
Another word is Postausgang (opposite word to inbox)
((it makes more sense because you have Posteingang on the other side)

Erhalte Plasma was changed to 'Plasma erhalten' because it's more logical. (It wouldn't make a huge difference, it just looks better and is grammatically correct.

Copy link
Collaborator

@Jeiii20 Jeiii20 left a comment

Choose a reason for hiding this comment

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

Comments are OK, thanks for your time. I will mention 2 native speakers so they can give their opinions. @0x8fv @Cedricnebu

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants