X-Git-Url: https://main.carlh.net/gitweb/?a=blobdiff_plain;ds=sidebyside;f=gtk2_ardour%2Fpatch_change.cc;h=4006cbdf469b7030ed3bf11d026edf18a4a8dc4a;hb=bd77b605f6b7075d99e479c7c0b4693c455b5506;hp=530f903bad520c6266101ee143f06f087ec828d2;hpb=56ca52651e910496084b992486ac7b8e3bbc88cf;p=ardour.git

diff --git a/gtk2_ardour/patch_change.cc b/gtk2_ardour/patch_change.cc
index 530f903bad..4006cbdf46 100644
--- a/gtk2_ardour/patch_change.cc
+++ b/gtk2_ardour/patch_change.cc
@@ -30,26 +30,26 @@
 
 #include "canvas/debug.h"
 
-#include "ardour_ui.h"
-#include "midi_region_view.h"
-#include "patch_change.h"
 #include "editor.h"
 #include "editor_drag.h"
+#include "midi_region_view.h"
+#include "patch_change.h"
+#include "ui_config.h"
 
 using namespace MIDI::Name;
 using namespace std;
+using Gtkmm2ext::Keyboard;
 
 /** @param x x position in pixels.
  */
-PatchChange::PatchChange(
-		MidiRegionView& region,
-		ArdourCanvas::Container* parent,
-		const string&   text,
-		double          height,
-		double          x,
-		double          y,
-		ARDOUR::InstrumentInfo& info,
-		ARDOUR::MidiModel::PatchChangePtr patch)
+PatchChange::PatchChange(MidiRegionView&                   region,
+                         ArdourCanvas::Container*          parent,
+                         const string&                     text,
+                         double                            height,
+                         double                            x,
+                         double                            y,
+                         ARDOUR::InstrumentInfo&           info,
+                         ARDOUR::MidiModel::PatchChangePtr patch)
 	: _region (region)
 	, _info (info)
 	, _patch (patch)
@@ -58,15 +58,15 @@ PatchChange::PatchChange(
 	_flag = new ArdourCanvas::Flag (
 		parent,
 		height,
-		ARDOUR_UI::config()->color ("midi patch change outline"),
-		ARDOUR_UI::config()->color ("midi patch change fill"),
-		ArdourCanvas::Duple (x, y)
-		);
-	
+		UIConfiguration::instance().color ("midi patch change outline"),
+		UIConfiguration::instance().color_mod ("midi patch change fill", "midi patch change fill"),
+		ArdourCanvas::Duple (x, y),
+		true);
+
 	CANVAS_DEBUG_NAME (_flag, text);
 
 	_flag->Event.connect (sigc::mem_fun (*this, &PatchChange::event_handler));
-	_flag->set_font_description (ARDOUR_UI::config()->get_SmallFont());
+	_flag->set_font_description (UIConfiguration::instance().get_SmallFont());
 	_flag->set_text(text);
 }
 
@@ -98,12 +98,12 @@ PatchChange::initialize_popup_menus()
 		     ++bank) {
 			Glib::RefPtr<Glib::Regex> underscores = Glib::Regex::create("_");
 			std::string replacement(" ");
-			
+
 			Gtk::Menu& patch_bank_menu = *manage(new Gtk::Menu());
-			
+
 			const PatchNameList& patches = (*bank)->patch_name_list();
 			Gtk::Menu::MenuList& patch_menus = patch_bank_menu.items();
-		
+
 			for (PatchNameList::const_iterator patch = patches.begin();
 			     patch != patches.end();
 			     ++patch) {
@@ -131,13 +131,13 @@ PatchChange::initialize_popup_menus()
 
 		const PatchNameList& patches = patch_banks.front()->patch_name_list();
 		Gtk::Menu::MenuList& patch_menus = _popup.items();
-		
+
 		for (PatchNameList::const_iterator patch = patches.begin();
 		     patch != patches.end();
 		     ++patch) {
 			std::string name = (*patch)->name();
 			boost::replace_all (name, "_", " ");
-			
+
 			patch_menus.push_back (
 				Gtk::Menu_Helpers::MenuElem (
 					name,
@@ -160,13 +160,12 @@ PatchChange::event_handler (GdkEvent* ev)
 	Editor* e = dynamic_cast<Editor*> (&_region.get_time_axis_view().editor());
 
 	if (!e->internal_editing()) {
-		// not in internal edit mode, so no patch change editing
 		return false;
 	}
 
 	switch (ev->type) {
 	case GDK_BUTTON_PRESS:
-		if (e->current_mouse_mode() == Editing::MouseObject) {
+		if (e->current_mouse_mode() == Editing::MouseContent) {
 
 			if (Gtkmm2ext::Keyboard::is_delete_event (&ev->button)) {
 
@@ -177,14 +176,14 @@ PatchChange::event_handler (GdkEvent* ev)
 
 				_region.edit_patch_change (this);
 				return true;
-				
+
 			} else if (ev->button.button == 1) {
 				e->drags()->set (new PatchChangeDrag (e, this, &_region), ev);
 				return true;
 			}
 		}
 
-		if (ev->button.button == 3) {
+		if (Gtkmm2ext::Keyboard::is_context_menu_event (&ev->button)) {
 			if (!_popup_initialized) {
 				initialize_popup_menus();
 				_popup_initialized = true;
@@ -199,13 +198,25 @@ PatchChange::event_handler (GdkEvent* ev)
 		case GDK_Up:
 		case GDK_KP_Up:
 		case GDK_uparrow:
-			_region.previous_patch (*this);
-			break;
+			_region.step_patch(
+				*this, Keyboard::modifier_state_contains(ev->key.state, Keyboard::TertiaryModifier), 1);
+			return true;
 		case GDK_Down:
 		case GDK_KP_Down:
 		case GDK_downarrow:
-			_region.next_patch (*this);
+			_region.step_patch(
+				*this, Keyboard::modifier_state_contains(ev->key.state, Keyboard::TertiaryModifier), -1);
+			return true;
+		default:
 			break;
+		}
+		break;
+
+	case GDK_KEY_RELEASE:
+		switch (ev->key.keyval) {
+		case GDK_BackSpace:
+		case GDK_Delete:
+			_region.delete_patch_change (this);
 		default:
 			break;
 		}
@@ -213,10 +224,12 @@ PatchChange::event_handler (GdkEvent* ev)
 
 	case GDK_SCROLL:
 		if (ev->scroll.direction == GDK_SCROLL_UP) {
-			_region.previous_patch (*this);
+			_region.step_patch(
+				*this, Keyboard::modifier_state_contains(ev->scroll.state, Keyboard::TertiaryModifier), 1);
 			return true;
 		} else if (ev->scroll.direction == GDK_SCROLL_DOWN) {
-			_region.next_patch (*this);
+			_region.step_patch(
+				*this, Keyboard::modifier_state_contains(ev->scroll.state, Keyboard::TertiaryModifier), -1);
 			return true;
 		}
 		break;