w3c / ttml2

Timed Text Markup Language 2 (TTML2)
https://w3c.github.io/ttml2/
Other
41 stars 16 forks source link

Percentage and writing mode relative edges apply to tts:padding #144

Closed skynavga closed 7 years ago

skynavga commented 8 years ago

Since TTML2 introduces padding on content elements (body, div, p, span), a better definition of percentages and edges is required.

nigelmegitt commented 7 years ago

There's no issue reference for this in the TTML2 WD - is this still current or has it been actioned but not referenced (question for @skynavga)?

skynavga commented 7 years ago

Not referenced but still actionable.

On Fri, Dec 16, 2016 at 11:13 AM, Nigel Megitt notifications@github.com wrote:

There's no issue reference for this in the TTML2 WD - is this still current or has it been actioned but not referenced (question for @skynavga https://github.com/skynavga)?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/w3c/ttml2/issues/144#issuecomment-267659378, or mute the thread https://github.com/notifications/unsubscribe-auth/AAXCb0Tn-zM806ayXmcw0-tNmEnG3h5Eks5rItS4gaJpZM4G0W6k .

palemieux commented 7 years ago

Since TTML2 introduces padding on content elements (body, div, p, span), a better definition of percentages and edges is required.

Why is padding necessary of elements other than <region>? Are the underlying use cases captured somewhere?

skynavga commented 7 years ago

Well, that is how TTML2 supports linePadding, i.e., as padding on span.

On Sat, Dec 17, 2016 at 3:48 PM, Pierre-Anthony Lemieux < notifications@github.com> wrote:

Since TTML2 introduces padding on content elements (body, div, p, span), a better definition of percentages and edges is required.

Why is padding necessary of elements other than ? Are the underlying use cases captured somewhere?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/w3c/ttml2/issues/144#issuecomment-267792061, or mute the thread https://github.com/notifications/unsubscribe-auth/AAXCbzupwVl39PBeLK-wBi_y7G7v12Paks5rJGahgaJpZM4G0W6k .

palemieux commented 7 years ago

Well, that is how TTML2 supports linePadding, i.e., as padding on span.

As noted at https://lists.w3.org/Archives/Public/public-tt/2016Dec/0021.html, linePadding is not padding on <span>, but padding on lines. In other words, applying linePadding to a paragraph containing two spans (whether nested or consecutive), does not yield additional space between the spans.

skynavga commented 7 years ago

I didn't say which span, but I mean an anonymous span that is synthesized to wrap the entire content of the paragraph.

On Sat, Dec 17, 2016 at 4:56 PM, Pierre-Anthony Lemieux < notifications@github.com> wrote:

Well, that is how TTML2 supports linePadding, i.e., as padding on span.

As noted at https://lists.w3.org/Archives/Public/public-tt/2016Dec/0021. html, linePadding is not padding on , but padding on lines. In other words, applying linePadding to a paragraph containing two spans (whether nested or consecutive), does not yield additional space between the spans.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/w3c/ttml2/issues/144#issuecomment-267794736, or mute the thread https://github.com/notifications/unsubscribe-auth/AAXCbzwZ__jA5NH6gJPHHoNbmJYtZDydks5rJHalgaJpZM4G0W6k .

palemieux commented 7 years ago

I didn't say which span, but I mean an anonymous span that is synthesized to wrap the entire content of the paragraph.

This will not work is there is more than one span in the paragraph, and each one has a different background color.

skynavga commented 7 years ago

On Sat, Dec 17, 2016 at 9:44 PM, Pierre-Anthony Lemieux < notifications@github.com> wrote:

I didn't say which span, but I mean an anonymous span that is synthesized to wrap the entire content of the paragraph.

This will not work is there is more than one span in the paragraph, and each one has a different background color.

Not necessarily. It depends on how we define behavior at line boundaries. We are not obligated to limit ourselves to CSS tools to do this.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/w3c/ttml2/issues/144#issuecomment-267803218, or mute the thread https://github.com/notifications/unsubscribe-auth/AAXCb0j35TokUNpyPE5XDYymzgaghaosks5rJLoTgaJpZM4G0W6k .

palemieux commented 7 years ago

I see no value in extending tts:padding to all content elements only to satisfy the narrow use case of linePadding, where in any case tts:padding would behave differently than it does on region. Using linePadding as it is used in EBU-TT-D and IMSC would achieve the same result, with fewer implementation requirements (no need to support padding outside of region) and smoother transition from EBU-TT-D and IMSC.

skynavga commented 7 years ago

This has been a long standing ask, along with border on content elements, that goes back at least 3 years, which has effectively been provisionally adopted by the group. If there are insufficient implementations to support going to PR with this feature, then that will be the time to remove it.

On Sat, Dec 17, 2016 at 10:02 PM, Pierre-Anthony Lemieux < notifications@github.com> wrote:

I see no value in extending tts:padding to all content elements only to satisfy the narrow use case of linePadding, where in any case tts:padding would behave differently than it does on region. Using linePadding as it is used in EBU-TT-D and IMSC would achieve the same result, with fewer implementation requirements (no need to support padding outside of region) and smoother transition from EBU-TT-D and IMSC.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/w3c/ttml2/issues/144#issuecomment-267803669, or mute the thread https://github.com/notifications/unsubscribe-auth/AAXCbwwDG_aLrwOI_sg9SH6lFrz-Kscuks5rJL5igaJpZM4G0W6k .

skynavga commented 7 years ago

I should also add that, since this is supported by XSL-FO and CSS, we can't argue against it due to lack of CSS support.

On Sat, Dec 17, 2016 at 10:06 PM, Glenn Adams glenn@skynav.com wrote:

This has been a long standing ask, along with border on content elements, that goes back at least 3 years, which has effectively been provisionally adopted by the group. If there are insufficient implementations to support going to PR with this feature, then that will be the time to remove it.

On Sat, Dec 17, 2016 at 10:02 PM, Pierre-Anthony Lemieux < notifications@github.com> wrote:

I see no value in extending tts:padding to all content elements only to satisfy the narrow use case of linePadding, where in any case tts:padding would behave differently than it does on region. Using linePadding as it is used in EBU-TT-D and IMSC would achieve the same result, with fewer implementation requirements (no need to support padding outside of region) and smoother transition from EBU-TT-D and IMSC.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/w3c/ttml2/issues/144#issuecomment-267803669, or mute the thread https://github.com/notifications/unsubscribe-auth/AAXCbwwDG_aLrwOI_sg9SH6lFrz-Kscuks5rJL5igaJpZM4G0W6k .

palemieux commented 7 years ago

This has been a long standing ask, along with border on content elements, that goes back at least 3 years,

Can you point to the use case?

I should also add that, since this is supported by XSL-FO and CSS, we can't argue against it due to lack of CSS support.

Right.

It is however possible to argue against adding CSS padding to meet the linePadding use case as currently specified in IMSC and EBU-TT-D since CSS padding does not in fact yield the expected result.

skynavga commented 7 years ago

On Sat, Dec 17, 2016 at 10:13 PM, Pierre-Anthony Lemieux < notifications@github.com> wrote:

This has been a long standing ask, along with border on content elements, that goes back at least 3 years,

Can you point to the use case?

adding padding

I should also add that, since this is supported by XSL-FO and CSS, we can't argue against it due to lack of CSS support.

Right.

It is however possible to argue against adding CSS padding to meet the linePadding use case as currently specified in IMSC and EBU-TT-D since CSS padding does not in fact yield the expected result.

supporting line padding is not the sole use case

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/w3c/ttml2/issues/144#issuecomment-267803922, or mute the thread https://github.com/notifications/unsubscribe-auth/AAXCb_Cw_WKuUvM2cyeIae1RfYsGBd62ks5rJMDlgaJpZM4G0W6k .

palemieux commented 7 years ago

Can you point to the use case? adding padding

What style of subtitles/captions require padding beyond region?

supporting line padding is not the sole use case

It is in fact a different use case since it does not behave like CSS padding.

skynavga commented 7 years ago

On Sat, Dec 17, 2016 at 10:24 PM, Pierre-Anthony Lemieux < notifications@github.com> wrote:

Can you point to the use case? adding padding

What style of subtitles/captions require padding beyond region?

those that wish to use padding on content elements; keep in mind that TTML is not restricted to subtitles and captions

supporting line padding is not the sole use case

It is in fact a different use case since it does not behave like CSS padding.

not so, since we haven't yet defined background behavior at line boundaries

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/w3c/ttml2/issues/144#issuecomment-267804219, or mute the thread https://github.com/notifications/unsubscribe-auth/AAXCb97SzIGa6RlIDsWmIvA9nD8-K39dks5rJMN0gaJpZM4G0W6k .

nigelmegitt commented 7 years ago

Marking as discussed and agreed as per the discussion in w3c/ttml1#205.

skynavga commented 7 years ago

Duplicate of #302.