LadybirdBrowser/ladybird
1
1
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2024-09-20 17:58:18 +03:00
Code Issues Packages Projects Releases Wiki Activity

PixelPaint: Always change cursor when active tool is set

Browse Source 
Previously, if you used one of the keyboard shortcuts to select a
different tool, it didn't change the cursor to the corresponding
one till you clicked somewhere / did something else to trigger an
update. This was pretty jarring since there's no indication as to
whether the tool change was successful or not.

This patch just calls `set_override_cursor()` when a valid active
tool is set to immediately update it.
This commit is contained in:
Mustafa Quraish 2021-08-27 23:24:38 -04:00 committed by Linus Groh
parent 01b38ffc9a
commit 5a8ac56fb9
Notes: sideshowbarker 2024-07-18 05:11:19 +09:00 
Author: https://github.com/mustafaquraish
Commit: https://github.com/SerenityOS/serenity/commit/5a8ac56fb99
Pull-request: https://github.com/SerenityOS/serenity/pull/9654
1 changed files with 1 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
Userland/Applications/PixelPaint/ImageEditor.cpp
View File

@ -318,6 +318,7 @@ void ImageEditor::set_active_tool(Tool* tool)
m_active_tool->setup(*this);
m_active_tool->on_tool_activation();
m_active_cursor = m_active_tool->cursor();
set_override_cursor(m_active_cursor);
}
}