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

<percentage> not defined #210

Closed
2 tasks done
nigelmegitt opened this issue Feb 10, 2017 · 5 comments
Closed
2 tasks done

<percentage> not defined #210

nigelmegitt opened this issue Feb 10, 2017 · 5 comments
Assignees
Labels

Comments

@nigelmegitt
Copy link
Contributor

nigelmegitt commented Feb 10, 2017

The new attribute ittp:activeArea takes 4 of <percentage> but this is not defined. One place it could potentially be derived from is a subset of the TTML1SE definition of <length> however that definition of percentage is based on number which permits negative values; in the case of ittp:activeArea negative values are prohibited.

Steps to resolve:

  • Clarify source of definition of <percentage>.
  • If it's from TTML1SE as above then add a normative statement prohibiting negative values.
palemieux added a commit that referenced this issue Feb 10, 2017
@palemieux palemieux self-assigned this Feb 10, 2017
@palemieux
Copy link
Contributor

Clarify source of definition of .

The source of <percentage> has been clarified.

If it's from TTML1SE as above then add a normative statement prohibiting negative values.

The ittp:activeArea syntax definition specifies that the lengths are non-negative.

@nigelmegitt
Copy link
Contributor Author

Thanks @palemieux that helps. I'm a bit unclear about the normative status of something with a syntax that looks like a code comment; I'm not sure if this is done elsewhere and if it is strong enough. I would prefer an explicit statement with a normative language keyword for the avoidance of any doubt.

@palemieux
Copy link
Contributor

@nigelmegitt Constraints expressed in-band the syntax definition is used throughout TTML1 and IMSC1, see ttp:cellResolution and ittp:aspectRatio, for instance.

@nigelmegitt
Copy link
Contributor Author

OK I'm not delighted by it, but am prepared to accept it.

@palemieux
Copy link
Contributor

Closed by #209

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

No branches or pull requests

2 participants