From 4f58a92c6d9ee845928280a919468d4183f50f44 Mon Sep 17 00:00:00 2001
From: Paul Davis <paul@linuxaudiosystems.com>
Date: Wed, 10 Nov 2021 17:51:04 -0700
Subject: [PATCH] show trigger/slot selection

---
 gtk2_ardour/triggerbox_ui.cc | 34 ++++++++++++++++++++++++++++++----
 gtk2_ardour/triggerbox_ui.h  |  7 +++++--
 2 files changed, 35 insertions(+), 6 deletions(-)

diff --git a/gtk2_ardour/triggerbox_ui.cc b/gtk2_ardour/triggerbox_ui.cc
index 00f921fe52..167102ef8c 100644
--- a/gtk2_ardour/triggerbox_ui.cc
+++ b/gtk2_ardour/triggerbox_ui.cc
@@ -106,7 +106,6 @@ TriggerEntry::TriggerEntry (Canvas* canvas, ARDOUR::Trigger& t)
 	prop_change (changed);
 }
 
-
 TriggerEntry::~TriggerEntry ()
 {
 }
@@ -123,14 +122,28 @@ void
 TriggerEntry::owner_color_changed ()
 {
 	set_fill_color (dynamic_cast<Stripable*> (_trigger.box().owner())->presentation_info().color());
-	set_outline_color (HSV (fill_color()).opposite().color());
+	selection_change ();
 	active_bar->set_fill_color (HSV (fill_color()).darker(0.3).color ());
 }
 
+void
+TriggerEntry::selection_change ()
+{
+	if (PublicEditor::instance().get_selection().selected (this)) {
+		set_outline_color (UIConfiguration::instance().color ("selection"));
+	} else {
+		set_outline_color (HSV (fill_color()).opposite().color());
+	}
+
+}
+
 bool
 TriggerEntry::event_handler (GdkEvent* ev)
 {
 	switch (ev->type) {
+	case GDK_BUTTON_PRESS:
+		PublicEditor::instance().get_selection().set (this);
+		break;
 	case GDK_ENTER_NOTIFY:
 		if (ev->crossing.detail != GDK_NOTIFY_INFERIOR) {
 			play_button->show ();
@@ -269,11 +282,22 @@ TriggerBoxUI::TriggerBoxUI (ArdourCanvas::Item* parent, TriggerBox& tb)
 	set_fill (true);
 
 	build ();
+
+	selection_connection = PublicEditor::instance().get_selection().TriggersChanged.connect (sigc::mem_fun (*this, &TriggerBoxUI::selection_changed));
 }
 
 TriggerBoxUI::~TriggerBoxUI ()
 {
 	update_connection.disconnect ();
+	selection_connection.disconnect ();
+}
+
+void
+TriggerBoxUI::selection_changed ()
+{
+	for (auto & slot : _slots) {
+		slot->selection_change ();
+	}
 }
 
 void
@@ -347,6 +371,8 @@ bool
 TriggerBoxUI::event (GdkEvent* ev, uint64_t n)
 {
 	switch (ev->type) {
+	case GDK_BUTTON_PRESS:
+		break;
 	case GDK_2BUTTON_PRESS:
 		edit_trigger (n);
 		return true;
@@ -662,8 +688,8 @@ TriggerBoxUI::stop_updating ()
 void
 TriggerBoxUI::rapid_update ()
 {
-	for (Slots::iterator s = _slots.begin(); s != _slots.end(); ++s) {
-		(*s)->maybe_update ();
+	for (auto & slot : _slots) {
+		slot->maybe_update ();
 	}
 }
 
diff --git a/gtk2_ardour/triggerbox_ui.h b/gtk2_ardour/triggerbox_ui.h
index c6aebe282d..caef6e66fa 100644
--- a/gtk2_ardour/triggerbox_ui.h
+++ b/gtk2_ardour/triggerbox_ui.h
@@ -61,6 +61,7 @@ class TriggerEntry : public ArdourCanvas::Rectangle
 	void _size_allocate (ArdourCanvas::Rect const &);
 	void maybe_update ();
 	bool event_handler (GdkEvent*);
+	void selection_change ();
 
   private:
 	ARDOUR::Trigger& _trigger;
@@ -118,9 +119,11 @@ class TriggerBoxUI : public ArdourCanvas::Table
 	void build ();
 	void rapid_update ();
 
-	sigc::connection update_connection;
-};
+	void selection_changed ();
 
+	sigc::connection update_connection;
+	sigc::connection selection_connection;
+};
 
 class TriggerBoxWidget : public ArdourCanvas::GtkCanvas
 {