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

Loosen coupling between usesForced and forced bound parameter (#609). #611

Closed
wants to merge 1 commit into from

Conversation

nigelmegitt
Copy link
Contributor

Closes #609 by changing the definition of the usesForced named metadata item in the way proposed there.

Closes #609 by changing the definition of the usesForced named metadata item in the way proposed there.
adheres to <loc href="http://www.w3.org/TR/xmlschema-2/#boolean">xsd:boolean</loc>. If this named metadata item
is present in a <loc href="#terms-document-instance">document instance</loc>, then it must be specified as a child
of the <loc href="#document-structure-vocabulary-head"><el>head</el></loc> element.</p>
<p>A boolean that expresses whether some content within the specifying element's scope makes use of forced display
Copy link
Collaborator

Choose a reason for hiding this comment

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

Remove all scoping semantics that are not document scoped.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I believe this change already meets the agreed relaxation of the "must be specified as a child of the head element" to express only significance - see #609 (comment)

Rather than expressing a boolean significance based on the location, it instead relates the position to the scope of significance.

@skynavga skynavga added this to the Post CR1 milestone Jan 30, 2018
Copy link
Contributor

@palemieux palemieux left a comment

Choose a reason for hiding this comment

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

What use case would require usesForced to be specified anywhere but <tt>?

@skynavga skynavga changed the title Loosen coupling between usesForced and forced bound parameter Loosen coupling between usesForced and forced bound parameter (#609). Jan 31, 2018
@nigelmegitt
Copy link
Contributor Author

What use case would require usesForced to be specified anywhere but <tt>?

@palemieux An author may wish to indicate specific content elements for example <div>s that contain forced content, perhaps because they have been introduced or modified as part of the authoring process and require some additional quality control, for example.

The counter-case is also relevant here. Rather than leaving a possible syntactic option open to differences in interpretation which would not be interoperable, I would prefer to state a baseline interpretation now, which would be open to review.

@palemieux
Copy link
Contributor

An author may wish to indicate specific content elements for example

s that contain forced content, perhaps because they have
been introduced or modified as part of the authoring process and require some additional quality control, for example.

Isnt't this true about any change? Should there be a usesStyle named metadata item? I really fail to see an actual use case for usesForced on anything but head. This is of course from the perspective of IMSC1.1. I do not object to adding it in TTML2 for potential future use, as long as it is prohibited in IMSC1.1.

@nigelmegitt
Copy link
Contributor Author

Isn't this true about any change?

Maybe, and ttm:item is extensible to cover that.

Should there be a usesStyle named metadata item?

Nobody has asked for one so far, so no.

I really fail to see an actual use case for usesForced on anything but head. This is of course from the perspective of IMSC1.1. I do not object to adding it in TTML2 for potential future use, as long as it is prohibited in IMSC1.1.

I'm not going accept a restriction on metadata in IMSC 1.1 just because you don't see the use case for it @palemieux , when it has no impact on processing. It just isn't dangerous enough to prohibit.

However I'm not really clear about the use case for the usesForced metadata item being defined in TTML2 at all, regardless of scope. Can you or @skynavga or anyone else here recall why it was added? Could we simply remove it and resolve this discussion with "less is more"?

@palemieux
Copy link
Contributor

palemieux commented Jan 31, 2018

Could we simply remove it and resolve this discussion with "less is more"?

Yes!

Can you or @skynavga or anyone else here recall why it was added?

I did not ask for it, and do not know of an outstanding requirement for it.

@nigelmegitt
Copy link
Contributor Author

Closing in favour of #621.

@nigelmegitt nigelmegitt closed this Feb 7, 2018
@skynavga skynavga deleted the issue-0609-usesForced branch March 9, 2018 21:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants