test(margin): fix top expectation in margin_not_collapse_if_padding_exists_2

TtTRz · TtTRz · commit 8df582edd920 · 2026-07-21T20:11:16.000+08:00
The outer wrapper has border:1px, so the child layout top = parent border (1) + margin-top (20) = 21, not 20. Corrected data-expect-top 20-&gt;21 and removed the temporary data-ignore. Other assertions (n3/n4 top via padding/margin, n2 height 90) were already correct.
diff --git a/float-pigment-forest/tests/cases/custom_css_margin/margin_not_collapse_if_padding_exists_2.html b/float-pigment-forest/tests/cases/custom_css_margin/margin_not_collapse_if_padding_exists_2.html
@@ -1,8 +1,6 @@
 <!DOCTYPE html>
-<!-- FIXME: layout-top expectations predate the style-attribute fixup; float-pigment
-     layout top includes the parent border (21 not 20, etc.). Needs re-derivation. Ignored meanwhile. -->
-<div style="border: 1px;" data-chrome="false" data-ignore="true">
-          <div style="padding-top: 30px; border-bottom: 10px; margin-top: 20px; margin-bottom: 20px;" data-expect-top="20" data-expect-height="90">
+<div style="border: 1px;" data-chrome="false">
+          <div style="padding-top: 30px; border-bottom: 10px; margin-top: 20px; margin-bottom: 20px;" data-expect-top="21" data-expect-height="90">
             <div style="margin-top: 10px; margin-bottom: 10px; width: 10px; height: 10px;" data-expect-top="40"></div>
             <div style="margin-top: 5px; margin-bottom: 10px; width: 10px; height: 10px;" data-expect-top="60"></div>
           </div>
