Skip to content
This repository was archived by the owner on Sep 6, 2021. It is now read-only.

Conversation

@marcelgerber
Copy link
Contributor

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I've never seen the context - should this be "Speichern unter" or "Speicher als"?

Copy link
Contributor

Choose a reason for hiding this comment

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

I would call it "Speichern als...".

Copy link
Member

Choose a reason for hiding this comment

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

yep, that's better

Copy link
Contributor Author

Choose a reason for hiding this comment

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

As I said, I've never seen the context - In case it means "Save file in selected dir", I would take Speichern unter... (as this is very common). But if it wants to say "Save the file with another extension", then I'd take Speichern als...

German:
Ich habe noch nie wirklich den Kontext gesehen - Wenn es heißen soll "Speichern in ausgewähltem Ordner (mit Dialog zum Auswählen des Speicherorts)", dann würde ich Speichern unter... lassen (da es sehr geläufig ist).
Wenn es aber sagen soll "Datei als anderer Dateityp speichern", dann würde ich auch Speichern als... nehmen.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

As I just saw this comment, I'd leave it as it is "Speichern unter"

Copy link
Contributor

Choose a reason for hiding this comment

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

@SAplayer the usual file save dialog will be opened and the user has the ability to provide and new name and new location.

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 exactly what I meant. Just checked back Notepad++, LibreOffice, SciTE and notepad (Windows Editor), they all use Speichern unter...:
image
image
image

@ghost ghost assigned ingorichter Jan 9, 2014
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sorry, this is not yet merged. But it'll in #6411

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Merged in now.

@marcelgerber marcelgerber mentioned this pull request Jan 10, 2014
@marcelgerber
Copy link
Contributor Author

Just pushed a new one so that everything is up to date.
Ready for review.

Copy link
Member

Choose a reason for hiding this comment

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

Ersetzen statt Überschreiben?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

If both of you (@couzteau and @ingorichter) agree with that, I'd change it.

Copy link
Contributor

Choose a reason for hiding this comment

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

@SAplayer I think Ersetzen is fine.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Just changed that, but FYI: "Überschreiben" is used in some more cases, just try to search for that.

@marcelgerber
Copy link
Contributor Author

FYI: I'll update this PR in a few minutes, as I just saw (git merge) that there were some changes in nls/root.

@marcelgerber
Copy link
Contributor Author

Ready for re-review.

Copy link
Contributor

Choose a reason for hiding this comment

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

This should be aligned properly

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sorry, my mistake. Just fixed.
Don't know why JSLint wasn't complaining about this one.

@WebsiteDeveloper
Copy link
Contributor

@SAplayer good translation.

@marcelgerber
Copy link
Contributor Author

@ingorichter Just saw Strings PRs should point to the release branch.
Is this PR ok or should I create a new one?

@ingorichter
Copy link
Contributor

@SAplayer that would be great If you could create a new one. Sorry, but we've branched master into release today in order to finish sprint 36. Thanks.

@marcelgerber marcelgerber mentioned this pull request Jan 24, 2014
@marcelgerber
Copy link
Contributor Author

Closing in favor of #6632

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants