-
Notifications
You must be signed in to change notification settings - Fork 16
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
Add display alignment and fix timing in examples in Appendix S (#459). #520
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The issue discussion and resolutions were possibly unclear here, but I think we reached the point where we agreed only to change the roll up example in S.2. The change made here isn't quite right, so requesting reversions to the paint on and pop on examples and a fix to the roll up example.
spec/examples/ex2-pop-on.xml
Outdated
<region xml:id="r1" tts:color="white" tts:origin="10c 4c" tts:extent="40c 1c" | ||
tts:displayAlign="after"/> | ||
<region xml:id="r2" tts:color="yellow" tts:origin="10c 8c" tts:extent="40c 1c" | ||
tts:displayAlign="after"/> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Modifying the Pop On example makes no difference to this example, and we agreed not to change this one. I think it better to revert the changes to this example. The descriptive paragraph above includes:
Each paragraph is non-overlapping in time and appears in the same single row of its targeted region.
spec/examples/ex2-roll-up.xml
Outdated
<p begin="14s" end="25s">et dolore magna aliqua</p> | ||
<p begin="18s" end="29s">Ut enim ad minim veniam quis, nostrud</p> | ||
<p> | ||
<span begin="0s" end="8s">Lorem ipsum dolor sit<br/></span> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't the additional <br/>
elements be at the beginning of all but the first <span>
rather than at the end of all of them? Otherwise the first <span>
is never on the bottom line, since the <br/>
always bumps it up one. The text is only ever bumped up by the next line becoming active.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@nigelmegitt an empty line area at the after edge of a block is always elided, so using <br/>
at the ends of lines suffices (and looks better in example code).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@skynavga I wondered if that is supposed to be the case. I'm aware that some documents use multiple trailing <br/>
s in a region with tts:displayAlign="after"
to position subtitles some lines off the bottom of the region. Does that mean that never works?
This technique is one of the strategies defined in EBU Tech3360 - see the example at §4.5.6.3.1 for example. This suggests that at least some implementations do not elide in the way that you say.
Since this is unclear in the world (regardless of what is correct according to any specifications), I would prefer to avoid the confusion, by moving the <br/>
s in this example to go at the beginning of each <p>
other than the first one.
spec/examples/ex2-paint-on.xml
Outdated
@@ -2,7 +2,8 @@ | |||
xmlns:ttp="http://www.w3.org/ns/ttml#parameter" xmlns:tts="http://www.w3.org/ns/ttml#styling"> | |||
<head> | |||
<layout> | |||
<region xml:id="r1" tts:color="white" tts:origin="10c 4c" tts:extent="40c 5c"/> | |||
<region xml:id="r1" tts:color="white" tts:origin="10c 4c" tts:extent="40c 5c" | |||
tts:displayAlign="after"/> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the consensus was not to modify the paint on example. The presentation does depend on the value of displayAlign
but demonstrating the paint on effect does not. It works equally well with before
and after
but arguably, since the region is defined near the top, the default before
makes sense here. Otherwise the example gets mixed with roll up. I think it better to revert this change.
@nigelmegitt please check with fixes |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Recent changes are helpful, however since we discussed this at the f2f I've found evidence (mentioned in the comment) about usage practice that suggests it would be useful to change the roll up example's <br/>
positioning.
spec/examples/ex2-roll-up.xml
Outdated
<p begin="14s" end="25s">et dolore magna aliqua</p> | ||
<p begin="18s" end="29s">Ut enim ad minim veniam quis, nostrud</p> | ||
<p> | ||
<span begin="0s" end="8s">Lorem ipsum dolor sit<br/></span> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@skynavga I wondered if that is supposed to be the case. I'm aware that some documents use multiple trailing <br/>
s in a region with tts:displayAlign="after"
to position subtitles some lines off the bottom of the region. Does that mean that never works?
This technique is one of the strategies defined in EBU Tech3360 - see the example at §4.5.6.3.1 for example. This suggests that at least some implementations do not elide in the way that you say.
Since this is unclear in the world (regardless of what is correct according to any specifications), I would prefer to avoid the confusion, by moving the <br/>
s in this example to go at the beginning of each <p>
other than the first one.
@nigelmegitt on further checking, I see that we map
which has semantics
I need to verify behavior on an XSL-FO implementation, but it may be that the start of line or end of line will produce the same behavior. |
Thanks for digging into that @skynavga - this confirms my view that we need to remove trailing |
@nigelmegitt please check |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, looks good to me. Good to learn something too.
Closes #459.