From 9bc7912f8485bde997c032bd7c08c74c409a3c15 Mon Sep 17 00:00:00 2001 From: Linus Groh Date: Sat, 26 Mar 2022 00:53:31 +0000 Subject: [PATCH] LibWeb: Paint the focus outline using Painter::draw_focus_rect() Now it actually looks like a classic focus outline and not some misplaced border :^) --- Userland/Libraries/LibWeb/Painting/PaintableBox.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/Userland/Libraries/LibWeb/Painting/PaintableBox.cpp b/Userland/Libraries/LibWeb/Painting/PaintableBox.cpp index e8e085aa25..5d13f85c57 100644 --- a/Userland/Libraries/LibWeb/Painting/PaintableBox.cpp +++ b/Userland/Libraries/LibWeb/Painting/PaintableBox.cpp @@ -156,7 +156,7 @@ void PaintableBox::paint(PaintContext& context, PaintPhase phase) const if (phase == PaintPhase::FocusOutline && layout_box().dom_node() && layout_box().dom_node()->is_element() && verify_cast(*layout_box().dom_node()).is_focused()) { // FIXME: Implement this as `outline` using :focus-visible in the default UA stylesheet to make it possible to override/disable. auto focus_outline_rect = enclosing_int_rect(absolute_border_box_rect()).inflated(4, 4); - context.painter().draw_rect(focus_outline_rect, context.palette().focus_outline()); + context.painter().draw_focus_rect(focus_outline_rect, context.palette().focus_outline()); } } @@ -468,7 +468,7 @@ void PaintableWithLines::paint(PaintContext& context, PaintPhase phase) const if (parent->is_focused()) { // FIXME: Implement this as `outline` using :focus-visible in the default UA stylesheet to make it possible to override/disable. auto focus_outline_rect = enclosing_int_rect(fragment.absolute_rect()).inflated(4, 4); - context.painter().draw_rect(focus_outline_rect, context.palette().focus_outline()); + context.painter().draw_focus_rect(focus_outline_rect, context.palette().focus_outline()); } } }