From f13ff1e10bba05be12f021bcee601d5c636bcc26 Mon Sep 17 00:00:00 2001 From: David Robillard Date: Fri, 20 Apr 2012 21:06:37 +0000 Subject: [PATCH] Oops. Turns out activate handles both keyboard and double click. Let Gtk do the right thing. git-svn-id: svn://localhost/ardour2/branches/3.0@12046 d708f5d6-7413-0410-9779-e7cbd77b26cf --- gtk2_ardour/plugin_selector.cc | 9 --------- gtk2_ardour/plugin_selector.h | 1 - 2 files changed, 10 deletions(-) diff --git a/gtk2_ardour/plugin_selector.cc b/gtk2_ardour/plugin_selector.cc index 85f4a8d75b..75040eca06 100644 --- a/gtk2_ardour/plugin_selector.cc +++ b/gtk2_ardour/plugin_selector.cc @@ -181,7 +181,6 @@ PluginSelector::PluginSelector (PluginManager& mgr) //plugin_display.set_name("PluginSelectorList"); added_list.set_name("PluginSelectorList"); - plugin_display.signal_button_press_event().connect_notify (sigc::mem_fun(*this, &PluginSelector::row_clicked)); plugin_display.signal_row_activated().connect_notify (sigc::mem_fun(*this, &PluginSelector::row_activated)); plugin_display.get_selection()->signal_changed().connect (sigc::mem_fun(*this, &PluginSelector::display_selection_changed)); plugin_display.grab_focus(); @@ -199,14 +198,6 @@ PluginSelector::~PluginSelector () delete _plugin_menu; } -void -PluginSelector::row_clicked(GdkEventButton* event) -{ - if (event->type == GDK_2BUTTON_PRESS) { - btn_add_clicked(); - } -} - void PluginSelector::row_activated(Gtk::TreeModel::Path path, Gtk::TreeViewColumn* col) { diff --git a/gtk2_ardour/plugin_selector.h b/gtk2_ardour/plugin_selector.h index 7be17b62aa..034622b386 100644 --- a/gtk2_ardour/plugin_selector.h +++ b/gtk2_ardour/plugin_selector.h @@ -118,7 +118,6 @@ class PluginSelector : public ArdourDialog Gtk::Menu* _plugin_menu; ARDOUR::PluginManager& manager; - void row_clicked(GdkEventButton *); void row_activated(Gtk::TreeModel::Path path, Gtk::TreeViewColumn* col); void btn_add_clicked(); void btn_remove_clicked();