From ebc8e352bfb743b8239b1738208254ba904386f8 Mon Sep 17 00:00:00 2001
From: Robin Gareus <robin@gareus.org>
Date: Mon, 3 Jul 2017 01:42:36 +0200
Subject: [PATCH] FP8: toggle editor/mixer (now that "link" button is used)

..and some TODO notes.
---
 libs/surfaces/faderport8/actions.cc   | 4 +++-
 libs/surfaces/faderport8/callbacks.cc | 3 ++-
 2 files changed, 5 insertions(+), 2 deletions(-)

diff --git a/libs/surfaces/faderport8/actions.cc b/libs/surfaces/faderport8/actions.cc
index a0879b1902..21ed2998c3 100644
--- a/libs/surfaces/faderport8/actions.cc
+++ b/libs/surfaces/faderport8/actions.cc
@@ -98,7 +98,7 @@ FaderPort8::setup_actions ()
 	BindMethod (BtnBypass, button_bypass);
 	BindAction (BtnBypassAll, "Mixer", "ab-plugins");
 
-	BindAction (BtnMacro, "Mixer", "show-editor");
+	BindAction (BtnMacro, "Common", "toggle-editor-and-mixer");
 	BindMethod (BtnOpen, button_open);
 
 	BindMethod (BtnLink, button_link);
@@ -239,6 +239,8 @@ FaderPort8::button_automation (ARDOUR::AutoState as)
 			break;
 	}
 
+	// TODO link/lock control automation?
+
 	// apply to all selected tracks
 	StripableList all;
 	session->get_stripables (all);
diff --git a/libs/surfaces/faderport8/callbacks.cc b/libs/surfaces/faderport8/callbacks.cc
index 112b1d96bb..ebd6869b8e 100644
--- a/libs/surfaces/faderport8/callbacks.cc
+++ b/libs/surfaces/faderport8/callbacks.cc
@@ -66,7 +66,8 @@ FaderPort8::send_session_state ()
 	notify_automation_mode_changed (); // XXX (stip specific, see below)
 }
 
-// TODO: AutomationState display of plugin & send automation ?!
+// TODO: AutomationState display of plugin & send automation
+// TODO: link/lock control AS.
 void
 FaderPort8::notify_automation_mode_changed ()
 {