From dd5ceb74e96a6a4677285c7c9b237961fa232b58 Mon Sep 17 00:00:00 2001 From: Mustafa Quraish Date: Thu, 2 Sep 2021 00:01:31 -0400 Subject: [PATCH] PixelPaint: Ctrl+click on palette colors now sets primary/secondary Previously it would only change the color of the ColorWidget itself, but not make it the primary/secondary color. I think it feels nicer this way, if I'm adding a color to the palette I likely want to use it. If you *really* need to only change the color of the palette, you can just Ctrl+Middle click. --- Userland/Applications/PixelPaint/PaletteWidget.cpp | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/Userland/Applications/PixelPaint/PaletteWidget.cpp b/Userland/Applications/PixelPaint/PaletteWidget.cpp index 9395a3c04d..846a40ad32 100644 --- a/Userland/Applications/PixelPaint/PaletteWidget.cpp +++ b/Userland/Applications/PixelPaint/PaletteWidget.cpp @@ -39,7 +39,7 @@ public: virtual void mousedown_event(GUI::MouseEvent& event) override { - if (event.modifiers() & KeyModifier::Mod_Ctrl && event.button() == GUI::MouseButton::Left) { + if (event.modifiers() & KeyModifier::Mod_Ctrl) { auto dialog = GUI::ColorPicker::construct(m_color, window()); if (dialog->exec() == GUI::Dialog::ExecOK) { m_color = dialog->color(); @@ -48,7 +48,6 @@ public: set_palette(pal); update(); } - return; } if (event.button() == GUI::MouseButton::Left)