From 56d1ae98bc268db5007b9594953cff0f2addc5c8 Mon Sep 17 00:00:00 2001 From: Andreas Kling Date: Sat, 5 Oct 2019 21:51:28 +0200 Subject: [PATCH] LibHTML: Always fill the entire paint event rect with background And do the filling before translating the painter. This fixes a bug where scrolling down in the Help app would render the bottom part of the page with a different background color. --- Libraries/LibHTML/HtmlView.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/Libraries/LibHTML/HtmlView.cpp b/Libraries/LibHTML/HtmlView.cpp index 596caed927..f63b67d67d 100644 --- a/Libraries/LibHTML/HtmlView.cpp +++ b/Libraries/LibHTML/HtmlView.cpp @@ -85,11 +85,11 @@ void HtmlView::paint_event(GPaintEvent& event) return; } + painter.fill_rect(event.rect(), m_document->background_color()); + painter.translate(frame_thickness(), frame_thickness()); painter.translate(-horizontal_scrollbar().value(), -vertical_scrollbar().value()); - painter.fill_rect(rect(), m_document->background_color()); - RenderingContext context { painter }; m_layout_root->render(context); }