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.

Sign up for GitHub

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

Remove applicability of tts:direction on <p> #1214

Closed
wants to merge 3 commits into from
Closed

Remove applicability of tts:direction on <p> #1214

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
ceb3a3f
Remove applicability of tts:direction on <p> (#1211)
palemieux Nov 12, 2020
18e8bf0
Address comments
palemieux Nov 12, 2020
a1e16a5
Remove applicability of tts:unicodeBidi on <p>
palemieux Nov 13, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
24 changes: 15 additions & 9 deletions spec/ttml2.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9787,8 +9787,8 @@ background color of the <loc href="#terms-root-container-region">root container
</div3>
<div3 id="style-attribute-direction">
<head>tts:direction</head>
<p>The <att>tts:direction</att> attribute is used to specify a style property that, depending on the context of use,
determines (1) the bidirectional paragraph level, or (2) the directionality of a bidirectional embedding or override,
<p>The <att>tts:direction</att> attribute is used to specify a style property that
determines the directionality of a bidirectional embedding or override,
about which see <bibref ref="uax9"/>.</p>
<table id="style-property-details-direction" role="common">
<col css="width: 25%;"/>
Expand All @@ -9810,7 +9810,6 @@ about which see <bibref ref="uax9"/>.</p>
<tr>
<td><emph>Applies to:</emph></td>
<td>
<loc href="#content-vocabulary-p"><el>p</el></loc>,
<loc href="#content-vocabulary-span"><el>span</el></loc>
</td>
</tr>
Expand All @@ -9832,11 +9831,13 @@ about which see <bibref ref="uax9"/>.</p>
</tr>
</tbody>
</table>
<p>When applied to a <el>p</el> element, the computed value of this property explicitly establishes the
<emph>paragraph level</emph> as specified by <bibref ref="uax9"/>, &sect;4.3, Higher Level Protocol <phrase role="strong">HL1</phrase>.</p>
<p>When applied to a <el>span</el> element (or <loc href="#terms-anonymous-span">anonymous span</loc>), the computed value of this property, in combination with the computed value
of the <att>tts:unicodeBidi</att> style property, determines the directionality of a bidirectional embedding or override as specified by
<bibref ref="uax9"/>, &sect;4.3, Higher Level Protocol <phrase role="strong">HL3</phrase>.</p>
<p>The computed value of this property, in combination with the computed value
of the <att>tts:unicodeBidi</att> style property, determines the directionality of a bidirectional
embedding or override as specified by <bibref ref="uax9"/>, &sect;4.3, Higher Level Protocol
<phrase role="strong">HL3</phrase>.</p>
<note role="elaboration">
<p>In earlier editions of this specification, the <att>tts:direction</att> property applied to the <code>p</code> element. This use is now <phrase role="deprecated">deprecated</phrase> in absence of definitive semantics. Authors are encouraged to avoid specifiying <att>tts:direction</att> on a <code>p</code> element, which can result in different rendering results depending on the implementation.</p>
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is inappropriate to deprecate functionality that has been deployed since TTML1 1st edition without providing a mechanism for an author to explicitly declare which behavior is desired (either the pre-existing behavior or the newly proposed, non-backwards compatible behavior). In other words, this change should not be treated as a candidate for deprecation.

</note>
<p>If a computed value of the property associated with this attribute is not supported,
then a <loc href="#terms-presentation-processor">presentation processor</loc> must use the value <code>ltr</code>.</p>
<p>The <att>tts:direction</att> style is illustrated by the following example.</p>
Expand Down Expand Up @@ -14152,7 +14153,6 @@ the Unicode bidirectional algorithm.</p>
<tr>
<td><emph>Applies to:</emph></td>
<td>
<loc href="#content-vocabulary-p"><el>p</el></loc>,
<loc href="#content-vocabulary-span"><el>span</el></loc>
</td>
</tr>
Expand All @@ -14176,6 +14176,9 @@ the Unicode bidirectional algorithm.</p>
</table>
<p>If a computed value of the property associated with this attribute is not supported,
then a <loc href="#terms-presentation-processor">presentation processor</loc> must use the value <code>normal</code>.</p>
<note role="elaboration">
<p>In earlier editions of this specification, the <att>tts:unicodeBidi</att> property applied to the <code>p</code> element. This use is now <phrase role="deprecated">deprecated</phrase> in absence of definitive semantics. Authors are encouraged to avoid specifiying <att>tts:unicodeBidi</att> on a <code>p</code> element, which can result in different rendering results depending on the implementation.</p>
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See above comment about deprecation.

</note>
<p>The <att>tts:unicodeBidi</att> style is illustrated by the following example.</p>
<table id="style-attribute-unicodeBidi-example-1" role="example">
<caption>Example Fragment &ndash; Unicode Bidirectionality</caption>
Expand Down Expand Up @@ -14465,6 +14468,9 @@ stacking block and inline areas within a region area.</p>
</tr>
</tbody>
</table>
<p>
</p>
<p>The <loc href="#terms-inline-progression-direction">inline progression direction</loc> determined by the computed value of this property explicitly establishes the <emph>paragraph level</emph> as specified by <bibref ref="uax9"/>, &sect;4.3, Higher Level Protocol <phrase role="strong">HL1</phrase>.</p>
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This language is semantically inconsistent with §10.2.12.1.

Copy link
Contributor Author

@palemieux palemieux Nov 19, 2020 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is the semantic inconsistency?

<p>If a computed value of the property associated with this attribute is not supported,
then a <loc href="#terms-presentation-processor">presentation processor</loc> must use the value <code>lrtb</code>.</p>
<note role="elaboration">
Expand Down