251178 2023-01-25 15:11:09 -0800 Whitespace incorrectly suppressed inside table. 2025-11-24 00:20:22 -0800 1 1 1 Unclassified WebKit Layout and Rendering Other Unspecified Unspecified NEW http://wpt.live/css/css-tables/whitespace-001.html BrowserCompat, InRadar, WPTImpact P2 Normal --- 1 emilio karlcow ahmad.saleem792 bfulgham karlcow simon.fraser webkit-bug-importer zalan oldest_to_newest 1928292 0 emilio 2023-01-25 15:11:09 -0800 In https://bug1811353.bmoattachments.org/attachment.cgi?id=9314059, Blink and WebKit render tables inconsistently from Gecko and the spec. See also https://github.com/w3c/csswg-drafts/issues/8358 and https://bugs.chromium.org/p/chromium/issues/detail?id=1410345 for context, spec references, and equivalent Chromium bug. 1928351 1 karlcow 2023-01-25 17:45:09 -0800 would it be https://searchfox.org/wubkat/rev/1c8b29ff201022e3a20578244e85d25d1b7706c4/Source/WebCore/rendering/updating/RenderTreeUpdater.cpp#429-431 1928360 2 emilio 2023-01-25 18:20:26 -0800 Yep that looks exactly like it. 1930291 3 webkit-bug-importer 2023-02-01 15:12:25 -0800 <rdar://problem/104934417> 1940029 4 ahmad.saleem792 2023-03-09 05:12:47 -0800 (In reply to Karl Dubost from comment #1) > would it be > https://searchfox.org/wubkat/rev/1c8b29ff201022e3a20578244e85d25d1b7706c4/ > Source/WebCore/rendering/updating/RenderTreeUpdater.cpp#429-431 I tested locally by removing just "parentRenderer.isTable()" make us pass "WPT" test, if we remove other bits, it makes tables go haywire. If you want to do PR, you can do so or if you want me to do, I am happy to do it as well. Thanks! 1940053 5 karlcow 2023-03-09 07:24:21 -0800 Ahmad, feel free to give it a go. 2125696 6 karlcow 2025-06-24 23:21:02 -0700 Pull request: https://github.com/WebKit/WebKit/pull/47169 2160999 7 ahmad.saleem792 2025-11-24 00:20:22 -0800 auto* previousRenderer = renderingParent.previousChildRenderer; if (parentRenderer.isRenderTableRow() || parentRenderer.isRenderTableSection() || parentRenderer.isRenderTableCol() || parentRenderer.isRenderFrameSet() || parentRenderer.isRenderGrid() || (parentRenderer.isRenderFlexibleBox() && !parentRenderer.isRenderButton())) return false; if (parentRenderer.isRenderTable()) { if (!previousRenderer) return false; // Suppress whitespace after table-internal or block-level elements if (previousRenderer->isRenderTableRow() || previousRenderer->isRenderTableSection() || previousRenderer->isRenderTableCol() || previousRenderer->isRenderTableCaption()) return false; // Preserve whitespace only if previous is inline-block or inline-table // (not regular table-cells or blocks) if (previousRenderer->isNonReplacedAtomicInlineLevelBox()) return true; // Keep the whitespace // Otherwise suppress (covers table-cells, blocks, etc.) return false; } This takes us to closet to our progress but still fails: https://jsfiddle.net/6Lbjd3qu/