Skip to content

Commit

Permalink
[css-text-4] Drop keyword and mark an open issue, since there's no co…
Browse files Browse the repository at this point in the history
…nsensus on what syntax is or isn't necessary for good text-wrapping algorithms. #672
  • Loading branch information
fantasai committed Sep 16, 2018
1 parent 06a6298 commit a0c27af
Showing 1 changed file with 3 additions and 13 deletions.
16 changes: 3 additions & 13 deletions css-text-4/Overview.bs
Original file line number Diff line number Diff line change
Expand Up @@ -172,7 +172,7 @@ Text Wrap Settings: the 'text-wrap' property</h3>

<pre class="propdef">
Name: text-wrap
Value: wrap | nowrap | balance | multi-line
Value: wrap | nowrap | balance
Initial: wrap
Applies to: <a>inline boxes</a> and <a>block containers</a>
Inherited: yes
Expand Down Expand Up @@ -223,20 +223,10 @@ Text Wrap Settings: the 'text-wrap' property</h3>
The exact algorithm is UA-defined.

UAs may treat this value as ''text-wrap/wrap'' if there are more than ten lines to balance.

<dt><dfn>multi-line</dfn>
<dd>
Same as ''text-wrap/wrap'' for <a>inline boxes</a>.
Same as ''text-wrap/wrap'' for block-level elements,
except as below.

The exact algorithm is UA-defined.
The algorithm should consider multiple lines when making break decisions.

This comment has been minimized.

Copy link
@astearns

astearns Feb 27, 2019

Member

These two lines (consider/bias) are present in the definition of the wrap value, and should probably be removed there too.

The UA should bias for best layout over speed.

ISSUE: This feature does not have CSSWG consensus, it is being proposed in <a href="https://github.com/w3c/csswg-drafts/issues/672#issuecomment-379723243">Issue 672</a>.
</dl>

ISSUE: There's some discussion about whether different wrapping algorithms (higher quality vs faster) need special keywords, see <a href="https://github.com/w3c/csswg-drafts/issues/672#issuecomment-379723243">Issue 672</a>.

Regardless of the 'text-wrap' value,
lines always break at forced breaks:
for all values,
Expand Down

0 comments on commit a0c27af

Please sign in to comment.