RGBCube/serenity
1
Fork 0
mirror of https://github.com/RGBCube/serenity synced 2025-07-25 14:27:35 +00:00
Code Issues Activity Actions

SpaceAnalyzer: Use raw_delta_y in TreeMapWidget::mousewheel_event

Browse source 
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.
This commit is contained in:
Arda Cinar 2022-12-06 18:18:39 +03:00 • committed by Sam Atkins
parent e0a904c852
commit f0b61e1443
1 changed files with 3 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

8
Userland/Applications/SpaceAnalyzer/TreeMapWidget.cpp
View file

@ -316,16 +316,14 @@ void TreeMapWidget::keydown_event(GUI::KeyEvent& event)
void TreeMapWidget::mousewheel_event(GUI::MouseEvent& event) void TreeMapWidget::mousewheel_event(GUI::MouseEvent& event)
{ {
int delta = event.wheel_delta_y(); int delta = event.wheel_raw_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();
if (delta > 0) { if (delta > 0) {
size_t step_back = delta / step_size; size_t step_back = delta;
if (step_back > m_viewpoint) if (step_back > m_viewpoint)
step_back = m_viewpoint; step_back = m_viewpoint;
set_viewpoint(m_viewpoint - step_back); set_viewpoint(m_viewpoint - step_back);
} else { } else {
size_t step_up = (-delta) / step_size; size_t step_up = -delta;
set_viewpoint(m_viewpoint + step_up); set_viewpoint(m_viewpoint + step_up);
} }
} }