diff --git a/Userland/Libraries/LibWeb/Painting/PaintContext.cpp b/Userland/Libraries/LibWeb/Painting/PaintContext.cpp index 394ac15d50..86c40fe89a 100644 --- a/Userland/Libraries/LibWeb/Painting/PaintContext.cpp +++ b/Userland/Libraries/LibWeb/Painting/PaintContext.cpp @@ -5,6 +5,7 @@ * SPDX-License-Identifier: BSD-2-Clause */ +#include #include namespace Web { @@ -140,4 +141,9 @@ CSSPixelRect PaintContext::scale_to_css_rect(DevicePixelRect rect) const }; } +bool PaintContext::would_be_fully_clipped_by_painter(DevicePixelRect rect) const +{ + return !painter().clip_rect().intersects(rect.to_type().translated(painter().translation())); +} + } diff --git a/Userland/Libraries/LibWeb/Painting/PaintContext.h b/Userland/Libraries/LibWeb/Painting/PaintContext.h index cf5ef72e25..d8fd425147 100644 --- a/Userland/Libraries/LibWeb/Painting/PaintContext.h +++ b/Userland/Libraries/LibWeb/Painting/PaintContext.h @@ -35,6 +35,8 @@ public: void set_device_viewport_rect(DevicePixelRect const& rect) { m_device_viewport_rect = rect; } CSSPixelRect css_viewport_rect() const; + [[nodiscard]] bool would_be_fully_clipped_by_painter(DevicePixelRect) const; + bool has_focus() const { return m_focus; } void set_has_focus(bool focus) { m_focus = focus; } diff --git a/Userland/Libraries/LibWeb/Painting/PaintableBox.cpp b/Userland/Libraries/LibWeb/Painting/PaintableBox.cpp index f7e29f5889..e243c8475f 100644 --- a/Userland/Libraries/LibWeb/Painting/PaintableBox.cpp +++ b/Userland/Libraries/LibWeb/Painting/PaintableBox.cpp @@ -598,9 +598,12 @@ void PaintableWithLines::paint(PaintContext& context, PaintPhase phase) const for (auto& line_box : m_line_boxes) { for (auto& fragment : line_box.fragments()) { + auto fragment_absolute_rect = fragment.absolute_rect(); + auto fragment_absolute_device_rect = context.enclosing_device_rect(fragment_absolute_rect); + if (context.would_be_fully_clipped_by_painter(fragment_absolute_device_rect)) + continue; if (context.should_show_line_box_borders()) { - auto fragment_absolute_rect = fragment.absolute_rect(); - context.painter().draw_rect(context.enclosing_device_rect(fragment_absolute_rect).to_type(), Color::Green); + context.painter().draw_rect(fragment_absolute_device_rect.to_type(), Color::Green); context.painter().draw_line( context.rounded_device_point(fragment_absolute_rect.top_left().translated(0, fragment.baseline())).to_type(), context.rounded_device_point(fragment_absolute_rect.top_right().translated(-1, fragment.baseline())).to_type(), Color::Red);