From 405b282bc3e102b4078e90b57d56996d7a028d80 Mon Sep 17 00:00:00 2001 From: Mustafa Quraish Date: Mon, 6 Sep 2021 22:16:17 -0400 Subject: [PATCH] PixelPaint: Show mouse position in statusbar without active tool Showing the position only with an active tool seems a bit confusing, if you've opened up an image just to find out the coordinates of a pixel for instance, there shouldn't be a need to have to select a tool first. --- Userland/Applications/PixelPaint/ImageEditor.cpp | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/Userland/Applications/PixelPaint/ImageEditor.cpp b/Userland/Applications/PixelPaint/ImageEditor.cpp index 441fe8262e..2bb4bd0bd5 100644 --- a/Userland/Applications/PixelPaint/ImageEditor.cpp +++ b/Userland/Applications/PixelPaint/ImageEditor.cpp @@ -213,17 +213,17 @@ void ImageEditor::mousemove_event(GUI::MouseEvent& event) return; } + auto image_event = event_with_pan_and_scale_applied(event); + if (on_image_mouse_position_change) { + on_image_mouse_position_change(image_event.position()); + } + if (!m_active_tool) return; auto layer_event = m_active_layer ? event_adjusted_for_layer(event, *m_active_layer) : event; - auto image_event = event_with_pan_and_scale_applied(event); Tool::MouseEvent tool_event(Tool::MouseEvent::Action::MouseDown, layer_event, image_event, event); m_active_tool->on_mousemove(m_active_layer.ptr(), tool_event); - - if (on_image_mouse_position_change) { - on_image_mouse_position_change(image_event.position()); - } } void ImageEditor::mouseup_event(GUI::MouseEvent& event)