PixelPaint: Update palette colors in real-time while picking

Palette color selections in `PaletteWidget` now update while interacting with `GUI::ColorPicker`
Poseydon42 · Jul 24, 2023 · 0388bb0 · 0388bb0
1 parent 5866a3a
commit 0388bb0
Showing 1 changed file with 8 additions and 5 deletions.
diff --git a/Userland/Applications/PixelPaint/PaletteWidget.cpp b/Userland/Applications/PixelPaint/PaletteWidget.cpp
@@ -33,13 +33,14 @@ class ColorWidget : public GUI::Frame {
     {
         if (event.modifiers() & KeyModifier::Mod_Ctrl) {
             auto dialog = GUI::ColorPicker::construct(m_color, window());
-            if (dialog->exec() == GUI::Dialog::ExecResult::OK) {
-                m_color = dialog->color();
+            dialog->on_color_changed = [this](Gfx::Color color) {
+                m_color = color;
                 auto pal = palette();
                 pal.set_color(ColorRole::Background, m_color);
                 set_palette(pal);
                 update();
-            }
+            };
+            dialog->exec();
         }
 
         if (event.button() == GUI::MouseButton::Primary)
@@ -72,8 +73,10 @@ class SelectedColorWidget : public GUI::Frame {
             return;
 
         auto dialog = GUI::ColorPicker::construct(m_color, window());
-        if (dialog->exec() == GUI::Dialog::ExecResult::OK)
-            on_color_change(dialog->color());
+        dialog->on_color_changed = [this](Gfx::Color color) {
+            on_color_change(color);
+        };
+        dialog->exec();
     }
 
     void set_background_color(Color color)