From 49916f494dde50d86b2af83bae3f5938e9b12911 Mon Sep 17 00:00:00 2001
From: Andreas Kling <kling@serenityos.org>
Date: Thu, 28 Oct 2021 19:35:54 +0200
Subject: [PATCH] LibWeb: Don't try to layout list-item markers in BFC

Marker boxes are laid out by the corresponding ListItemBox. BFC should
just leave them alone. This fixes a jiggling issue on welcome.html :^)
---
 Userland/Libraries/LibWeb/Layout/BlockFormattingContext.cpp | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/Userland/Libraries/LibWeb/Layout/BlockFormattingContext.cpp b/Userland/Libraries/LibWeb/Layout/BlockFormattingContext.cpp
index 34985f7633..84b8796ec2 100644
--- a/Userland/Libraries/LibWeb/Layout/BlockFormattingContext.cpp
+++ b/Userland/Libraries/LibWeb/Layout/BlockFormattingContext.cpp
@@ -12,6 +12,7 @@
 #include <LibWeb/Layout/InitialContainingBlock.h>
 #include <LibWeb/Layout/InlineFormattingContext.h>
 #include <LibWeb/Layout/ListItemBox.h>
+#include <LibWeb/Layout/ListItemMarkerBox.h>
 #include <LibWeb/Layout/ReplacedBox.h>
 #include <LibWeb/Page/BrowsingContext.h>
 
@@ -387,6 +388,10 @@ void BlockFormattingContext::layout_block_level_children(BlockContainer& block_c
         if (child_box.is_absolutely_positioned())
             return IterationDecision::Continue;
 
+        // NOTE: ListItemMarkerBoxes are placed by their corresponding ListItemBox.
+        if (is<ListItemMarkerBox>(child_box))
+            return IterationDecision::Continue;
+
         if (child_box.is_floating()) {
             layout_floating_child(child_box, block_container);
             return IterationDecision::Continue;