RGBCube/serenity
1
Fork 0
mirror of https://github.com/RGBCube/serenity synced 2025-10-31 22:02:44 +00:00
Code Issues Activity Actions

PixelPaint: Use layer coordinates when color picking using Alt key

Browse source 
Previously, raw coordinates were being used, which meant that color
was being sampled from the wrong position.
This commit is contained in:
Tim Ledbetter 2023-01-18 18:35:17 +00:00 • committed by Andreas Kling
parent 81bd91c1c3
commit 45e60a416b
1 changed files with 5 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

10
Userland/Applications/PixelPaint/ImageEditor.cpp
View file

@ -370,11 +370,6 @@ void ImageEditor::mousedown_event(GUI::MouseEvent& event)
return;
}
if (event.alt() && !m_active_tool->is_overriding_alt()) {
set_editor_color_to_color_at_mouse_position(event);
return; // Pick Color instead of acivating active tool when holding alt.
}
if (!m_active_tool)
return;
@ -384,6 +379,11 @@ void ImageEditor::mousedown_event(GUI::MouseEvent& event)
}
auto layer_event = m_active_layer ? event_adjusted_for_layer(event, *m_active_layer) : event;
if (event.alt() && !m_active_tool->is_overriding_alt()) {
set_editor_color_to_color_at_mouse_position(layer_event);
return; // Pick Color instead of acivating active tool when holding alt.
}
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_mousedown(m_active_layer.ptr(), tool_event);