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

Update terminology about surrogates #28444

Merged
merged 4 commits into from
Aug 9, 2023

Conversation

JasonLamv-t
Copy link
Member

Description

According to https://unicode.org/faq/utf_bom.html and https://en.wikipedia.org/wiki/UTF-16, adjust some terms about surrogates

Motivation

Ditto.

Additional details

None.

Related issues and pull requests

@JasonLamv-t JasonLamv-t requested a review from a team as a code owner August 8, 2023 08:57
@JasonLamv-t JasonLamv-t requested review from Josh-Cena and removed request for a team August 8, 2023 08:57
@github-actions github-actions bot added the Content:JS JavaScript docs label Aug 8, 2023
@Josh-Cena Josh-Cena changed the title Update the overview about String Update terminology about surrogates Aug 8, 2023
Copy link
Member

@Josh-Cena Josh-Cena left a comment

Choose a reason for hiding this comment

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

Thank you! Let's align with the terminology in the ES spec as well.

…ex.md

Co-authored-by: Joshua Chen <sidachen2003@gmail.com>
Copy link
Member

@Josh-Cena Josh-Cena left a comment

Choose a reason for hiding this comment

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

Thank you!

@Josh-Cena Josh-Cena merged commit 96450d8 into mdn:main Aug 9, 2023
@JasonLamv-t JasonLamv-t deleted the JasonLamv-t-patch-1 branch August 15, 2023 01:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content:JS JavaScript docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants