From f0b61e14436d27ccd632e73f407872a1892a8940 Mon Sep 17 00:00:00 2001 From: Arda Cinar Date: Tue, 6 Dec 2022 18:18:39 +0300 Subject: [PATCH] SpaceAnalyzer: Use raw_delta_y in TreeMapWidget::mousewheel_event There was a FIXME about using raw delta y value of the mousewheel event in TreeMapWidget::mousewheel_event. Some time after that code was written, a raw delta x/y API was added to GUI::MouseEvent. This patch simply uses that API and removes the FIXME message there. --- Userland/Applications/SpaceAnalyzer/TreeMapWidget.cpp | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/Userland/Applications/SpaceAnalyzer/TreeMapWidget.cpp b/Userland/Applications/SpaceAnalyzer/TreeMapWidget.cpp index 90dfeffc90..55f214bdb0 100644 --- a/Userland/Applications/SpaceAnalyzer/TreeMapWidget.cpp +++ b/Userland/Applications/SpaceAnalyzer/TreeMapWidget.cpp @@ -316,16 +316,14 @@ void TreeMapWidget::keydown_event(GUI::KeyEvent& event) void TreeMapWidget::mousewheel_event(GUI::MouseEvent& event) { - int delta = event.wheel_delta_y(); - // FIXME: The wheel_delta_y is premultiplied in the window server, we actually want a raw value here. - int step_size = GUI::ConnectionToWindowServer::the().get_scroll_step_size(); + int delta = event.wheel_raw_delta_y(); if (delta > 0) { - size_t step_back = delta / step_size; + size_t step_back = delta; if (step_back > m_viewpoint) step_back = m_viewpoint; set_viewpoint(m_viewpoint - step_back); } else { - size_t step_up = (-delta) / step_size; + size_t step_up = -delta; set_viewpoint(m_viewpoint + step_up); } }