From 786494b62dd6f369e6758f33ac4414403b8e833c Mon Sep 17 00:00:00 2001 From: Andreas Kling Date: Thu, 3 Oct 2019 09:17:27 +0200 Subject: [PATCH] LibHTML: Adjust mouse event positions before using them for hit testing This fixes the issue where you couldn't click on links at the bottom of scrollable documents. :^) --- 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 1f5c96bdc6..b3d277602b 100644 --- a/Libraries/LibHTML/HtmlView.cpp +++ b/Libraries/LibHTML/HtmlView.cpp @@ -88,7 +88,7 @@ void HtmlView::mousemove_event(GMouseEvent& event) return GScrollableWidget::mousemove_event(event); bool hovered_node_changed = false; - auto result = m_layout_root->hit_test(event.position()); + auto result = m_layout_root->hit_test(to_content_position(event.position())); if (result.layout_node) { auto* node = result.layout_node->node(); hovered_node_changed = node != m_document->hovered_node(); @@ -121,7 +121,7 @@ void HtmlView::mousedown_event(GMouseEvent& event) return GScrollableWidget::mousemove_event(event); bool hovered_node_changed = false; - auto result = m_layout_root->hit_test(event.position()); + auto result = m_layout_root->hit_test(to_content_position(event.position())); if (result.layout_node) { auto* node = result.layout_node->node(); hovered_node_changed = node != m_document->hovered_node();