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

Possible misinterpretation of note in §11.3.1.2 Inline Regions. #592

Closed
nigelmegitt opened this issue Jan 26, 2018 · 8 comments
Closed

Comments

@nigelmegitt
Copy link
Contributor

The note:

A content element can only be associated with a single region. Consequently, if a content element specifies a region attribute, then any explicit inline region specification is ignored.

is incorrect - we have established through discussion of other issues that some content elements are associated with multiple regions. I think this is about explicit association by specifying a region attribute. Suggest rewording to:

A content element can only specify a single region. Consequently, if a content element specifies a region attribute, then any explicit inline region specification is ignored.

@skynavga skynavga changed the title Error in note in §11.3.1.2 Inline Regions Possible misinterpretation of note in §11.3.1.2 Inline Regions. Jan 26, 2018
nigelmegitt added a commit that referenced this issue Feb 2, 2018
Closes #592 by editing the note text about region association with a content element to clarify that it refers to direct association via a region attribute or child region element.
@nigelmegitt nigelmegitt self-assigned this Feb 2, 2018
skynavga added a commit that referenced this issue Feb 7, 2018
Clarify the note text around region association (#592).
@skynavga skynavga added pr merged and removed pr open labels Feb 7, 2018
@skynavga skynavga added this to the Editor's CR Work List milestone Feb 7, 2018
@palemieux palemieux reopened this Feb 7, 2018
@palemieux
Copy link
Contributor

The PR was closed before the 14 day period was over. I do not have a concern with the contents of the note. It should however not be a note since it sets a normative precedence order.

@skynavga
Copy link
Collaborator

skynavga commented Feb 7, 2018

This issue and PR was marked editorial. I have just reviewed the details of the change (again) and it is indeed editorial. Therefore, we have followed the rules.

@skynavga skynavga closed this as completed Feb 7, 2018
@skynavga
Copy link
Collaborator

skynavga commented Feb 7, 2018

And, no, it does not set a precedence order, normative or not.

@skynavga
Copy link
Collaborator

skynavga commented Feb 7, 2018

Just a reminder, an addition, change, or removal of a note is, by definition, editorial, since all notes are non-normative.

@palemieux palemieux reopened this Feb 7, 2018
@palemieux
Copy link
Contributor

Where is the normative prose that specifies that explicit inline region specification is ignored, if not in this note?

@skynavga
Copy link
Collaborator

skynavga commented Feb 7, 2018

Repeating myself, any and all information in a note is non-normative in TTML:

2.3 Document Conventions
All paragraphs marked as a Note are considered non-normative.

Therefore, it is NOT normative. If you want to open another issue with a complaint that someone might (foolishly) interpret the text of this note as a normative statement, then feel free to do so, but do not keep reopening this issue.

@skynavga skynavga closed this as completed Feb 7, 2018
@palemieux palemieux reopened this Feb 7, 2018
@skynavga skynavga closed this as completed Feb 7, 2018
@palemieux palemieux reopened this Feb 8, 2018
@skynavga
Copy link
Collaborator

skynavga commented Feb 8, 2018

I can keep closing this all night. Please file another issue if you wish.

@skynavga skynavga closed this as completed Feb 8, 2018
@nigelmegitt
Copy link
Contributor Author

Where is the normative prose that specifies that explicit inline region specification is ignored, if not in this note?

@palemieux I'm content that this is normatively stated, since the first step in [process inline regions] is:

  1. if the [attributes] information item property of B contains a region attribute, then exit this procedure

So I agree this is editorial.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants