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

Editorial: Document behaviour of U extension values locale slots #823

Merged
merged 4 commits into from
Sep 19, 2023

Conversation

ben-allen
Copy link
Contributor

Fix #311 by adding a non-normative note explaining the behaviour of the [[locale]] slot in ResolveLocale AO with regard to what U Extension tags appear or don't appear in the string stored in this slot.

Replaces #817

@ben-allen ben-allen force-pushed the u-extension-values-locale-slots branch from 5c5868a to cf845cd Compare August 9, 2023 20:42
spec/negotiation.html Outdated Show resolved Hide resolved
Copy link
Contributor

@sffc sffc left a comment

Choose a reason for hiding this comment

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

Also please note that we only remove the extension tag if it is different than the options bag.

ben-allen and others added 2 commits August 31, 2023 14:34
Co-authored-by: Richard Gibson <[email protected]>
…rom locale strings if an option is specified with a value for that extension that _differs_ from the vvalue set in the extension sequence
@ben-allen
Copy link
Contributor Author

@gibson042 @ryzokuken
Is this one ready to merge?

@gibson042 gibson042 merged commit 4dbe8df into tc39:master Sep 19, 2023
2 checks 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.

strip out u-extension in the locale returned by resolvedOptions()
3 participants