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

updated and synchronized i18n files #247

Merged
merged 2 commits into from
May 7, 2024
Merged

Conversation

arnedierick
Copy link
Contributor

Ik heb beide i18n files overlopen, spelfouten eruit gehaald en uniform Title Case toegepast waar het hoorde.

Dit artikel geeft meer info. Ik heb Title Case toegepast op (tussen)titels, knoppen en boodschappen van maximaal 3 woorden. Dit maakt de site professioneler.

@arnedierick arnedierick requested a review from usserwoutV2 May 5, 2024 09:46
@arnedierick arnedierick linked an issue May 5, 2024 that may be closed by this pull request
@usserwoutV2
Copy link
Contributor

Om eerlijk te zijn, ben ik geen fan van titelcase. Als we naar websites zoals Dodona, Ufora, GitHub, en eigenlijk het grootste deel van alle websites kijken, zien we dat deze ook geen titelcase gebruiken. Persoonlijk vind ik dat het er niet professioneler uitziet. Misschien moeten we eens vragen hoe anderen hierover denken.

@arnedierick
Copy link
Contributor Author

De hele groep heeft beslist dat we toch voor Sentence Case kiezen. Ik pas het aan

Copy link
Contributor

@usserwoutV2 usserwoutV2 left a comment

Choose a reason for hiding this comment

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

Looks good

@arnedierick arnedierick merged commit 2c77831 into frontend May 7, 2024
1 check passed
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.

Taalfouten/translatie verbeteren
2 participants