Pārlūkot izejas kodu

LibWeb: Ignore `font-size: calc(...)` for now

This doesn't work correctly in the new world where fonts are resolved
during the CSS cascade. Let's patch it out with a FIXME and get back to
it once everything has fallen into place.
Andreas Kling 3 gadi atpakaļ
vecāks
revīzija
71f371f6b1
1 mainītis faili ar 6 papildinājumiem un 3 dzēšanām
  1. 6 3
      Userland/Libraries/LibWeb/CSS/StyleComputer.cpp

+ 6 - 3
Userland/Libraries/LibWeb/CSS/StyleComputer.cpp

@@ -735,9 +735,12 @@ void StyleComputer::compute_font(StyleProperties& style, DOM::Element const* ele
             maybe_length = length;
         }
         if (maybe_length.has_value()) {
-            auto calculated_size = maybe_length.value().to_px(viewport_rect, font_metrics, root_font_size);
-            if (calculated_size != 0)
-                size = calculated_size;
+            // FIXME: Support font-size: calc(...)
+            if (!maybe_length->is_calculated()) {
+                auto px = maybe_length.value().to_px(viewport_rect, font_metrics, root_font_size);
+                if (px != 0)
+                    size = px;
+            }
         }
     }