From 8d05ff1fe7832599aeb81630d19ea394eff02e4e Mon Sep 17 00:00:00 2001 From: Aliaksandr Kalenik Date: Tue, 5 Sep 2023 15:21:10 +0200 Subject: [PATCH] LibWeb: Print FIXME if vertical-align value is not supported in TFC Producing incorrect layout for table cells is better than crashing :) --- .../LibWeb/Layout/TableFormattingContext.cpp | 16 ++++++++++++++++ 1 file changed, 16 insertions(+) diff --git a/Userland/Libraries/LibWeb/Layout/TableFormattingContext.cpp b/Userland/Libraries/LibWeb/Layout/TableFormattingContext.cpp index ddb8c20182..5cd7c91927 100644 --- a/Userland/Libraries/LibWeb/Layout/TableFormattingContext.cpp +++ b/Userland/Libraries/LibWeb/Layout/TableFormattingContext.cpp @@ -1102,6 +1102,22 @@ void TableFormattingContext::position_cell_boxes() cell_state.padding_bottom += height_diff; break; } + case CSS::VerticalAlign::Sub: { + dbgln("FIXME: Implement \"vertical-align: sub\" support for table cells"); + break; + } + case CSS::VerticalAlign::Super: { + dbgln("FIXME: Implement \"vertical-align: super\" support for table cells"); + break; + } + case CSS::VerticalAlign::TextBottom: { + dbgln("FIXME: Implement \"vertical-align: text-bottom\" support for table cells"); + break; + } + case CSS::VerticalAlign::TextTop: { + dbgln("FIXME: Implement \"vertical-align: text-top\" support for table cells"); + break; + } default: VERIFY_NOT_REACHED(); }