Skip to content

Language tags are case-insensitive. Closes #517 - #528

Merged
AmeliaBR merged 2 commits into
w3c:masterfrom
GLRoylance:issue517
Aug 27, 2018
Merged

AmeliaBR merged 2 commits into
w3c:masterfrom
GLRoylance:issue517

Conversation

@GLRoylance

Copy link
Copy Markdown
Contributor

Changed description of systemLanguage matching to make it clear that it is a case-insensitive match rather than an "exact match".

@ewilligers

Copy link
Copy Markdown
Contributor

My informal testing suggests Chrome and Firefox are case-sensitive, and Edge is case-insensitive.

Has this been discussed by the Working Group?

@GLRoylance

GLRoylance commented Aug 16, 2018 via email

Copy link
Copy Markdown
Contributor Author

@ewilligers ewilligers left a comment

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.

The "Document Structure chapter" entry in changes.html should also mention this change.

@AmeliaBR

Copy link
Copy Markdown
Contributor

I've added a changelog entry, and I think this is good to merge.

@svgeesus Are we ok marking this non-substantive for IP? It's a normative correction, but not really adding any intellectual property.

Alternatively, @GLRoylance, if you are a W3C contributor, please link up your W3C & Github accounts following the instructions here: https://labs.w3.org/hatchery/repo-manager/pr/id/w3c/svgwg/528

@GLRoylance

GLRoylance commented Aug 24, 2018 •

Copy link
Copy Markdown
Contributor Author

There ain't no intellectual property here. The edit only clarifies what BCP 47 says.
I release my pull request as CC0.
https://creativecommons.org/share-your-work/public-domain/cc0/
I am not an employee of any W3C entity.

@ewilligers

Copy link
Copy Markdown
Contributor

Marked as non substantive for IPR from ash-nazg.

@AmeliaBR
AmeliaBR merged commit 2736064 into w3c:master Aug 27, 2018
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