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

Set screen language to Dutch #710

Merged
merged 1 commit into from
Nov 7, 2023

Conversation

VincentSmedinga
Copy link
Contributor

@VincentSmedinga VincentSmedinga commented Nov 2, 2023

The default value of lang is unknown, therefore it is recommended to always specify this attribute with the appropriate value. (MDN)

Adding this to the Screen component is appropriate as it’s the outermost component we have control over and it applies to the entire tree (“If these attributes are omitted from an element, then the language of this element is the same as the language of its parent element, if any” – Spec)

Support for more languages through a prop with a number of options may follow – see https://amsterdam.nl/stijlweb/heldere-taal/engels-talen/.

@VincentSmedinga VincentSmedinga requested a review from dlnr November 2, 2023 23:09
@VincentSmedinga VincentSmedinga merged commit 6a111da into main Nov 7, 2023
3 checks passed
@VincentSmedinga VincentSmedinga deleted the feature/DES-444-specify-language branch November 7, 2023 13:05
VincentSmedinga added a commit that referenced this pull request Nov 8, 2023
alimpens added a commit that referenced this pull request Nov 9, 2023
* Revert "Set screen language to Dutch (#710)"

This reverts commit 6a111da.

* Update documentation on the language attribute

---------

Co-authored-by: Aram <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants