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

Clarify we depend on UTS46 and not IDNA2008 #722

Merged
merged 3 commits into from
Dec 12, 2022
Merged

Clarify we depend on UTS46 and not IDNA2008 #722

merged 3 commits into from
Dec 12, 2022

Conversation

annevk
Copy link
Member

@annevk annevk commented Dec 9, 2022

Closes #498.


Preview | Diff

@annevk annevk requested a review from TimothyGu December 9, 2022 11:24
@annevk annevk requested a review from domenic December 12, 2022 08:11
Copy link
Member

@domenic domenic left a comment

Choose a reason for hiding this comment

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

LGTM with cite usage nit

url.bs Outdated
@@ -635,6 +635,10 @@ concepts.
<li><p>Return <var>result</var>.
</ol>

<p class=note>This document and the web platform at large use
<cite>Unicode IDNA Compatibility Processing</cite> and not <cite>IDNA2008</cite>. For instance,
Copy link
Member

Choose a reason for hiding this comment

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

It looks like IDNA2008 is not a document name, but instead an informal term for a collection of documents? The relevant document name is instead "Internationalized Domain Names for Applications (IDNA): Definitions and Document Framework".

Copy link
Member Author

@annevk annevk Dec 12, 2022

Choose a reason for hiding this comment

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

So I want to mention IDNA2008 as that is the term in people's minds. I guess I could just not use <cite>. Would I have to quote it, as in "IDNA2008"?

Copy link
Member

Choose a reason for hiding this comment

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

It seems fine to just not use <cite>: and not IDNA2008.

@annevk annevk merged commit 50f0b09 into main Dec 12, 2022
@annevk annevk deleted the annevk/uts46 branch December 12, 2022 08:47
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.

Explain that we are using UTS #46 for IDNA support rather than IDNA2008 directly
2 participants