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

Apply tt:initial to extension style properties #396

Merged
merged 1 commit into from Jun 28, 2018

Conversation

Projects
None yet
3 participants
@palemieux
Copy link
Contributor

commented Jun 21, 2018

Closes #386

@palemieux palemieux added this to the imsc1.1 CR2 milestone Jun 21, 2018

@palemieux palemieux self-assigned this Jun 21, 2018

@palemieux palemieux requested a review from nigelmegitt Jun 21, 2018

[[!TTML2]]:</p>
<p>The following style properties are subject to the <em>Style Resolution</em> procedures specified at Section 10.4 of
[[!TTML2]], and MAY be specified as an attribute of the <code>initial</code> element specified at Section 10.1.1 of
[[!TTML2]], instead of an attribute in the <em>TT Style Namespaces</em>:</p>

This comment has been minimized.

Copy link
@nigelmegitt

nigelmegitt Jun 28, 2018

Contributor

Suggest changing "instead of" to "with the same semantics as".

This comment has been minimized.

Copy link
@palemieux

palemieux Jun 28, 2018

Author Contributor

I think it is important to specify that an attribute in the TT Style Namespaces shall not also be specified.

This comment has been minimized.

Copy link
@nigelmegitt

nigelmegitt Jun 28, 2018

Contributor

I agree.

@nigelmegitt

This comment has been minimized.

Copy link
Contributor

commented Jun 28, 2018

There's an additional new reference to TTML2 in here which will need to be modified to point to the dated CR2 version either in this pull request or in #412 depending on which is merged first.

@palemieux palemieux merged commit ef53bf5 into master Jun 28, 2018

1 check passed

ipr PR deemed acceptable.
Details

@palemieux palemieux deleted the issue-386-clarify-initial-application branch Jul 17, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.