Apply suggestions from code review

Co-authored-by: Florian Rivoal <git@florian.rivoal.net>

Author: andreubotella

css-overflow-4/Overview.bs

@@ -712,10 +712,11 @@ Indicating Block-Axis Overflow: the 'block-ellipsis' property</h3>
 	It only affects line boxes contained directly by the <a>block container</a> itself,
 	but as it inherits, will have an effect on descendants’ line boxes unless overridden.
 
-	This property only affects a line box if it immediately precedes
-	eiher a [=region break=] or a [=clamp point=] in the [=block formatting context=].
-	If the [=clamp point=] is placed at the end of the [=line-clamp container=], then the line will
-	not be affected.
+	This property only affects a line box
+	if it immediately precedes eiher a [=region break=]
+	or a [=clamp point=] in the [=block formatting context=].
+	If the [=clamp point=] is placed at the end of the [=line-clamp container=],
+	then the line will not be affected.
 
 	Note: See [[css-overflow-4#fragmentation]] for a way to generate boxes with such a [=region break=].
 
@@ -784,7 +785,8 @@ Indicating Block-Axis Overflow: the 'block-ellipsis' property</h3>
 	there is a subsequent <a>fragmentation container</a> in the [=fragmentation context=]
 	that would receive subsequent content,
 	then the content displaced by the <a>block overflow ellipsis</a>
-	must be pushed to that <a>fragmentation container</a>. If it is placed before a [=clamp point=],
+	must be pushed to that <a>fragmentation container</a>.
+	If it is placed before a [=clamp point=],
 	then the displaced content must be pushed to the remainder of the [=inline formatting context=].
 
 	The UA must treat the <a>block overflow ellipsis</a> as an unbreakable string,
@@ -849,6 +851,9 @@ Limiting Visible Lines: the 'line-clamp' shorthand property</h3>
 		<dt><dfn><<integer [1,∞]>></dfn>
 		<dt><dfn><'block-ellipsis'></dfn>
 		<dd>
+
+			Issue(7708): Whether this should imply ''discard'' or ''collapse'' is an unresolved question.
+                        
 			Sets 'continue' to ''collapse''
 			if either or both values ares specified.
 
@@ -959,19 +964,20 @@ Forcing a Break After a Set Number of Lines: the 'max-lines' property</h3>
 		Animation type: by computed value type
 	</pre>
 
-	If the value of 'max-lines' is not <dfn for=max-lines dfn-type=value>none</dfn>, then, where
-	<var>N</var> is the computed value of 'max-lines':
+	If the value of 'max-lines' is not <dfn for=max-lines dfn-type=value>none</dfn>,
+	then, given a [=computed value=] of <var>N</var>:
 
 	- If the box is a [=fragmentation container=] that captures [=region breaks=],
-	    a <a>region break</a> or
+	    a <a>region break</a>
 	    is forced after its <var>N</var>th
 	    descendant <a>in-flow</a> <a>line box</a>.
 	    If fewer than <var>N</var> line boxes exist,
 	    then 'max-lines' introduces no <a>region break</a>.
-	- If the box is a [=line-clamp container=], its [=clamp point=] is set after its <var>N</var>th
+	- If the box is a [=line-clamp container=],
+	its [=clamp point=] is set after its <var>N</var>th
 	    descendant <a>in-flow</a> <a>line box</a>.
-	    If fewer than <var>N</var> line boxes exist, then the clamp point is set to the end of the
-	    line-clamp container.
+	    If fewer than <var>N</var> line boxes exist,
+	    then the clamp point is set to the end of the line-clamp container.
 
 
 	Only line boxes in the same <a>Block Formatting Context</a>
@@ -1024,8 +1030,9 @@ Fragmentation of Overflow: the 'continue' property</h3>
 	</pre>
 
 	The 'continue' property gives authors the ability
-	to truncate/interrupt the content inside a box, by either visually hiding
-	the remaining content, or by turning the box into a [=fragmentation container=] (see [[!CSS-BREAK-3]])
+	to truncate/interrupt the content inside a box,
+	either by visually hiding the remaining content,
+	or by turning the box into a [=fragmentation container=] (see [[!CSS-BREAK-3]])
 	and discarding the content after the [=fragmentation break=].
 
 	Issue: ''continue: discard'' is meant to generalize and replace the <code>region-fragment</code> property from [[CSS-REGIONS-1]].
@@ -1134,8 +1141,8 @@ Fragmentation of Overflow: the 'continue' property</h3>
 	would cause intrinsic sizes to depend on layout, which would cause circularities.
 -->
 
-	Note: This differs from the ''visibility: hidden''-like behavior of content after the [=clamp
-	point=] with ''continue: collapse''.
+	Note: This differs from the ''visibility: hidden''-like behavior
+	of content after the [=clamp point=] with ''continue: collapse''.
 
 	Note: In the case of <a href="https://www.w3.org/TR/css-break-3/#parallel-flows">parallel fragmentation flows</a>,
 	content occurring after the <a>fragmentation break</a> in the box tree
@@ -1164,41 +1171,52 @@ Fragmentation of Overflow: the 'continue' property</h3>
 
 <h4 id=line-clamp-containers>Line-clamp containers</h4>
 
-	A <dfn>line-clamp container</dfn> is an [=independent formatting context|independent=] [=block formatting context=] that
-	additionally follows the rules in this section. All line-clamp containers contain a
-	<dfn>clamp point</dfn>, which is one of the following positions inside it:
-
-	- A point immediately after a [=line box=] in the line-clamp container's [=block formatting
-	  context=].
-	- The end of the line-clamp container, if there are any boxes after the last [=inline formatting
-	  context=] in the line-clamp container's [=block formatting context=].
-
-	If the [=line-clamp container=]'s [=block formatting context root=] has a [=computed value=] of
-	'max-lines' other than ''max-lines/none'', then that property will determine the [=clamp point=].
-	Otherwise, the clamp point will be set to the last possible clamp point such that, for it and
-	all previous possible clamp points, the line-clamp container's [=automatic block size=] (as
-	determined below) is not greater than the [=block size=] the box would have if its automatic
-	block size were infinite; or if that is not the case for any clamp points, to the first clamp
-	point in the block formatting context.
-
-	Any boxes in a [=line-clamp container=] that follow its [=clamp point=] in the box tree, as well
-	as any [=line boxes=] that follow it inside an [=inline formatting context=], will be invisible
-	(behaving like ''visibility: hidden''). This includes [=independent formatting contexts=] as
-	well as [=out-of-flow=] boxes, and all of their descendants. Any overflow such boxes and line
-	boxes might have is always counted as [=ink overflow=] rather than [=scrollable overflow=].
-
-	NOTE: This differs from the ''display: none''-like behavior of not rendered content with
-	''continue: discard''.
-
-	If a [=block container=] contains a [=clamp point=], within itself or in any of its descendants,
+	A <dfn>line-clamp container</dfn> is an [=independent formatting context|independent=] [=block formatting context=]
+	that additionally follows the rules in this section.
+	All line-clamp containers contain a <dfn>clamp point</dfn>,
+	which is one of the following positions inside it:
+
+	- A point immediately after a [=line box=]
+		in the line-clamp container's [=block formatting context=].
+	- The end of the line-clamp container,
+		if there are any boxes after the last [=inline formatting context=]
+		in the line-clamp container's [=block formatting context=].
+
+	If the [=line-clamp container=]'s [=block formatting context root=]
+	has a [=computed value=] of 'max-lines' other than ''max-lines/none'',
+	then that property will determine the [=clamp point=].
+	Otherwise, the clamp point will be set to the last possible clamp point
+	such that, for it and all previous possible clamp points,
+	the line-clamp container's [=automatic block size=]
+	(as determined below)
+	is not greater than the [=block size=] the box would have
+	if its automatic block size were infinite;
+	or if that is not the case for any possible clamp points,
+	to the first possible clamp point in the block formatting context.
+
+	Any boxes in a [=line-clamp container=] that follow its [=clamp point=] in the box tree,
+	as well as any [=line boxes=] that follow it inside an [=inline formatting context=],
+	will be invisible (behaving like ''visibility: hidden'').
+	This includes [=independent formatting contexts=]
+	as well as [=out-of-flow=] boxes,
+	and all of their descendants.
+	Any overflow such boxes and line boxes might have
+	is always counted as [=ink overflow=] rather than [=scrollable overflow=].
+
+	NOTE: This differs from the ''display: none''-like behavior
+	of not rendered content with ''continue: discard''.
+
+	If a [=block container=] contains a [=clamp point=],
+	within itself or in any of its descendants,
 	its [=automatic block size=] will not take into account any of its children after the clamp point.
 	This also applies for the [=line-clamp container=] itself.
 
-	NOTE: If there are any floats before the [=clamp point=], the [=line-clamp container=]'s
-	automatic size must grow to encompass the clearance, just like it would if it were a regular
-	[=block formatting context=] root that only contained its contents before the [=clamp point=].
-	This is regardless of whether this would cause any content after the [=clamp point=] to be
-	within the container's bounds.
+	NOTE: If there are any floats before the [=clamp point=],
+	the [=line-clamp container=]'s automatic size must grow to encompass the clearance,
+	just like it would if it were a regular [=block formatting context=] root
+	that only contained its contents before the [=clamp point=].
+	This is regardless of whether this would cause any content after the [=clamp point=]
+	to be within the container's bounds.
 
 <h2 id=sbg-ext class=nonum>
 Appendix A: Possible extensions for ''scrollbar-gutter''</h2>