Fixes to bundle manager to make it vaguely usable.
[ardour.git] / gtk2_ardour / io_selector.cc
index 6bc6aa6c235640e958f9c924e989f6b28a78f332..76892dcd3ddc48cbd8f26a9d25391ee97e0d97b5 100644 (file)
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2002-2003 Paul Davis 
+    Copyright (C) 2002-2007 Paul Davis 
 
     This program is free software; you can redistribute it and/or modify
     it under the terms of the GNU General Public License as published by
 
 */
 
-#include <gtkmm/label.h>
-#include <gtkmm/enums.h>
-#include <gtkmm/image.h>
-#include <gtkmm/stock.h>
 #include <gtkmm/messagedialog.h>
 #include <glibmm/objectbase.h>
+
 #include <gtkmm2ext/doi.h>
+
 #include <ardour/port_insert.h>
 #include "ardour/session.h"
 #include "ardour/io.h"
 #include "ardour/audioengine.h"
+#include "ardour/track.h"
+#include "ardour/audio_track.h"
+#include "ardour/midi_track.h"
+#include "ardour/data_type.h"
+#include "ardour/port.h"
+#include "ardour/bundle.h"
+
 #include "io_selector.h"
 #include "utils.h"
 #include "gui_thread.h"
 #include "i18n.h"
 
-RotatedLabelSet::RotatedLabelSet ()
-       : Glib::ObjectBase ("RotatedLabelSet"), Gtk::Widget (), _base_start (64), _base_width (128)
-{
-       set_flags (Gtk::NO_WINDOW);
-       set_angle (30);
-}
-
-RotatedLabelSet::~RotatedLabelSet ()
-{
-       
-}
-
-void
-RotatedLabelSet::set_angle (int degrees)
-{
-       _angle_degrees = degrees;
-       _angle_radians = M_PI * _angle_degrees / 180;
-
-       queue_resize ();
-}
+using namespace ARDOUR;
+using namespace Gtk;
 
-void
-RotatedLabelSet::on_size_request (Gtk::Requisition* requisition)
+IOSelector::IOSelector (ARDOUR::Session& session, boost::shared_ptr<ARDOUR::IO> io, bool in)
+       : PortMatrix (session, io->default_type())
+       , _io (io)
+       , _find_inputs_for_io_outputs (in)
 {
-       *requisition = Gtk::Requisition ();
-
-       if (_pango_layout == 0) {
-               return;
-       }
-
-       /* Our height is the highest label */
-       requisition->height = 0;
-       for (std::vector<std::string>::const_iterator i = _labels.begin(); i != _labels.end(); ++i) {
-               std::pair<int, int> const d = setup_layout (*i);
-               if (d.second > requisition->height) {
-                       requisition->height = d.second;
-               }
-       }
-
-       /* And our width is the base plus the width of the last label */
-       requisition->width = _base_start + _base_width;
-       if (_labels.empty() == false) {
-               std::pair<int, int> const d = setup_layout (_labels.back());
-               requisition->width += d.first;
-       }
-}
-
-void
-RotatedLabelSet::on_size_allocate (Gtk::Allocation& allocation)
-{
-       set_allocation (allocation);
-
-       if (_gdk_window) {
-               _gdk_window->move_resize (
-                       allocation.get_x(), allocation.get_y(), allocation.get_width(), allocation.get_height()
-                       );
+       /* signal flow from 0 to 1 */
+       if (_find_inputs_for_io_outputs) {
+               _other = 1;
+               _ours = 0;
+       } else {
+               _other = 0;
+               _ours = 1;
        }
-}
 
-void
-RotatedLabelSet::on_realize ()
-{
-       Gtk::Widget::on_realize ();
-
-       Glib::RefPtr<Gtk::Style> style = get_style ();
-
-       if (!_gdk_window) {
-               GdkWindowAttr attributes;
-               memset (&attributes, 0, sizeof (attributes));
-
-               Gtk::Allocation allocation = get_allocation ();
-               attributes.x = allocation.get_x ();
-               attributes.y = allocation.get_y ();
-               attributes.width = allocation.get_width ();
-               attributes.height = allocation.get_height ();
-
-               attributes.event_mask = get_events () | Gdk::EXPOSURE_MASK; 
-               attributes.window_type = GDK_WINDOW_CHILD;
-               attributes.wclass = GDK_INPUT_OUTPUT;
-
-               _gdk_window = Gdk::Window::create (get_window (), &attributes, GDK_WA_X | GDK_WA_Y);
-               unset_flags (Gtk::NO_WINDOW);
-               set_window (_gdk_window);
-
-               _bg_colour = style->get_bg (Gtk::STATE_NORMAL );
-               modify_bg (Gtk::STATE_NORMAL, _bg_colour);
-               _fg_colour = style->get_fg (Gtk::STATE_NORMAL);
-;
-               _gdk_window->set_user_data (gobj ());
-
-               /* Set up Pango stuff */
-               _pango_context = create_pango_context ();
-
-               Pango::Matrix matrix = PANGO_MATRIX_INIT;
-               pango_matrix_rotate (&matrix, _angle_degrees);
-               _pango_context->set_matrix (matrix);
-
-               _pango_layout = Pango::Layout::create (_pango_context);
-               _gc = Gdk::GC::create (get_window ());
-       }
+       _port_group = boost::shared_ptr<PortGroup> (new PortGroup (""));
+       _ports[_ours].add_group (_port_group);
+       
+       setup_all_ports ();
 }
 
 void
-RotatedLabelSet::on_unrealize()
-{
-       _gdk_window.clear ();
-
-       Gtk::Widget::on_unrealize ();
-}
-
-
-/**
- *    Set up our Pango layout to plot a given string, and compute its dimensions once it has been rotated.
- *    @param s String to use.
- *    @return width and height of the rotated string, in pixels.
- */
-
-std::pair<int, int>
-RotatedLabelSet::setup_layout (std::string const & s)
+IOSelector::setup_ports (int dim)
 {
-       _pango_layout->set_text (s);
+       _ports[dim].suspend_signals ();
 
-       /* Here's the unrotated size */
-       int w;
-       int h;
-       _pango_layout->get_pixel_size (w, h);
+       if (dim == _other) {
 
-       /* Rotate the width and height as appropriate.  I thought Pango might be able to do this for us,
-          but I can't find out how... */
-       std::pair<int, int> d;
-       d.first = int (w * cos (_angle_radians) - h * sin (_angle_radians));
-       d.second = int (w * sin (_angle_radians) + h * cos (_angle_radians));
-
-       return d;
-}
-
-bool RotatedLabelSet::on_expose_event (GdkEventExpose* event)
-{
-       if (!_gdk_window) {
-               return true;
-       }
-
-       int const height = get_allocation().get_height ();
-       double const spacing = double (_base_width) / _labels.size();
+               _ports[_other].gather (_session, _find_inputs_for_io_outputs);
+               
+       } else {
 
-       /* Plot all the labels; really we should clip for efficiency */
-       for (uint32_t i = 0; i < _labels.size(); ++i) {
-               std::pair<int, int> const d = setup_layout (_labels[i]);
-               get_window()->draw_layout (_gc, _base_start + int ((i + 0.25) * spacing), height - d.second, _pango_layout, _fg_colour, _bg_colour);
+               _port_group->clear ();
+               _port_group->add_bundle (
+                       _find_inputs_for_io_outputs ? _io->bundle_for_outputs() : _io->bundle_for_inputs()
+                       );
        }
 
-       return true;
-}
-
-void
-RotatedLabelSet::set_n_labels (int n)
-{
-       _labels.resize (n);
-       queue_resize ();
-}
-
-void
-RotatedLabelSet::set_label (int n, std::string const & l)
-{
-       _labels[n] = l;
-       queue_resize ();
-}
-
-std::string
-RotatedLabelSet::get_label (int n) const
-{
-       return _labels[n];
+       _ports[dim].resume_signals ();
 }
 
-/**
- *  Set the `base dimensions'.  These are the dimensions of the area at which the labels start, and
- *  have to be set up to match whatever they are labelling.
- *
- *  Roughly speaking, we have
- *
- *             L L L L
- *            E E E E
- *           B B B B
- *          A A A A
- *         L L L L
- * <--s--><--w--->
- */
-    
 void
-RotatedLabelSet::set_base_dimensions (int s, int w)
-{
-       _base_start = s;
-       _base_width = w;
-       queue_resize ();
-}
-
-
-/**
- *  Construct an IOSelector.
- *  @param session Session to operate on.
- *  @param io IO to operate on.
- *  @param for_input true if the selector is for an input, otherwise false.
- */
-IOSelector::IOSelector (ARDOUR::Session& session, boost::shared_ptr<ARDOUR::IO> io, bool for_input)
-       : _session (session), _io (io), _for_input (for_input), _width (0), _height (0),
-         _ignore_check_button_toggle (false), _add_remove_box_added (false)
+IOSelector::set_state (ARDOUR::BundleChannel c[2], bool s)
 {
-       /* Column labels */
-       pack_start (_column_labels, true, true);
-
-       /* Buttons for adding and removing ports */
-       _add_port_button.set_image (*Gtk::manage (new Gtk::Image (Gtk::Stock::ADD, Gtk::ICON_SIZE_MENU)));
-       _add_port_button.set_label (_("Add port"));
-       _add_port_button.signal_clicked().connect (mem_fun (*this, &IOSelector::add_port_button_clicked));
-       _add_remove_box.pack_start (_add_port_button);
-       _remove_port_button.set_image (*Gtk::manage (new Gtk::Image (Gtk::Stock::REMOVE, Gtk::ICON_SIZE_MENU)));
-       _remove_port_button.set_label (_("Remove port"));
-       _remove_port_button.signal_clicked().connect (mem_fun (*this, &IOSelector::remove_port_button_clicked));
-       _add_remove_box.pack_start (_remove_port_button);
-       set_button_sensitivity ();
-
-       /* Table.  We need to put in a HBox, with a dummy label to its right,
-          so that the rotated column labels can overhang the right hand side of the table. */
-       setup_table_size ();
-       setup_row_labels ();
-       setup_column_labels ();
-       setup_check_button_states ();
-       _table_hbox.pack_start (_table, false, false);
-       _table_hbox.pack_start (_dummy);
-       _table.set_col_spacings (4);
-       pack_start (_table_hbox);
-
-       show_all ();
+       ARDOUR::Bundle::PortList const & our_ports = c[_ours].bundle->channel_ports (c[_ours].channel);
+       ARDOUR::Bundle::PortList const & other_ports = c[_other].bundle->channel_ports (c[_other].channel);
 
-       update_column_label_dimensions ();
+       for (ARDOUR::Bundle::PortList::const_iterator i = our_ports.begin(); i != our_ports.end(); ++i) {
+               for (ARDOUR::Bundle::PortList::const_iterator j = other_ports.begin(); j != other_ports.end(); ++j) {
 
-       /* Listen for ports changing on the IO */
-       if (_for_input) {
-               _io->input_changed.connect (mem_fun(*this, &IOSelector::ports_changed));
-       } else {
-               _io->output_changed.connect (mem_fun(*this, &IOSelector::ports_changed));
-       }
-       
-}
-
-IOSelector::~IOSelector ()
-{
-       for (std::vector<Gtk::Label*>::iterator i = _row_labels.begin(); i != _row_labels.end(); ++i) {
-               delete *i;
-       }
+                       Port* f = _session.engine().get_port_by_name (*i);
+                       if (!f) {
+                               return;
+                       }
 
-       for (uint32_t x = 0; x < _check_buttons.size(); ++x) {
-               for (uint32_t y = 0; y < _check_buttons[x].size(); ++y) {
-                       delete _check_buttons[x][y];
+                       if (s) {
+                               if (!_find_inputs_for_io_outputs) {
+                                       _io->connect_input (f, *j, 0);
+                               } else {
+                                       _io->connect_output (f, *j, 0);
+                               }
+                       } else {
+                               if (!_find_inputs_for_io_outputs) {
+                                       _io->disconnect_input (f, *j, 0);
+                               } else {
+                                       _io->disconnect_output (f, *j, 0);
+                               }
+                       }
                }
        }
 }
 
-
-/**
- *    Sets up the sizing of the column label widget to match the table that's underneath it.
- */
-
-void
-IOSelector::update_column_label_dimensions ()
+PortMatrix::State
+IOSelector::get_state (ARDOUR::BundleChannel c[2]) const
 {
-       if (_row_labels.empty() || _check_buttons.empty() || _check_buttons[0].empty()) {
-               return;
-       }
-       
-       _column_labels.set_base_dimensions (
-               /* width of the row label + a column spacing */
-               _row_labels.front()->get_width() + _table.get_col_spacing (0),
-               /* width of a check button + a column spacing for each column */
-               _check_buttons.size() * ( _check_buttons[0][0]->get_width() + _table.get_col_spacing(1))
-               );
-}
-
-void
-IOSelector::setup_table_size ()
-{
-       if (_add_remove_box_added) {
-               _table.remove (_add_remove_box);
-       }
-       
-       /* New width */
-       
-       int const old_width = _width;
-       _width = 0;
-       
-       const char **ports = _session.engine().get_ports (
-               "", _io->default_type().to_jack_type(), _for_input ? JackPortIsOutput : JackPortIsInput
-               );
-
-       if (ports) {
-               while (ports[_width]) {
-                       ++_width;
-               }
-       }
-
-       /* New height */
-
-       int const old_height = _height;
-
-       ARDOUR::DataType const t = _io->default_type();
-       
-       if (_for_input) {
-               _height = _io->n_inputs().get(t);
-       } else {
-               _height = _io->n_outputs().get(t);
-       }
+       ARDOUR::Bundle::PortList const & our_ports = c[_ours].bundle->channel_ports (c[_ours].channel);
+       ARDOUR::Bundle::PortList const & other_ports = c[_other].bundle->channel_ports (c[_other].channel);
 
-       _table.resize (_width + 1, _height + 1);
+       for (ARDOUR::Bundle::PortList::const_iterator i = our_ports.begin(); i != our_ports.end(); ++i) {
+               for (ARDOUR::Bundle::PortList::const_iterator j = other_ports.begin(); j != other_ports.end(); ++j) {
 
-       /* Add checkboxes where required, and remove those that aren't */
-       for (int x = _width; x < old_width; ++x) {
-               for (int y = 0; y < old_height; ++y) {
-                       delete _check_buttons[x][y];
-               }
-       }
-       _check_buttons.resize (_width);
-       
-       for (int x = 0; x < _width; x++) {
+                       Port* f = _session.engine().get_port_by_name (*i);
 
-               for (int y = _height; y < old_height; ++y) {
-                       delete _check_buttons[x][y];
-               }
-               _check_buttons[x].resize (_height);
-               
-               for (int y = 0; y < _height; y++) {
-
-                       if (x >= old_width || y >= old_height) {
-                               Gtk::CheckButton* button = new Gtk::CheckButton;
-                               button->signal_toggled().connect (
-                                       sigc::bind (sigc::mem_fun (*this, &IOSelector::check_button_toggled), x, y)
-                                       );
-                               _check_buttons[x][y] = button;
-                               _table.attach (*button, x + 1, x + 2, y, y + 1);
+                       /* since we are talking about an IO, our ports should all have an associated Port *,
+                          so the above call should never fail */
+                       assert (f);
+                       
+                       if (!f->connected_to (*j)) {
+                               /* if any one thing is not connected, all bets are off */
+                               return NOT_ASSOCIATED;
                        }
                }
        }
 
-       /* Add more row labels where required, and remove those that aren't */
-       for (int y = _height; y < old_height; ++y) {
-               delete _row_labels[y];
-       }
-       _row_labels.resize (_height);
-       for (int y = old_height; y < _height; ++y) {
-               Gtk::Label* label = new Gtk::Label;
-               _row_labels[y] = label;
-               _table.attach (*label, 0, 1, y, y + 1);
-       }
-
-       _table.attach (_add_remove_box, 0, 1, _height, _height + 1);
-       _add_remove_box_added = true;
-
-       show_all ();
+       return ASSOCIATED;
 }
 
-
-/**
- *  Write the correct text to the row labels.
- */
-
-void
-IOSelector::setup_row_labels ()
+uint32_t
+IOSelector::n_io_ports () const
 {
-       for (int y = 0; y < _height; y++) {
-               _row_labels[y]->set_text (_for_input ? _io->input(y)->name() : _io->output(y)->name());
+       if (!_find_inputs_for_io_outputs) {
+               return _io->inputs().num_ports (_io->default_type());
+       } else {
+               return _io->outputs().num_ports (_io->default_type());
        }
 }
 
-/**
- *  Write the correct text to the column labels.
- */
-
-void
-IOSelector::setup_column_labels ()
+uint32_t
+IOSelector::maximum_io_ports () const
 {
-       /* Find the ports that we can connect to */
-       const char **ports = _session.engine().get_ports (
-               "", _io->default_type().to_jack_type(), _for_input ? JackPortIsOutput : JackPortIsInput
-               );
-
-       if (ports == 0) {
-               return;
-       }
-
-       int n = 0;
-       while (ports[n]) {
-               ++n;
-       }
-                       
-       _column_labels.set_n_labels (n);
-
-       for (int i = 0; i < n; ++i) {
-               _column_labels.set_label (i, ports[i]);
-       }
-}
-
-
-/**
- *  Set up the state of each check button according to what connections are made.
- */
-
-void
-IOSelector::setup_check_button_states ()
-{
-       _ignore_check_button_toggle = true;
-
-       /* Set the state of the check boxes according to current connections */
-       for (int i = 0; i < _height; ++i) {
-               const char **connections = _for_input ? _io->input(i)->get_connections() : _io->output(i)->get_connections();
-               for (int j = 0; j < _width; ++j) {
-
-                       std::string const t = _column_labels.get_label (j);
-                       int k = 0;
-                       bool required_state = false;
-
-                       while (connections && connections[k]) {
-                               if (std::string(connections[k]) == t) {
-                                       required_state = true;
-                                       break;
-                               }
-                               ++k;
-                       }
-
-                       _check_buttons[j][i]->set_active (required_state);
-               }
+       if (!_find_inputs_for_io_outputs) {
+               return _io->input_maximum ().get (_io->default_type());
+       } else {
+               return _io->output_maximum ().get (_io->default_type());
        }
-
-       _ignore_check_button_toggle = false;
 }
 
 
-/**
- *  Handle a toggle of a check button.
- */
-
-void
-IOSelector::check_button_toggled (int x, int y)
+uint32_t
+IOSelector::minimum_io_ports () const
 {
-       if (_ignore_check_button_toggle) {
-               return;
-       }
-       
-       bool const new_state = _check_buttons[x][y]->get_active ();
-       std::string const port = _column_labels.get_label (x);
-
-       if (new_state) {
-               if (_for_input) {
-                       _io->connect_input (_io->input(y), port, 0);
-               } else {
-                       _io->connect_output (_io->output(y), port, 0);
-               }
+       if (!_find_inputs_for_io_outputs) {
+               return _io->input_minimum ().get (_io->default_type());
        } else {
-               if (_for_input) {
-                       _io->disconnect_input (_io->input(y), port, 0);
-               } else {
-                       _io->disconnect_output (_io->output(y), port, 0);
-               }
+               return _io->output_minimum ().get (_io->default_type());
        }
 }
 
 void
-IOSelector::add_port_button_clicked ()
+IOSelector::add_channel (boost::shared_ptr<ARDOUR::Bundle> b)
 {
-       /* add a new port, then hide the button if we're up to the maximum allowed */
-
+       /* we ignore the bundle parameter, as we know what it is that we're adding to */
+       
        // The IO selector only works for single typed IOs
-       const ARDOUR::DataType t = _io->default_type();
+       const ARDOUR::DataType t = _io->default_type ();
 
-       if (_for_input) {
+       if (!_find_inputs_for_io_outputs) {
 
                try {
                        _io->add_input_port ("", this);
                }
 
-               catch (ARDOUR::AudioEngine::PortRegistrationFailure& err) {
-                       Gtk::MessageDialog msg (0,  _("There are no more JACK ports available."));
+               catch (AudioEngine::PortRegistrationFailure& err) {
+                       MessageDialog msg (_("There are no more JACK ports available."));
                        msg.run ();
                }
 
@@ -523,154 +193,117 @@ IOSelector::add_port_button_clicked ()
                        _io->add_output_port ("", this);
                }
 
-               catch (ARDOUR::AudioEngine::PortRegistrationFailure& err) {
-                       Gtk::MessageDialog msg (0, _("There are no more JACK ports available."));
+               catch (AudioEngine::PortRegistrationFailure& err) {
+                       MessageDialog msg (_("There are no more JACK ports available."));
                        msg.run ();
                }
        }
-               
-       set_button_sensitivity ();
 }
 
-
 void
-IOSelector::remove_port_button_clicked ()
+IOSelector::remove_channel (ARDOUR::BundleChannel bc)
 {
-       uint32_t nports;
-
-       // The IO selector only works for single typed IOs
-       const ARDOUR::DataType t = _io->default_type();
-       
-       // always remove last port
-       
-       if (_for_input) {
-               if ((nports = _io->n_inputs().get(t)) > 0) {
-                       _io->remove_input_port (_io->input(nports - 1), this);
-               }
-       } else {
-               if ((nports = _io->n_outputs().get(t)) > 0) {
-                       _io->remove_output_port (_io->output(nports - 1), this);
-               }
+       Port* f = _session.engine().get_port_by_name (bc.bundle->channel_ports(bc.channel)[0]);
+       if (!f) {
+               return;
        }
        
-       set_button_sensitivity ();
-}
-
-
-void 
-IOSelector::set_button_sensitivity ()
-{
-       ARDOUR::DataType const t = _io->default_type();
-
-       if (_for_input) {
-
-               _add_port_button.set_sensitive (
-                       _io->input_maximum().get(t) > _io->n_inputs().get(t)
-                       );
+       if (_find_inputs_for_io_outputs) {
+               _io->remove_output_port (f, this);
        } else {
-
-               _add_port_button.set_sensitive (
-                       _io->output_maximum().get(t) > _io->n_outputs().get(t)
-                       );
-       }
-
-       if (_for_input) {
-               
-               _remove_port_button.set_sensitive (
-                       _io->n_inputs().get(t) && _io->input_minimum().get(t) < _io->n_inputs().get(t)
-                       );
-       } else {
-
-               _remove_port_button.set_sensitive (
-                       _io->n_outputs().get(t) && _io->output_minimum().get(t) < _io->n_outputs().get(t)
-                       );
+               _io->remove_input_port (f, this);
        }
 }
 
-void
-IOSelector::ports_changed (ARDOUR::IOChange change, void *src)
+bool
+IOSelector::list_is_global (int dim) const
 {
-       ENSURE_GUI_THREAD (bind (mem_fun (*this, &IOSelector::ports_changed), change, src));
-
-       redisplay ();
+       return (dim == _other);
 }
 
-
-void
-IOSelector::redisplay ()
-{
-       setup_table_size ();
-       setup_column_labels ();
-       setup_row_labels ();
-       setup_check_button_states ();
-       update_column_label_dimensions ();
-}
-
-
-
-IOSelectorWindow::IOSelectorWindow (
-       ARDOUR::Session& session, boost::shared_ptr<ARDOUR::IO> io, bool for_input, bool can_cancel
-       )
-       : ArdourDialog ("I/O selector"),
-         _selector (session, io, for_input),
-         ok_button (can_cancel ? _("OK"): _("Close")),
-         cancel_button (_("Cancel")),
-         rescan_button (_("Rescan"))
+IOSelectorWindow::IOSelectorWindow (ARDOUR::Session& session, boost::shared_ptr<ARDOUR::IO> io, bool for_input, bool can_cancel)
+       : ArdourDialog ("I/O selector")
+       , _selector (session, io, !for_input)
+       , add_button (_("Add Port"))
+       , disconnect_button (_("Disconnect All"))
+       , ok_button (can_cancel ? _("OK"): _("Close"))
+       , cancel_button (_("Cancel"))
+       , rescan_button (_("Rescan"))
 
 {
+       /* XXX: what's this for? */
        add_events (Gdk::KEY_PRESS_MASK | Gdk::KEY_RELEASE_MASK);
+       
        set_name ("IOSelectorWindow2");
 
-       string title;
-       if (for_input) {
-               title = string_compose(_("%1 input"), io->name());
-       } else {
-               title = string_compose(_("%1 output"), io->name());
-       }
-
-       ok_button.set_name ("IOSelectorButton");
-       if (!can_cancel) {
-               ok_button.set_image (*Gtk::manage (new Gtk::Image (Gtk::Stock::CLOSE, Gtk::ICON_SIZE_BUTTON)));
-       }
-       cancel_button.set_name ("IOSelectorButton");
-       rescan_button.set_name ("IOSelectorButton");
+       /* Disconnect All button */
+       disconnect_button.set_name ("IOSelectorButton");
+       disconnect_button.set_image (*Gtk::manage (new Gtk::Image (Gtk::Stock::DISCONNECT, Gtk::ICON_SIZE_BUTTON)));
+       disconnect_button.signal_clicked().connect (sigc::mem_fun (_selector, &IOSelector::disassociate_all));
+       get_action_area()->pack_start (disconnect_button, false, false);
+
+       /* Add Port button */
+       if (_selector.maximum_io_ports() > _selector.n_io_ports()) {
+               add_button.set_name ("IOSelectorButton");
+               add_button.set_image (*Gtk::manage (new Gtk::Image (Gtk::Stock::ADD, Gtk::ICON_SIZE_BUTTON)));
+               get_action_area()->pack_start (add_button, false, false);
+               add_button.signal_clicked().connect (sigc::bind (sigc::mem_fun (_selector, &IOSelector::add_channel), boost::shared_ptr<Bundle> ()));
+       } 
+
+       /* Rescan button */
+       rescan_button.set_name ("IOSelectorButton");
        rescan_button.set_image (*Gtk::manage (new Gtk::Image (Gtk::Stock::REFRESH, Gtk::ICON_SIZE_BUTTON)));
-
+       rescan_button.signal_clicked().connect (sigc::mem_fun (*this, &IOSelectorWindow::rescan));
        get_action_area()->pack_start (rescan_button, false, false);
 
+       io->PortCountChanged.connect (sigc::hide (mem_fun (*this, &IOSelectorWindow::ports_changed)));
+
+       /* Cancel button */
        if (can_cancel) {
+               cancel_button.set_name ("IOSelectorButton");
                cancel_button.set_image (*Gtk::manage (new Gtk::Image (Gtk::Stock::CANCEL, Gtk::ICON_SIZE_BUTTON)));
                get_action_area()->pack_start (cancel_button, false, false);
        } else {
                cancel_button.hide();
        }
-               
+       cancel_button.signal_clicked().connect (mem_fun(*this, &IOSelectorWindow::cancel));
+
+       /* OK button */
+       ok_button.set_name ("IOSelectorButton");
+       if (!can_cancel) {
+               ok_button.set_image (*Gtk::manage (new Gtk::Image (Gtk::Stock::CLOSE, Gtk::ICON_SIZE_BUTTON)));
+       }
+       ok_button.signal_clicked().connect (mem_fun(*this, &IOSelectorWindow::accept));
        get_action_area()->pack_start (ok_button, false, false);
 
        get_vbox()->set_spacing (8);
-       get_vbox()->pack_start (_selector);
 
-       ok_button.signal_clicked().connect (mem_fun(*this, &IOSelectorWindow::accept));
-       cancel_button.signal_clicked().connect (mem_fun(*this, &IOSelectorWindow::cancel));
-       rescan_button.signal_clicked().connect (mem_fun(*this, &IOSelectorWindow::rescan));
+       get_vbox()->pack_start (_selector, true, true);
 
-       set_title (title);
        set_position (Gtk::WIN_POS_MOUSE);
 
+       io_name_changed (this);
+       ports_changed ();
+
        show_all ();
 
-       signal_delete_event().connect (bind (sigc::ptr_fun (just_hide_it), reinterpret_cast<Window *> (this)));
+       signal_delete_event().connect (bind (sigc::ptr_fun (just_hide_it), this));
 }
 
-IOSelectorWindow::~IOSelectorWindow()
+void
+IOSelectorWindow::ports_changed ()
 {
-       
+       if (_selector.maximum_io_ports() > _selector.n_io_ports()) {
+               add_button.set_sensitive (true);
+       } else {
+               add_button.set_sensitive (false);
+       }
 }
 
 void
 IOSelectorWindow::rescan ()
 {
-       _selector.redisplay ();
+       _selector.setup_ports (_selector.other());
 }
 
 void
@@ -690,26 +323,39 @@ IOSelectorWindow::accept ()
 void
 IOSelectorWindow::on_map ()
 {
-       _selector.redisplay ();
+       _selector.setup_all_ports ();
        Window::on_map ();
 }
 
+void
+IOSelectorWindow::io_name_changed (void* src)
+{
+       ENSURE_GUI_THREAD(bind (mem_fun(*this, &IOSelectorWindow::io_name_changed), src));
+       
+       string title;
+
+       if (!_selector.find_inputs_for_io_outputs()) {
+               title = string_compose(_("%1 input"), _selector.io()->name());
+       } else {
+               title = string_compose(_("%1 output"), _selector.io()->name());
+       }
+
+       set_title (title);
+}
 
 PortInsertUI::PortInsertUI (ARDOUR::Session& sess, boost::shared_ptr<ARDOUR::PortInsert> pi)
        : input_selector (sess, pi->io(), true),
          output_selector (sess, pi->io(), false)
 {
-       hbox.pack_start (output_selector, true, true);
-       hbox.pack_start (input_selector, true, true);
-
-       pack_start (hbox);
+       pack_start (output_selector, true, true);
+       pack_start (input_selector, true, true);
 }
 
 void
 PortInsertUI::redisplay ()
 {
-       input_selector.redisplay();
-       output_selector.redisplay();
+       input_selector.setup_ports (input_selector.other());
+       output_selector.setup_ports (output_selector.other());
 }
 
 void
@@ -754,7 +400,6 @@ PortInsertWindow::PortInsertWindow (ARDOUR::Session& sess, boost::shared_ptr<ARD
 
        ok_button.signal_clicked().connect (mem_fun (*this, &PortInsertWindow::accept));
        cancel_button.signal_clicked().connect (mem_fun (*this, &PortInsertWindow::cancel));
-       rescan_button.signal_clicked().connect (mem_fun (*this, &PortInsertWindow::rescan));
 
        signal_delete_event().connect (bind (sigc::ptr_fun (just_hide_it), reinterpret_cast<Window *> (this))); 
 
@@ -778,12 +423,6 @@ PortInsertWindow::on_map ()
 }
 
 
-void
-PortInsertWindow::rescan ()
-{
-       _portinsertui.redisplay ();
-}
-
 void
 PortInsertWindow::cancel ()
 {
@@ -797,3 +436,4 @@ PortInsertWindow::accept ()
        _portinsertui.finished (IOSelector::Accepted);
        hide ();
 }
+