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.
lanmonster · Sep 26, 2020 · 8b29311 · 8b29311
1 parent 01915a3
commit 8b29311
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git 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<Widget>();

diff --git a/Libraries/LibGUI/ColorPicker.h b/Libraries/LibGUI/ColorPicker.h
@@ -42,7 +42,7 @@ class ColorPicker final : public Dialog {
  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: