beenull/ladybird
1
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2024-11-23 08:00:20 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 6

LibWeb: Flex, grid and table containers should not create BFC

Browse source 
Fixes https://github.com/SerenityOS/serenity/issues/16082
This commit is contained in:
Aliaksandr Kalenik 2022-11-25 06:44:33 +03:00 committed by Andreas Kling
parent 1944e8936d
commit 7bc7790912
Notes: sideshowbarker 2024-07-17 07:43:44 +09:00 
Author: https://github.com/kalenikaliaksandr
Commit: https://github.com/SerenityOS/serenity/commit/7bc7790912
Pull-request: https://github.com/SerenityOS/serenity/pull/16179
Issue: https://github.com/SerenityOS/serenity/issues/16082
1 changed files with 19 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

28
Userland/Libraries/LibWeb/Layout/FormattingContext.cpp
View file

@ -37,6 +37,21 @@ bool FormattingContext::creates_block_formatting_context(Box const& box)
if (box.is_replaced_box())
return false;
// display: table
if (box.display().is_table_inside()) {
return false;
}
// display: flex
if (box.display().is_flex_inside()) {
return false;
}
// display: grid
if (box.display().is_grid_inside()) {
return false;
}
// NOTE: This function uses MDN as a reference, not because it's authoritative,
// but because they've gathered all the conditions in one convenient location.
@ -85,16 +100,11 @@ bool FormattingContext::creates_block_formatting_context(Box const& box)
auto parent_display = box.parent()->display();
// Flex items (direct children of the element with display: flex or inline-flex) if they are neither flex nor grid nor table containers themselves.
if (parent_display.is_flex_inside()) {
if (!box.display().is_flex_inside())
return true;
}
if (parent_display.is_flex_inside())
return true;
// Grid items (direct children of the element with display: grid or inline-grid) if they are neither flex nor grid nor table containers themselves.
if (parent_display.is_grid_inside()) {
if (!box.display().is_grid_inside()) {
return true;
}
}
if (parent_display.is_grid_inside())
return true;
}
// FIXME: Multicol containers (elements where column-count or column-width isn't auto, including elements with column-count: 1).