From 8b293119aba6432a80f73a608557d5531de92c52 Mon Sep 17 00:00:00 2001 From: Tibor Nagy Date: Sat, 26 Sep 2020 19:31:27 +0200 Subject: [PATCH] LibGUI: Update color widgets when has_alpha is toggled in ColorPicker Fixing a bug where the alpha spinbox wasn't immediately disabled when color_has_alpha_channel was set to false. --- Libraries/LibGUI/ColorPicker.cpp | 9 +++++++++ Libraries/LibGUI/ColorPicker.h | 2 +- 2 files changed, 10 insertions(+), 1 deletion(-) diff --git a/Libraries/LibGUI/ColorPicker.cpp b/Libraries/LibGUI/ColorPicker.cpp index ff471e1131..ce4db5e5df 100644 --- a/Libraries/LibGUI/ColorPicker.cpp +++ b/Libraries/LibGUI/ColorPicker.cpp @@ -161,6 +161,15 @@ ColorPicker::~ColorPicker() { } +void ColorPicker::set_color_has_alpha_channel(bool has_alpha) +{ + if (m_color_has_alpha_channel == has_alpha) + return; + + m_color_has_alpha_channel = has_alpha; + update_color_widgets(); +} + void ColorPicker::build_ui() { auto& root_container = set_main_widget(); diff --git a/Libraries/LibGUI/ColorPicker.h b/Libraries/LibGUI/ColorPicker.h index 0d0e2334c4..320614805f 100644 --- a/Libraries/LibGUI/ColorPicker.h +++ b/Libraries/LibGUI/ColorPicker.h @@ -42,7 +42,7 @@ public: virtual ~ColorPicker() override; bool color_has_alpha_channel() const { return m_color_has_alpha_channel; } - void set_color_has_alpha_channel(bool has_alpha) { m_color_has_alpha_channel = has_alpha; } + void set_color_has_alpha_channel(bool); Color color() const { return m_color; } private: