fix some layering/display problems with AU GUIs (requires new patches for GDK/Quartz)
[ardour.git] / gtk2_ardour / redirect_box.cc
index 2ab3b7af4af423c174f3ebeae99282368c45aed0..bc2d1e63414f812fe3314edcd6541e7c6c0ff49d 100644 (file)
@@ -65,8 +65,8 @@
 
 #include "i18n.h"
 
-#ifdef HAVE_AUDIOUNIT
-#include "au_pluginui.h"
+#ifdef HAVE_AUDIOUNITS
+class AUPluginUI;
 #endif
 
 using namespace sigc;
@@ -294,7 +294,7 @@ RedirectBox::redirect_button_press_event (GdkEventButton *ev)
                
        }
 
-       if (redirect && (Keyboard::is_edit_event (ev) || (ev->button == 1 && ev->type == GDK_2BUTTON_PRESS && ev->state == 0))) {
+       if (redirect && (Keyboard::is_edit_event (ev) || (ev->button == 1 && ev->type == GDK_2BUTTON_PRESS))) {
                
                if (_session.engine().connected()) {
                        /* XXX giving an error message here is hard, because we may be in the midst of a button press */
@@ -339,8 +339,10 @@ RedirectBox::redirect_button_release_event (GdkEventButton *ev)
                show_redirect_menu(ev->time);
                ret = true;
 
-       } else if (redirect && ev->button == 2 && ev->state == GDK_BUTTON2_MASK) {
+       } else if (redirect && (ev->button == 2) && (Keyboard::no_modifier_keys_pressed (ev) && ((ev->state & Gdk::BUTTON2_MASK) == Gdk::BUTTON2_MASK))) {
                
+               /* button2-click with no modifiers */
+
                redirect->set_active (!redirect->active(), this);
                ret = true;
 
@@ -395,13 +397,13 @@ RedirectBox::insert_plugin_chosen (boost::shared_ptr<Plugin> plugin)
 
                boost::shared_ptr<Redirect> redirect (new PluginInsert (_session, plugin, _placement));
                
-               redirect->active_changed.connect (bind (mem_fun (*this, &RedirectBox::show_redirect_active_r), boost::weak_ptr<Redirect>(redirect)));
-
                uint32_t err_streams;
 
                if (_route->add_redirect (redirect, this, &err_streams)) {
                        weird_plugin_dialog (*plugin, err_streams, _route);
-                       // XXX SHAREDPTR delete plugin here .. do we even need to care? 
+               } else {
+                       redirect->set_active (true, 0);
+                       redirect->active_changed.connect (bind (mem_fun (*this, &RedirectBox::show_redirect_active_r), boost::weak_ptr<Redirect>(redirect)));
                }
        }
 }
@@ -414,7 +416,7 @@ RedirectBox::weird_plugin_dialog (Plugin& p, uint32_t streams, boost::shared_ptr
 
        /* i hate this kind of code */
 
-       if (streams > p.get_info()->n_inputs) {
+       if (streams > (unsigned)p.get_info()->n_inputs) {
                label.set_text (string_compose (_(
 "You attempted to add a plugin (%1).\n"
 "The plugin has %2 inputs\n"
@@ -426,7 +428,7 @@ RedirectBox::weird_plugin_dialog (Plugin& p, uint32_t streams, boost::shared_ptr
                                         p.name(),
                                         p.get_info()->n_inputs,
                                         streams));
-       } else if (streams < p.get_info()->n_inputs) {
+       } else if (streams < (unsigned)p.get_info()->n_inputs) {
                label.set_text (string_compose (_(
 "You attempted to add a plugin (%1).\n"
 "The plugin has %2 inputs\n"
@@ -648,14 +650,14 @@ RedirectBox::show_redirect_active_r (Redirect* r, void *src, boost::weak_ptr<Red
 void
 RedirectBox::show_redirect_active (boost::weak_ptr<Redirect> weak_redirect)
 {
-       ENSURE_GUI_THREAD(bind (mem_fun(*this, &RedirectBox::show_redirect_active), weak_redirect));
-       
        boost::shared_ptr<Redirect> redirect (weak_redirect.lock());
        
        if (!redirect) {
                return;
        }
 
+       ENSURE_GUI_THREAD(bind (mem_fun(*this, &RedirectBox::show_redirect_active), weak_redirect));
+       
        Gtk::TreeModel::Children children = model->children();
        Gtk::TreeModel::Children::iterator iter = children.begin();
 
@@ -760,15 +762,19 @@ RedirectBox::cut_redirects ()
 
        no_redirect_redisplay = true;
        for (vector<boost::shared_ptr<Redirect> >::iterator i = to_be_removed.begin(); i != to_be_removed.end(); ++i) {
+               // Do not cut inserts or sends
+               if (boost::dynamic_pointer_cast<PluginInsert>((*i)) != 0) {
+                       void* gui = (*i)->get_gui ();
                
-               void* gui = (*i)->get_gui ();
-               
-               if (gui) {
-                       static_cast<Gtk::Widget*>(gui)->hide ();
-               }
+                       if (gui) {
+                               static_cast<Gtk::Widget*>(gui)->hide ();
+                       }
                
-               if (_route->remove_redirect (*i, this)) {
-                       /* removal failed */
+                       if (_route->remove_redirect (*i, this)) {
+                               /* removal failed */
+                               _rr_selection.remove (*i);
+                       }
+               } else {
                        _rr_selection.remove (*i);
                }
 
@@ -790,10 +796,40 @@ RedirectBox::copy_redirects ()
        }
 
        for (vector<boost::shared_ptr<Redirect> >::iterator i = to_be_copied.begin(); i != to_be_copied.end(); ++i) {
-               copies.push_back (Redirect::clone (*i));
+               // Do not copy inserts or sends
+               if (boost::dynamic_pointer_cast<PluginInsert>((*i)) != 0) {
+                       copies.push_back (Redirect::clone (*i));
+               }
        }
 
        _rr_selection.set (copies);
+
+}
+
+void
+RedirectBox::delete_redirects ()
+{
+       vector<boost::shared_ptr<Redirect> > to_be_deleted;
+       
+       get_selected_redirects (to_be_deleted);
+
+       if (to_be_deleted.empty()) {
+               return;
+       }
+
+       for (vector<boost::shared_ptr<Redirect> >::iterator i = to_be_deleted.begin(); i != to_be_deleted.end(); ++i) {
+               
+               void* gui = (*i)->get_gui ();
+               
+               if (gui) {
+                       static_cast<Gtk::Widget*>(gui)->hide ();
+               }
+
+               _route->remove_redirect( *i, this);
+       }
+
+       no_redirect_redisplay = false;
+       redisplay_redirects (this);
 }
 
 gint
@@ -957,21 +993,31 @@ could not match the configuration of this track.");
 void
 RedirectBox::all_redirects_active (bool state)
 {
-       _route->all_redirects_active (state);
+       _route->all_redirects_active (_placement, state);
 }
 
 void
-RedirectBox::clear_redirects()
+RedirectBox::clear_redirects ()
 {
        string prompt;
        vector<string> choices;
 
        if (boost::dynamic_pointer_cast<AudioTrack>(_route) != 0) {
-               prompt = _("Do you really want to remove all redirects from this track?\n"
-                          "(this cannot be undone)");
+               if (_placement == PreFader) {
+                       prompt = _("Do you really want to remove all pre-fader redirects from this track?\n"
+                                  "(this cannot be undone)");
+               } else {
+                       prompt = _("Do you really want to remove all post-fader redirects from this track?\n"
+                                  "(this cannot be undone)");
+               }
        } else {
-               prompt = _("Do you really want to remove all redirects from this bus?\n"
-                          "(this cannot be undone)");
+               if (_placement == PreFader) {
+                       prompt = _("Do you really want to remove all pre-fader redirects from this bus?\n"
+                                  "(this cannot be undone)");
+               } else {
+                       prompt = _("Do you really want to remove all post-fader redirects from this bus?\n"
+                                  "(this cannot be undone)");
+               }
        }
 
        choices.push_back (_("Cancel"));
@@ -980,7 +1026,7 @@ RedirectBox::clear_redirects()
        Gtkmm2ext::Choice prompter (prompt, choices);
 
        if (prompter.run () == 1) {
-               _route->clear_redirects (this);
+               _route->clear_redirects (_placement, this);
        }
 }
 
@@ -1026,66 +1072,45 @@ RedirectBox::edit_redirect (boost::shared_ptr<Redirect> redirect)
                        send_ui->get_window()->raise ();
                } else {
                        send_ui->show_all ();
+                       send_ui->present ();
                }
                
        } else {
                
                /* it's an insert */
-               
+
                boost::shared_ptr<PluginInsert> plugin_insert;
                boost::shared_ptr<PortInsert> port_insert;
                
                if ((plugin_insert = boost::dynamic_pointer_cast<PluginInsert> (insert)) != 0) {
                        
-                       ARDOUR::PluginType type = plugin_insert->type();
-
-                       if (type == ARDOUR::LADSPA || type == ARDOUR::VST) {
-                               PluginUIWindow *plugin_ui;
+                       PluginUIWindow *plugin_ui;
                        
-                               if (plugin_insert->get_gui() == 0) {
-                                                               
-                                       plugin_ui = new PluginUIWindow (plugin_insert);
-
-                                       if (_owner_is_mixer) {
-                                               ARDOUR_UI::instance()->the_mixer()->ensure_float (*plugin_ui);
-                                       } else {
-                                               ARDOUR_UI::instance()->the_editor().ensure_float (*plugin_ui);
-                                       }
-
-                                       WindowTitle title(Glib::get_application_name());
-                                       title += generate_redirect_title (plugin_insert);
-                                       plugin_ui->set_title (title.get_string());
-
-                                       plugin_insert->set_gui (plugin_ui);
-                                       
-                                       // change window title when route name is changed
-                                       _route->name_changed.connect (bind (mem_fun(*this, &RedirectBox::route_name_changed), plugin_ui, boost::weak_ptr<PluginInsert> (plugin_insert)));
-                                       
+                       if (plugin_insert->get_gui() == 0) {
+                               
+                               plugin_ui = new PluginUIWindow (plugin_insert);
+                               
+                               WindowTitle title(Glib::get_application_name());
+                               title += generate_redirect_title (plugin_insert);
+                               plugin_ui->set_title (title.get_string());
+                               
+                               plugin_insert->set_gui (plugin_ui);
+                               
+                               // change window title when route name is changed
+                               _route->name_changed.connect (bind (mem_fun(*this, &RedirectBox::route_name_changed), plugin_ui, boost::weak_ptr<PluginInsert> (plugin_insert)));
                                
-                               } else {
-                                       plugin_ui = reinterpret_cast<PluginUIWindow *> (plugin_insert->get_gui());
-                               }
-                       
-                               if (plugin_ui->is_visible()) {
-                                       plugin_ui->get_window()->raise ();
-                               } else {
-                                       plugin_ui->show_all ();
-                               }
-#ifdef HAVE_AUDIOUNIT
-                       } else if (type == ARDOUR::AudioUnit) {
-                               AUPluginUI* plugin_ui;
-                               if (plugin_insert->get_gui() == 0) {
-                                       plugin_ui = new AUPluginUI (plugin_insert);
-                               } else {
-                                       plugin_ui = reinterpret_cast<AUPluginUI*> (plugin_insert->get_gui());
-                               }
                                
-                               // raise window, somehow
-#endif                         
                        } else {
-                               warning << "Unsupported plugin sent to RedirectBox::edit_redirect()" << endmsg;
-                               return;
+                               plugin_ui = reinterpret_cast<PluginUIWindow *> (plugin_insert->get_gui());
+                       }
+                       
+                       if (plugin_ui->is_visible()) {
+                               plugin_ui->get_window()->raise ();
+                       } else {
+                               plugin_ui->show_all ();
+                               plugin_ui->present ();
                        }
+
                } else if ((port_insert = boost::dynamic_pointer_cast<PortInsert> (insert)) != 0) {
                        
                        if (!_session.engine().connected()) {
@@ -1108,6 +1133,7 @@ RedirectBox::edit_redirect (boost::shared_ptr<Redirect> redirect)
                                io_selector->get_window()->raise ();
                        } else {
                                io_selector->show_all ();
+                               io_selector->present ();
                        }
                }
        }
@@ -1151,6 +1177,10 @@ RedirectBox::register_actions ()
        ActionManager::plugin_selection_sensitive_actions.push_back(act);
        act = ActionManager::register_action (popup_act_grp, X_("copy"), _("Copy"),  sigc::ptr_fun (RedirectBox::rb_copy));
        ActionManager::plugin_selection_sensitive_actions.push_back(act);
+
+       act = ActionManager::register_action (popup_act_grp, X_("delete"), _("Delete"),  sigc::ptr_fun (RedirectBox::rb_delete));
+       ActionManager::plugin_selection_sensitive_actions.push_back(act); // ??
+
        paste_action = ActionManager::register_action (popup_act_grp, X_("paste"), _("Paste"),  sigc::ptr_fun (RedirectBox::rb_paste));
        act = ActionManager::register_action (popup_act_grp, X_("rename"), _("Rename"),  sigc::ptr_fun (RedirectBox::rb_rename));
        ActionManager::plugin_selection_sensitive_actions.push_back(act);
@@ -1221,6 +1251,16 @@ RedirectBox::rb_cut ()
        _current_redirect_box->cut_redirects ();
 }
 
+void
+RedirectBox::rb_delete ()
+{
+       if (_current_redirect_box == 0) {
+               return;
+       }
+
+       _current_redirect_box->delete_redirects ();
+}
+
 void
 RedirectBox::rb_copy ()
 {
@@ -1334,7 +1374,7 @@ RedirectBox::route_name_changed (void* src, PluginUIWindow* plugin_ui, boost::we
 string 
 RedirectBox::generate_redirect_title (boost::shared_ptr<PluginInsert> pi)
 {
-       string maker = pi->plugin()->maker();
+       string maker = pi->plugin()->maker() ? pi->plugin()->maker() : "";
        string::size_type email_pos;
 
        if ((email_pos = maker.find_first_of ('<')) != string::npos) {