Skip to content

General overhaul of the de-localization#430

Merged
Rokt33r merged 5 commits intoBoostIO:masterfrom
moltenbit:master
Apr 25, 2020
Merged

General overhaul of the de-localization#430
Rokt33r merged 5 commits intoBoostIO:masterfrom
moltenbit:master

Conversation

@moltenbit
Copy link
Copy Markdown
Contributor

General overhaul of the de-localization.

I feel like Speicherort as the main translation for storage is generally correct but quite unusual. You should think about maybe changing it to something like Notizbuch = Notebook on many occasions. If you wish to implement that feel free to ask me to translate it.

'note.date': currently it says e.g. 38 minutes her but it should either be 38 Minuten her or vor 38 Minuten. IMO the second one is better, but I think more code adjustment is necessary somewhere else to rephrase the wording.

Btw: where can I translate other things like e.g. Storage Settings or Remove Storage when you edit your storage?

sysadt added 2 commits April 23, 2020 18:50
General overhaul of the de-localization.

I feel like ```Speicherort``` as the main translation for ```storage``` is generally correct, but quite unusual. You should think about maybe changing it to something like ```Notizbuch = Notebook``` on many occasions. If you wish to implement that feel free to ask me to translate it.

'note.date': currently it says ```38 minutes her``` but it should either be ```38 Minuten her``` or ```vor 38 Minuten```. IMO the second one is better, but I think more code adjustment is necessary somewhere else to rephrase the wording.
@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Apr 23, 2020

CLA assistant check
All committers have signed the CLA.

@arcturus140
Copy link
Copy Markdown
Contributor

are storage locations deleted permanentely or simply unlinked? I am not sure about the behaviour for each storage type. This requires some clarification.

Copy link
Copy Markdown
Contributor

@arcturus140 arcturus140 left a comment

Choose a reason for hiding this comment

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

I have one suggestion, LGTM otherwise.

Comment thread src/locales/de.ts Outdated
'tag.removeMessage': 'Das Etikett wird von allen Notizen entfernt.',
'tag.tag': 'Tag',
'tag.remove': 'Tag löschen',
'tag.removeMessage': 'Der Tag wird von allen Notizen entfernt.',
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Das Tag

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Der and Das Tag is both correct according to the sources I looked at

Copy link
Copy Markdown
Contributor

@arcturus140 arcturus140 Apr 24, 2020

Choose a reason for hiding this comment

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

this is correct. This is an entirely personal preference. I prefer the latter. I'll leave it to you.

Comment thread src/locales/de.ts Outdated
//Billing
'billing.billing': 'Billing',
'billing.message': 'Bitte melde dich an um deinen Plan zu ugpraden.',
'billing.message': 'Bitte melde dich an, um deinen Plan zu ugpraden.',
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

suggestion: zu erweitern

Comment thread src/locales/de.ts Outdated
@@ -145,19 +145,19 @@ export default {
// Preferences MarkdownTab
'preferences.previewStyle': 'Vorschau Style',
'preferences.markdownCodeBlockTheme': 'Code Block Theme',
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Block Design

Comment thread src/locales/de.ts Outdated
'storage.renameMessage': 'Name des Speicherorts ändern',
'storage.remove': 'Speicherort entfernen',
'storage.removeMessage': 'Speicherort wird von App entknüpft',
'storage.removeMessage': 'Speicherort wird gelöscht',
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

are they really deleted? In the old app they where merely unlinked. gelöscht would cause confusion, then.

If they aren't, which I don't know if that is the case, then Verknüpfung zum Speicherort entfernen would be better.

@arcturus140
Copy link
Copy Markdown
Contributor

IMO the second one is better

Yes, the second one is correct.

I prefer Speicherort over Notizbuch

@ZeroX-DG ZeroX-DG added the awaiting review ❇️ Pull request is awaiting a review. label Apr 24, 2020
small adjustments thanks to @arcturus140
@moltenbit
Copy link
Copy Markdown
Contributor Author

Things that need clarification:

  • 'storage.removeMessage': are storage locations deleted permanently or just unlinked?
  • 'note.date': where to edit the other part of the sentence where this variable is being used?
  • in general, where to edit other parts of the software? E.g. when clicking Preferences, a lot is still not translated, as well as the buttons at the very top: File, Edit, View, Window, Help

@arcturus140
Copy link
Copy Markdown
Contributor

as far as this PR is concerned, you should assume an unlink.

discussion on your bullet points can be continued here but at the end of the day a separate issue is required since every language scheme is affected.

@moltenbit
Copy link
Copy Markdown
Contributor Author

as far as this PR is concerned, you should assume an unlink.

discussion on your bullet points can be continued here but at the end of the day a separate issue is required since every language scheme is affected.

Done. Ready for pull.

@arcturus140
Copy link
Copy Markdown
Contributor

it doesn't compile. apparently you're short on a string terminator or a comma.

@arcturus140
Copy link
Copy Markdown
Contributor

I'll give this PR my 100% 🇩🇪 rating.

thank you for your contribution

Copy link
Copy Markdown
Member

@ZeroX-DG ZeroX-DG left a comment

Choose a reason for hiding this comment

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

Alright, thanks for this PR guys. I'll approve it

@ZeroX-DG ZeroX-DG added approved 👍 Pull request has been approved by sufficient reviewers. and removed awaiting review ❇️ Pull request is awaiting a review. labels Apr 25, 2020
@Rokt33r Rokt33r merged commit 630c15d into BoostIO:master Apr 25, 2020
@Rokt33r Rokt33r added this to the v0.5.0 milestone Apr 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved 👍 Pull request has been approved by sufficient reviewers.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants