From 0da02b582d0a7d69a349b1858b41a06bfa5b2d61 Mon Sep 17 00:00:00 2001
From: Robin Gareus <robin@gareus.org>
Date: Sun, 30 Jan 2022 22:42:13 +0100
Subject: [PATCH] Trigger clip picker: button to open library folder

---
 gtk2_ardour/trigger_clip_picker.cc | 33 ++++++++++++++++++++++++++----
 gtk2_ardour/trigger_clip_picker.h  |  2 ++
 2 files changed, 31 insertions(+), 4 deletions(-)

diff --git a/gtk2_ardour/trigger_clip_picker.cc b/gtk2_ardour/trigger_clip_picker.cc
index bcfe546a9c..94b597f098 100644
--- a/gtk2_ardour/trigger_clip_picker.cc
+++ b/gtk2_ardour/trigger_clip_picker.cc
@@ -23,6 +23,7 @@
 
 #include "pbd/basename.h"
 #include "pbd/file_utils.h"
+#include "pbd/openuri.h"
 #include "pbd/pathexpand.h"
 #include "pbd/search_path.h"
 #include "pbd/unwind.h"
@@ -44,6 +45,8 @@
 #include "gtkmm2ext/utils.h"
 
 #include "widgets/paths_dialog.h"
+#include "widgets/tooltips.h"
+#include "widgets/ardour_icon.h"
 
 #include "trigger_clip_picker.h"
 #include "ui_config.h"
@@ -98,18 +101,28 @@ TriggerClipPicker::TriggerClipPicker ()
 	_stop_btn.set_icon (ArdourWidgets::ArdourIcon::TransportStop);
 	_stop_btn.signal_clicked.connect (sigc::mem_fun (*this, &TriggerClipPicker::stop_audition));
 
+	_open_library_btn.set_name ("generic button");
+	_open_library_btn.set_icon (ArdourWidgets::ArdourIcon::Folder);
+	_open_library_btn.signal_clicked.connect (sigc::mem_fun (*this, &TriggerClipPicker::open_library));
+	_open_library_btn.set_no_show_all ();
+
 	_play_btn.set_sensitive (false);
 	_stop_btn.set_sensitive (false);
 
 	_autoplay_btn.set_can_focus(false);
 	_autoplay_btn.signal_toggled ().connect (sigc::mem_fun (*this, &TriggerClipPicker::autoplay_toggled));
 
+	ArdourWidgets::set_tooltip (_play_btn, _("Audition selected clip"));
+	ArdourWidgets::set_tooltip (_stop_btn, _("Stop the audition"));
+	ArdourWidgets::set_tooltip (_open_library_btn, _("Open clip library folder"));
+
 	/* Layout */
 	_auditable.set_homogeneous(false);
-	_auditable.attach (_play_btn,     0, 1, 0, 1, SHRINK, SHRINK);
-	_auditable.attach (_stop_btn,     1, 2, 0, 1, SHRINK, SHRINK);
-	_auditable.attach (_autoplay_btn, 2, 3, 0, 1, EXPAND | FILL, SHRINK);
-	_auditable.attach (_seek_slider,  0, 3, 1, 2, EXPAND | FILL, SHRINK);
+	_auditable.attach (_play_btn,         0, 1, 0, 1, SHRINK, SHRINK);
+	_auditable.attach (_stop_btn,         1, 2, 0, 1, SHRINK, SHRINK);
+	_auditable.attach (_autoplay_btn,     2, 3, 0, 1, EXPAND | FILL, SHRINK);
+	_auditable.attach (_open_library_btn, 3, 4, 0, 1, SHRINK, SHRINK);
+	_auditable.attach (_seek_slider,      0, 4, 1, 2, EXPAND | FILL, SHRINK);
 	_auditable.set_border_width (4);
 	_auditable.set_spacings (4);
 
@@ -573,6 +586,12 @@ TriggerClipPicker::list_dir (std::string const& path, Gtk::TreeNodeChildren cons
 
 	_current_path = path;
 
+	if (_clip_library_dir == path) {
+		_open_library_btn.show ();
+	} else {
+		_open_library_btn.hide ();
+	}
+
 	std::vector<std::string> dirs;
 	std::vector<std::string> files;
 
@@ -645,6 +664,12 @@ TriggerClipPicker::list_dir (std::string const& path, Gtk::TreeNodeChildren cons
 	}
 }
 
+void
+TriggerClipPicker::open_library ()
+{
+	PBD::open_folder (_clip_library_dir);
+}
+
 /* ****************************************************************************
  * Auditioner
  */
diff --git a/gtk2_ardour/trigger_clip_picker.h b/gtk2_ardour/trigger_clip_picker.h
index 1f617bd746..dbe412cee0 100644
--- a/gtk2_ardour/trigger_clip_picker.h
+++ b/gtk2_ardour/trigger_clip_picker.h
@@ -69,6 +69,7 @@ private:
 	void audition_progress (ARDOUR::samplecnt_t, ARDOUR::samplecnt_t);
 	void stop_audition ();
 	void autoplay_toggled ();
+	void open_library ();
 	bool seek_button_press (GdkEventButton*);
 	bool seek_button_release (GdkEventButton*);
 
@@ -96,6 +97,7 @@ private:
 	Gtk::Table                   _auditable;
 	ArdourWidgets::ArdourButton  _play_btn;
 	ArdourWidgets::ArdourButton  _stop_btn;
+	ArdourWidgets::ArdourButton  _open_library_btn;
 	Gtk::HScale                  _seek_slider;
 	Gtk::CheckButton             _autoplay_btn;