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

Use of 'em' units in tts:fontSize on region element is not well defined. #216

Closed
palemieux opened this issue Nov 2, 2016 · 9 comments

Comments

Projects
None yet
3 participants
@palemieux
Copy link
Contributor

commented Nov 2, 2016

The resolution to w3c/ttml1#206 needs to be implemented here in order to remain compatible with TTML1 documents.

@palemieux palemieux added the bug label Nov 2, 2016

@nigelmegitt

This comment has been minimized.

Copy link
Contributor

commented Jan 12, 2017

Also incorporate the fix to w3c/ttml1#214.

@skynavga

This comment has been minimized.

Copy link
Collaborator

commented Feb 9, 2017

TTML2 will require a somewhat different treatment, since region inherits from tt, unlike TTML1, which doesn't allow inheritable styles on tt.

@skynavga

This comment has been minimized.

Copy link
Collaborator

commented Feb 22, 2017

Resolved in 8abbce4.

@palemieux

This comment has been minimized.

Copy link
Contributor Author

commented Mar 1, 2017

The definition styled element is not necessary (and confusing) since Style Resolution Process occurs in Synchronic Flow Processing, after Intermediate Synchronic Document Construction. Just as body elements are reparented to region elements, region elements can be reparented to tt, eleminating the need for styled element .

@palemieux palemieux reopened this Mar 1, 2017

@skynavga

This comment has been minimized.

Copy link
Collaborator

commented Mar 1, 2017

I'm re-closing this issue. Please open a new issue if you wish.

@skynavga skynavga closed this Mar 1, 2017

@palemieux palemieux reopened this Mar 1, 2017

@palemieux

This comment has been minimized.

Copy link
Contributor Author

commented Mar 1, 2017

For discussion at the next TTWG call.

@skynavga

This comment has been minimized.

Copy link
Collaborator

commented Mar 1, 2017

Closing again. If you want to open a new issue, please do so.

@skynavga skynavga closed this Mar 1, 2017

@nigelmegitt

This comment has been minimized.

Copy link
Contributor

commented Mar 2, 2017

[as Chair] @skynavga please remember that we have a 10 day review period - since @palemieux is requesting a change to the pull request that was merged early, and marked as such, it is reasonable to allow the issue to remain open until there is consensus on the edits. I am reopening this.

@nigelmegitt nigelmegitt reopened this Mar 2, 2017

@skynavga

This comment has been minimized.

Copy link
Collaborator

commented Mar 8, 2017

Closing again. The original resolution was not an early merge. Please file a new issue regarding use of "styled element".

@skynavga skynavga closed this Mar 8, 2017

@skynavga skynavga removed their assignment May 22, 2017

@skynavga skynavga added this to the Editor's WR Work List milestone May 25, 2017

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.