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

Dutch translation #2999

Merged
merged 2 commits into from
Oct 28, 2017
Merged

Dutch translation #2999

merged 2 commits into from
Oct 28, 2017

Conversation

m-burg
Copy link
Contributor

@m-burg m-burg commented Oct 28, 2017

I have translated everything except for the style guide.

When saving your preferences the confirmation still shows up as 'Saved changes', but I don not know enough web development to fix that.

Also some text boxes are a little too small for the words, this is noticeable for instance in the user settings.

Michel van der Burg added 2 commits October 24, 2017 12:12
"intro": {
"explanation": "<a href=\"https://sandstorm.io\">Sandstorm.io</a> is een open source hosting platform voor persoonlijke/privé instanties van web apps. Gebruikers kunnen software uploaden en installeren via een simpele app-store achtige interface. Naast betere privacy <a href=\"https://blog.sandstorm.io/news/2014-07-21-open-source-web-apps-require-federated-hosting.html\">is dit de enige manier om Open Source web apps haalbaar te maken.</a>",
"version": "Versie: %s",
"unofficialWarning": "<b>WAARSCHUWING:</b> deze server draait een onofficiële build van Sandstorm die aanpassingen kan bevatten die niet zijn gecontroleerd door de ontwikkelaars."
Copy link
Collaborator

Choose a reason for hiding this comment

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

It's in the existing localizations too, so orthoganal to this pr, but we should probably not be using the <b> tag. @kentonv, maybe we should change this to <strong>?

Copy link
Member

Choose a reason for hiding this comment

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

I don't have a strong opinion (... so to speak), but would accept a pull request.

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'll get on it

@kentonv kentonv merged commit 877941e into sandstorm-io:master Oct 28, 2017
@kentonv
Copy link
Member

kentonv commented Oct 28, 2017

Awesome, thanks!

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.

3 participants