From dd7063277a52efd0a335156f055a778613d63283 Mon Sep 17 00:00:00 2001
From: Robin Gareus <robin@gareus.org>
Date: Thu, 30 Mar 2017 15:20:48 +0200
Subject: [PATCH] Add ColorButton drop-in replacement with palette support

---
 gtk2_ardour/stripable_colorpicker.cc | 60 +++++++++++++++++++++-------
 gtk2_ardour/stripable_colorpicker.h  | 17 ++++++++
 2 files changed, 63 insertions(+), 14 deletions(-)

diff --git a/gtk2_ardour/stripable_colorpicker.cc b/gtk2_ardour/stripable_colorpicker.cc
index b9c4990cfe..3defed8120 100644
--- a/gtk2_ardour/stripable_colorpicker.cc
+++ b/gtk2_ardour/stripable_colorpicker.cc
@@ -77,6 +77,25 @@ StripableColorDialog::reset ()
 	_color_changed_connection.disconnect ();
 }
 
+void
+StripableColorDialog::popup (const std::string& name, uint32_t color)
+{
+	set_title (string_compose (_("Color Selection: %1"), name));
+	_initial_color = color;
+
+	get_colorsel()->set_has_opacity_control (false);
+	get_colorsel()->set_has_palette (true);
+
+	Gdk::Color c = gdk_color_from_rgba (_initial_color);
+
+	get_colorsel()->set_previous_color (c);
+	get_colorsel()->set_current_color (c);
+	_color_changed_connection.disconnect ();
+	_color_changed_connection = get_colorsel()->signal_color_changed().connect (sigc::mem_fun (*this, &StripableColorDialog::color_changed));
+
+	present ();
+}
+
 void
 StripableColorDialog::popup (boost::shared_ptr<ARDOUR::Stripable> s)
 {
@@ -92,27 +111,18 @@ StripableColorDialog::popup (boost::shared_ptr<ARDOUR::Stripable> s)
 
 	_stripable = s;
 	_stripable->set_active_color_picker (this);
-	_initial_color = _stripable->presentation_info().color ();
-	set_title (string_compose (_("Color Selection: %1"), s->name()));
-
-	get_colorsel()->set_has_opacity_control (false);
-	get_colorsel()->set_has_palette (true);
-
-	Gdk::Color c = gdk_color_from_rgba (_initial_color);
-
-	get_colorsel()->set_previous_color (c);
-	get_colorsel()->set_current_color (c);
-	_color_changed_connection = get_colorsel()->signal_color_changed().connect (sigc::mem_fun (*this, &StripableColorDialog::color_changed));
-
-	present ();
+	popup (s->name(), _stripable->presentation_info().color ());
 }
 
 void
 StripableColorDialog::finish_color_edit (int response)
 {
+	if (response == RESPONSE_OK) {
+		ColorChanged (gdk_color_to_rgba (get_colorsel()->get_current_color())); /* EMIT SIGNAL */
+	}
 	if (_stripable && response == RESPONSE_OK) {
 		_stripable->presentation_info().set_color (gdk_color_to_rgba (get_colorsel()->get_current_color()));
-	} else {
+	} else if (_stripable) {
 		_stripable->presentation_info().set_color (_initial_color);
 	}
 	reset ();
@@ -125,3 +135,25 @@ StripableColorDialog::color_changed ()
 		_stripable->presentation_info().set_color (gdk_color_to_rgba (get_colorsel()->get_current_color()));
 	}
 }
+
+
+ArdourColorButton::ArdourColorButton ()
+{
+	_color_picker.ColorChanged.connect (sigc::mem_fun(*this, &ArdourColorButton::color_selected));
+}
+
+void
+ArdourColorButton::on_clicked ()
+{
+	_color_picker.popup ("", gdk_color_to_rgba (get_color ()));
+	_color_picker.get_window ()->set_transient_for (get_window ());
+}
+
+void
+ArdourColorButton::color_selected (uint32_t color)
+{
+	Gdk::Color c;
+	set_color_from_rgba (c, color);
+	set_color (c);
+	g_signal_emit_by_name (GTK_WIDGET(gobj()), "color-set", 0);
+}
diff --git a/gtk2_ardour/stripable_colorpicker.h b/gtk2_ardour/stripable_colorpicker.h
index 952640da70..4af5bbd7e1 100644
--- a/gtk2_ardour/stripable_colorpicker.h
+++ b/gtk2_ardour/stripable_colorpicker.h
@@ -20,7 +20,9 @@
 #define __gtkardour_stripable_colorpicker_h__
 
 #include <boost/shared_ptr.hpp>
+#include <gtkmm/colorbutton.h>
 #include <gtkmm/colorselection.h>
+
 #include "ardour/stripable.h"
 
 class StripableColorDialog : public Gtk::ColorSelectionDialog
@@ -30,6 +32,8 @@ public:
 	~StripableColorDialog ();
 	void reset ();
 	void popup (boost::shared_ptr<ARDOUR::Stripable> s);
+	void popup (const std::string&, uint32_t);
+	sigc::signal<void, uint32_t> ColorChanged;
 
 private:
 	void initialize_color_palette ();
@@ -47,4 +51,17 @@ private:
 	static Gtk::ColorSelection::SlotChangePaletteHook gtk_palette_changed_hook;
 };
 
+class ArdourColorButton : public Gtk::ColorButton
+{
+public:
+	ArdourColorButton ();
+
+protected:
+	void on_clicked();
+	void color_selected (uint32_t color);
+
+private:
+	StripableColorDialog _color_picker;
+};
+
 #endif
-- 
2.30.2