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

Property tts:origin permits rw/rh but #origin is defined per TTML1. #972

Closed
palemieux opened this issue Aug 6, 2018 · 5 comments
Closed

Comments

@palemieux
Copy link
Contributor

Define an additional feature that specifies processing of rw/rh lengths on tts:origin

@nigelmegitt
Copy link
Contributor

I'm not actually sure we need this. There is a feature gap in which <length> admits rw and rh syntactically, but the #origin features is based on the TTML1 definition that does not allow those units. The actual impact of that gap seems small. I'm assuming we have other theoretical feature gaps if we look hard enough, but that we've taken the view that they don't necessarily need to be filled.

Another resolution would be to prohibit use of rw and rh on tts:origin.

@skynavga
Copy link
Collaborator

skynavga commented Aug 6, 2018

Technically, such a feature definition is not needed, the reason being is that the feature #length-root-container-relative is intended to operate as a mix-in feature, just like, e.g., #length-em #length-cell, etc., with all properties that employ a <length> expression, such as tts:outline.

I suggest closing this issue with no further action.

@skynavga skynavga changed the title tts:origin permits rw/rh but #origin is defined per TTML1 Property tts:origin permits rw/rh but #origin is defined per TTML1. Aug 6, 2018
@palemieux
Copy link
Contributor Author

Ok with me.

@skynavga
Copy link
Collaborator

skynavga commented Aug 6, 2018

Closing this per #972 (comment).

@nigelmegitt
Copy link
Contributor

@palemieux #972 (comment) seems to imply that if, in IMSC 1.1, we permit #origin and #length-root-container-relative, then the correct interpretation is that tts:origin values using rw or rh units need to be supported. I don't think this mix-in model is well described or clear, and I think we actually don't want that behaviour in IMSC1.1. So maybe a clarification action in IMSC1.1.

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