-
Notifications
You must be signed in to change notification settings - Fork 40
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
Fix #217: Clarify that only validators will validate language tags #256
Conversation
cc @aphillips |
Ping @aphillips @r12a |
448b109
to
4e13961
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For some reason, we missed closing out issue 217 during the pre-CR phase of webvtt. The I18N WG actioned me to following up in our teleconference of 2020-03-26 as part of our periodic housekeeping.
@@ -2842,6 +2846,10 @@ Object">WebVTT Internal Node Objects</a> also have an ordered list of class name | |||
their <dfn lt="WebVTT Node Object's applicable language">applicable language</dfn>, which is to be | |||
interpreted as a BCP 47 language tag. [[!BCP47]]</p> | |||
|
|||
<p class=note>User agents will add a language tag as the <a lt="WebVTT Node Object's applicable |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I understand the goal of this note, but it seems like it isn't worded correctly, since we want to encourage user agents to supply a well-formed and valid language tag, but not require them to check the language tag. I would probably say:
User agents must add a language tag as the applicable language. While language tag should be a valid language tag [[!BCP47]], the user agent is not required to check the structure or contents of the language tag to ensure that this is so.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Except notes are non-normative and shouldn't use normative keywords (must, should, required).
@@ -3160,6 +3163,7 @@ <h3 class="heading settled" data-algorithm="WebVTT cue text parsing rules" data- | |||
parent-child relationships so constructed form a tree structure. <a data-link-type="dfn" href="#webvtt-internal-node-object">WebVTT Internal Node Objects</a> also have an ordered list of class names, known as their <dfn data-dfn-type="dfn" data-lt="WebVTT Node Object’s applicable classes" data-noexport="" id="webvtt-node-objects-applicable-classes">applicable classes<a class="self-link" href="#webvtt-node-objects-applicable-classes"></a></dfn>, and a language, known as | |||
their <dfn data-dfn-type="dfn" data-lt="WebVTT Node Object’s applicable language" data-noexport="" id="webvtt-node-objects-applicable-language">applicable language<a class="self-link" href="#webvtt-node-objects-applicable-language"></a></dfn>, which is to be | |||
interpreted as a BCP 47 language tag. <a data-link-type="biblio" href="#biblio-bcp47">[BCP47]</a></p> | |||
<p class="note" role="note">User agents will add a language tag as the <a data-link-type="dfn" href="#webvtt-node-objects-applicable-language">applicable language</a> even if it is not a valid or not even well-formed language tag. <a data-link-type="biblio" href="#biblio-bcp47">[BCP47]</a></p> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See above.
See #217