Skip to content

Commit b752cd9

Browse files
committed
[css-rhythm] Revert the addition of 'none' to 'line-height-step'
As per the discussion in w3c#1151. The reason we added is gone, and we can add it back if the reason came back.
1 parent b8ac85c commit b752cd9

File tree

1 file changed

+3
-4
lines changed

1 file changed

+3
-4
lines changed

css-rhythm-1/Overview.bs

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -350,14 +350,14 @@ Adjusting Line Box Heights: the 'line-height-step' property {#line-height-step}
350350

351351
<pre class='propdef'>
352352
Name: line-height-step
353-
Value: none | <<length>>
354-
Initial: none
353+
Value: <<length>>
354+
Initial: 0
355355
Applies to: block containers
356356
Inherited: yes
357357
Animatable: no
358358
Percentages: N/A
359359
Media: visual
360-
Computed Value: the absolute length, 0 for none
360+
Computed Value: the absolute length
361361
</pre>
362362

363363
This property defines the <dfn>step unit</dfn> for line box heights.
@@ -392,7 +392,6 @@ by making heights of all lines an integer multiple of the <a>step unit</a>.
392392
<p class="caption">Rounding up the computed line box height.
393393
</div>
394394

395-
<div class="issue">''line-height-step/none'' and ''0'' are equivalent. Is this ok?</div>
396395
<div class="issue">Should this be animatable?
397396
There doesn't seem to be use cases but needed for consistency?</div>
398397

0 commit comments

Comments
 (0)