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

The #extent feature implies #extent-auto. #837

Closed
palemieux opened this issue Jun 18, 2018 · 3 comments
Closed

The #extent feature implies #extent-auto. #837

palemieux opened this issue Jun 18, 2018 · 3 comments

Comments

@palemieux
Copy link
Contributor

Since auto was defined in TTML1 as part of the tts:extent attribute

@skynavga skynavga added this to the CR2 milestone Jun 19, 2018
@skynavga skynavga changed the title #extent should imply #extent-auto The #extent feature implies #extent-auto. Jun 19, 2018
@skynavga skynavga self-assigned this Jun 19, 2018
@skynavga
Copy link
Collaborator

No change is required, since by transitive closure with support for #extent-{region,root}, we already have support for #extent-auto (and #extent-length), as well as a proscription against their prohibition.

@nigelmegitt
Copy link
Contributor

I follow that logic @skynavga but I think the request was to include #extent-auto in the note about implied related feature support under #extent, which would not be false, and may be useful. @palemieux is that something you still want to add?

@skynavga
Copy link
Collaborator

@nigelmegitt I don't want to do that because that would mean I need to add all implications and proscriptions to other feature definitions that involve transitive closures. That is a bad specification strategy and subject to errors from saying the same thing twice (or more).

@skynavga skynavga removed their assignment Jun 21, 2018
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