Fix scrollwheel operation in editor.
[ardour.git] / gtk2_ardour / automation_time_axis.cc
index 465945fadff25b07df84d15dc65a9ac096284445..eaed1e519951e80127ea4a8e42488392fc64219d 100644 (file)
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2000-2007 Paul Davis 
+    Copyright (C) 2000-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 <utility>
-#include <ardour/route.h>
-#include <ardour/automation_control.h>
-#include <pbd/memento_command.h>
 #include <gtkmm2ext/barcontroller.h>
+#include "pbd/memento_command.h"
+#include "ardour/automation_control.h"
+#include "ardour/event_type_map.h"
+#include "ardour/route.h"
+#include "ardour/session.h"
 
 #include "ardour_ui.h"
 #include "automation_time_axis.h"
+#include "automation_streamview.h"
+#include "route_time_axis.h"
 #include "automation_line.h"
 #include "public_editor.h"
 #include "simplerect.h"
 #include "selection.h"
-#include "ghostregion.h"
 #include "rgb_macros.h"
 #include "automation_selectable.h"
 #include "point_selection.h"
 
 #include "i18n.h"
 
+using namespace std;
 using namespace ARDOUR;
 using namespace PBD;
 using namespace Gtk;
 using namespace Gtkmm2ext;
 using namespace Editing;
 
-Pango::FontDescription AutomationTimeAxisView::name_font;
+Pango::FontDescription* AutomationTimeAxisView::name_font = 0;
 bool AutomationTimeAxisView::have_name_font = false;
+const string AutomationTimeAxisView::state_node_name = "AutomationChild";
 
+
+/** \a a the automatable object this time axis is to display data for.
+ * For route/track automation (e.g. gain) pass the route for both \r and \a.
+ * For route child (e.g. plugin) automation, pass the child for \a.
+ * For region automation (e.g. MIDI CC), pass null for \a.
+ */
 AutomationTimeAxisView::AutomationTimeAxisView (Session& s, boost::shared_ptr<Route> r,
                boost::shared_ptr<Automatable> a, boost::shared_ptr<AutomationControl> c,
-               PublicEditor& e, TimeAxisView& rent, 
-               ArdourCanvas::Canvas& canvas, const string & nom, 
-               const string & state_name, const string & nomparent)
-
-       : AxisView (s), 
-         TimeAxisView (s, e, &rent, canvas),
+               PublicEditor& e, TimeAxisView& parent, bool show_regions,
+               ArdourCanvas::Canvas& canvas, const string & nom, const string & nomparent)
+       : AxisView (s),
+         TimeAxisView (s, e, &parent, canvas),
          _route (r),
          _control (c),
          _automatable (a),
-         _controller(AutomationController::create(s, c->list(), c)),
+         _controller(AutomationController::create(a, c->parameter(), c)),
          _base_rect (0),
-         _xml_node (0),
+         _view (show_regions ? new AutomationStreamView(*this) : NULL),
          _name (nom),
-         _state_name (state_name),
-         height_button (_("h")),
-         clear_button (_("clear")),
          auto_button (X_("")) /* force addition of a label */
 {
        if (!have_name_font) {
@@ -73,41 +79,47 @@ AutomationTimeAxisView::AutomationTimeAxisView (Session& s, boost::shared_ptr<Ro
        }
 
        automation_menu = 0;
-       in_destructor = false;
        auto_off_item = 0;
        auto_touch_item = 0;
        auto_write_item = 0;
        auto_play_item = 0;
+       mode_discrete_item = 0;
+       mode_line_item = 0;
+
        ignore_state_request = false;
        first_call_to_set_height = true;
-       
-       _base_rect = new SimpleRect(*canvas_display);
+
+       _base_rect = new SimpleRect(*_canvas_display);
        _base_rect->property_x1() = 0.0;
        _base_rect->property_y1() = 0.0;
-       _base_rect->property_x2() = editor.frame_to_pixel (max_frames);
+       _base_rect->property_x2() = LONG_MAX - 2;
        _base_rect->property_outline_color_rgba() = ARDOUR_UI::config()->canvasvar_AutomationTrackOutline.get();
-       
+
        /* outline ends and bottom */
        _base_rect->property_outline_what() = (guint32) (0x1|0x2|0x8);
        _base_rect->property_fill_color_rgba() = ARDOUR_UI::config()->canvasvar_AutomationTrackFill.get();
-       
+
        _base_rect->set_data ("trackview", this);
 
-       _base_rect->signal_event().connect (bind (mem_fun (editor, &PublicEditor::canvas_automation_track_event),
-                                                _base_rect, this));
+       _base_rect->signal_event().connect (bind (
+                       mem_fun (_editor, &PublicEditor::canvas_automation_track_event),
+                       _base_rect, this));
+
+       if (!a) {
+               _base_rect->lower_to_bottom();
+       }
 
        hide_button.add (*(manage (new Gtk::Image (::get_icon("hide")))));
 
-       height_button.set_name ("TrackSizeButton");
        auto_button.set_name ("TrackVisualButton");
-       clear_button.set_name ("TrackVisualButton");
        hide_button.set_name ("TrackRemoveButton");
 
+       auto_button.unset_flags (Gtk::CAN_FOCUS);
+       hide_button.unset_flags (Gtk::CAN_FOCUS);
+
        controls_table.set_no_show_all();
 
-       ARDOUR_UI::instance()->tooltips().set_tip(height_button, _("track height"));
        ARDOUR_UI::instance()->tooltips().set_tip(auto_button, _("automation state"));
-       ARDOUR_UI::instance()->tooltips().set_tip(clear_button, _("clear track"));
        ARDOUR_UI::instance()->tooltips().set_tip(hide_button, _("hide track"));
 
        /* rearrange the name display */
@@ -124,7 +136,7 @@ AutomationTimeAxisView::AutomationTimeAxisView (Session& s, boost::shared_ptr<Ro
        bool shortened = false;
 
        int ignore_width;
-       shortpname = fit_to_pixels (_name, 60, name_font, ignore_width, true);
+       shortpname = fit_to_pixels (_name, 60, *name_font, ignore_width, true);
 
        if (shortpname != _name ){
                shortened = true;
@@ -137,12 +149,12 @@ AutomationTimeAxisView::AutomationTimeAxisView (Session& s, boost::shared_ptr<Ro
 
                /* limit the plug name string */
 
-               string pname = fit_to_pixels (nomparent, 60, name_font, ignore_width, true);
+               string pname = fit_to_pixels (nomparent, 60, *name_font, ignore_width, true);
                if (pname != nomparent) {
                        shortened = true;
                }
 
-               plugname = new Label (pname);
+               plugname = new Label (pname);
                plugname->set_name (X_("TrackPlugName"));
                plugname->show();
                name_label.set_name (X_("TrackParameterName"));
@@ -163,21 +175,17 @@ AutomationTimeAxisView::AutomationTimeAxisView (Session& s, boost::shared_ptr<Ro
                tipname += _name;
                ARDOUR_UI::instance()->tooltips().set_tip(controls_ebox, tipname);
        }
-       
+
        /* add the buttons */
        controls_table.attach (hide_button, 0, 1, 0, 1, Gtk::FILL|Gtk::EXPAND, Gtk::FILL|Gtk::EXPAND);
-       controls_table.attach (height_button, 0, 1, 1, 2, Gtk::FILL|Gtk::EXPAND, Gtk::FILL|Gtk::EXPAND);
 
        controls_table.attach (auto_button, 5, 8, 0, 1, Gtk::FILL|Gtk::EXPAND, Gtk::FILL|Gtk::EXPAND);
-       controls_table.attach (clear_button, 5, 8, 1, 2, Gtk::FILL|Gtk::EXPAND, Gtk::FILL|Gtk::EXPAND);
-       
+
        /* add bar controller */
-       controls_table.attach (*_controller.get(), 0, 8, 2, 3, Gtk::FILL|Gtk::EXPAND, Gtk::FILL|Gtk::EXPAND);
+       controls_table.attach (*_controller.get(), 0, 8, 1, 2, Gtk::FILL|Gtk::EXPAND, Gtk::FILL|Gtk::EXPAND);
 
        controls_table.show_all ();
 
-       height_button.signal_clicked().connect (mem_fun(*this, &AutomationTimeAxisView::height_clicked));
-       clear_button.signal_clicked().connect (mem_fun(*this, &AutomationTimeAxisView::clear_clicked));
        hide_button.signal_clicked().connect (mem_fun(*this, &AutomationTimeAxisView::hide_clicked));
        auto_button.signal_clicked().connect (mem_fun(*this, &AutomationTimeAxisView::auto_clicked));
 
@@ -185,23 +193,31 @@ AutomationTimeAxisView::AutomationTimeAxisView (Session& s, boost::shared_ptr<Ro
        controls_base_unselected_name = X_("AutomationTrackControlsBase");
        controls_ebox.set_name (controls_base_unselected_name);
 
-       controls_frame.set_shadow_type (Gtk::SHADOW_ETCHED_OUT);
-
-       XMLNode* xml_node = get_parent_with_state()->get_child_xml_node (_state_name);
+       XMLNode* xml_node = get_parent_with_state()->get_automation_child_xml_node (
+                       _control->parameter());
 
        if (xml_node) {
-               set_state (*xml_node);
-       } 
-               
-       boost::shared_ptr<AutomationLine> line(new AutomationLine (
-                               _control->list()->param_id().to_string(),
-                               *this,
-                               *canvas_display,
-                               _control->list()));
-               
-       line->set_line_color (ARDOUR_UI::config()->canvasvar_ProcessorAutomationLine.get());
-       line->queue_reset ();
-       add_line (line);
+               set_state (*xml_node, Stateful::loading_state_version);
+       }
+
+       /* ask for notifications of any new RegionViews */
+       if (show_regions) {
+
+               assert(_view);
+               _view->attach ();
+
+       /* no regions, just a single line for the entire track (e.g. bus gain) */
+       } else {
+               boost::shared_ptr<AutomationLine> line(new AutomationLine (
+                                       ARDOUR::EventTypeMap::instance().to_symbol(_control->parameter()),
+                                       *this,
+                                       *_canvas_display,
+                                       _control->alist()));
+
+               line->set_line_color (ARDOUR_UI::config()->canvasvar_ProcessorAutomationLine.get());
+               line->queue_reset ();
+               add_line (line);
+       }
 
        /* make sure labels etc. are correct */
 
@@ -211,11 +227,6 @@ AutomationTimeAxisView::AutomationTimeAxisView (Session& s, boost::shared_ptr<Ro
 
 AutomationTimeAxisView::~AutomationTimeAxisView ()
 {
-       in_destructor = true;
-
-       for (list<GhostRegion*>::iterator i = ghosts.begin(); i != ghosts.end(); ++i) {
-               delete *i;
-       }
 }
 
 void
@@ -228,14 +239,14 @@ AutomationTimeAxisView::auto_clicked ()
                automation_menu->set_name ("ArdourContextMenu");
                MenuList& items (automation_menu->items());
 
-               items.push_back (MenuElem (_("Manual"), 
-                                          bind (mem_fun(*this, &AutomationTimeAxisView::set_automation_state), (AutoState) Off)));
-               items.push_back (MenuElem (_("Play"),
-                                          bind (mem_fun(*this, &AutomationTimeAxisView::set_automation_state), (AutoState) Play)));
-               items.push_back (MenuElem (_("Write"),
-                                          bind (mem_fun(*this, &AutomationTimeAxisView::set_automation_state), (AutoState) Write)));
-               items.push_back (MenuElem (_("Touch"),
-                                          bind (mem_fun(*this, &AutomationTimeAxisView::set_automation_state), (AutoState) Touch)));
+               items.push_back (MenuElem (_("Manual"), bind (mem_fun(*this,
+                               &AutomationTimeAxisView::set_automation_state), (AutoState) Off)));
+               items.push_back (MenuElem (_("Play"), bind (mem_fun(*this,
+                               &AutomationTimeAxisView::set_automation_state), (AutoState) Play)));
+               items.push_back (MenuElem (_("Write"), bind (mem_fun(*this,
+                               &AutomationTimeAxisView::set_automation_state), (AutoState) Write)));
+               items.push_back (MenuElem (_("Touch"), bind (mem_fun(*this,
+                               &AutomationTimeAxisView::set_automation_state), (AutoState) Touch)));
        }
 
        automation_menu->popup (1, gtk_get_current_event_time());
@@ -245,14 +256,21 @@ void
 AutomationTimeAxisView::set_automation_state (AutoState state)
 {
        if (!ignore_state_request) {
-               if (_route == _automatable) { // FIXME: ew
-                       _route->set_parameter_automation_state (
-                                       _control->list()->param_id(),
-                                       state);
+               if (_automatable) {
+                       _automatable->set_parameter_automation_state (_control->parameter(), state);
+               }
+#if 0
+               if (_route == _automatable) { // This is a time axis for route (not region) automation
+                       _route->set_parameter_automation_state (_control->parameter(), state);
                }
 
-               _control->list()->set_automation_state(state);
+               if (_control->list())
+                       _control->alist()->set_automation_state(state);
+#endif
+       }
 
+       if (_view) {
+               _view->set_automation_state (state);
        }
 }
 
@@ -266,7 +284,7 @@ AutomationTimeAxisView::automation_state_changed ()
        if (!_line) {
                state = Off;
        } else {
-               state = _control->list()->automation_state ();
+               state = _control->alist()->automation_state ();
        }
 
        switch (state & (Off|Play|Touch|Write)) {
@@ -321,152 +339,146 @@ AutomationTimeAxisView::automation_state_changed ()
 }
 
 void
-AutomationTimeAxisView::height_clicked ()
+AutomationTimeAxisView::interpolation_changed ()
+{
+       AutomationList::InterpolationStyle style = _control->list()->interpolation();
+
+       if (mode_line_item && mode_discrete_item) {
+               if (style == AutomationList::Discrete) {
+                       mode_discrete_item->set_active(true);
+                       mode_line_item->set_active(false);
+               } else {
+                       mode_line_item->set_active(true);
+                       mode_discrete_item->set_active(false);
+               }
+       }
+
+       if (_line) {
+               _line->set_interpolation(style);
+       }
+}
+
+void
+AutomationTimeAxisView::set_interpolation (AutomationList::InterpolationStyle style)
 {
-       popup_size_menu (0);
+       _control->list()->set_interpolation(style);
+       if (_line) {
+               _line->set_interpolation(style);
+       }
 }
 
 void
 AutomationTimeAxisView::clear_clicked ()
 {
        _session.begin_reversible_command (_("clear automation"));
-       _line->clear ();
+       if (_line) {
+               _line->clear ();
+       }
        _session.commit_reversible_command ();
 }
 
 void
-AutomationTimeAxisView::set_height (TrackHeight ht)
+AutomationTimeAxisView::set_height (uint32_t h)
 {
-       uint32_t h = height_to_pixels (ht);
-       bool changed = (height != (uint32_t) h);
-
-       //bool changed_between_small_and_normal = ( (ht == Small || ht == Smaller) ^ (height_style == Small || height_style == Smaller) );
+       bool changed = (height != (uint32_t) h) || first_call_to_set_height;
+       bool changed_between_small_and_normal = (
+                  (height < hNormal && h >= hNormal)
+               || (height >= hNormal || h < hNormal) );
 
        TimeAxisView* state_parent = get_parent_with_state ();
-       XMLNode* xml_node = state_parent->get_child_xml_node (_state_name);
+       assert(state_parent);
+       XMLNode* xml_node = state_parent->get_automation_child_xml_node (_control->parameter());
 
-       TimeAxisView::set_height (ht);
+       TimeAxisView::set_height (h);
        _base_rect->property_y2() = h;
 
        if (_line)
-               _line->set_y_position_and_height (0, h);
+               _line->set_height(h);
 
-       for (list<GhostRegion*>::iterator i = ghosts.begin(); i != ghosts.end(); ++i) {
-               (*i)->set_height ();
+       if (_view) {
+               _view->set_height(h);
+               _view->update_contents_height();
        }
 
+       char buf[32];
+       snprintf (buf, sizeof (buf), "%u", height);
+       if (xml_node) {
+               xml_node->add_property ("height", buf);
+       }
 
-       switch (ht) {
-       case Largest:
-               xml_node->add_property ("track_height", "largest");
-               break;
-
-       case Large:
-               xml_node->add_property ("track_height", "large");
-               break;
-
-       case Larger:
-               xml_node->add_property ("track_height", "larger");
-               break;
-
-       case Normal:
-               xml_node->add_property ("track_height", "normal");
-               break;
+       if (changed_between_small_and_normal || first_call_to_set_height) {
 
-       case Smaller:
-               xml_node->add_property ("track_height", "smaller");
-               break;
+               first_call_to_set_height = false;
 
-       case Small:
-               xml_node->add_property ("track_height", "small");
-               break;
-       }
+               if (h >= hNormal) {
+                       controls_table.remove (name_hbox);
 
-       //if (changed_between_small_and_normal || first_call_to_set_height) {
-               first_call_to_set_height = false;
-               switch (ht) {
-                       case Normal:
-                               _controller->hide();
-
-                               controls_table.remove (name_hbox);
-
-                               if (plugname) {
-                                       if (plugname_packed) {
-                                               controls_table.remove (*plugname);
-                                               plugname_packed = false;
-                                       }
-                                       controls_table.attach (*plugname, 1, 5, 0, 1, Gtk::FILL|Gtk::EXPAND, Gtk::FILL|Gtk::EXPAND);
-                                       plugname_packed = true;
-                                       controls_table.attach (name_hbox, 1, 5, 1, 2, Gtk::FILL|Gtk::EXPAND, Gtk::FILL|Gtk::EXPAND);
-                               } else {
-                                       controls_table.attach (name_hbox, 1, 5, 0, 1, Gtk::FILL|Gtk::EXPAND, Gtk::FILL|Gtk::EXPAND);
-                               }
-                               hide_name_entry ();
-                               show_name_label ();
-                               name_hbox.show_all ();
-
-                               auto_button.show();
-                               height_button.show();
-                               clear_button.show();
-                               hide_button.show_all();
-                               break;
-                       
-                       case Large:
-                       case Larger:
-                       case Largest:
-                               _controller->show ();
-                               break;
-
-                       case Smaller:
-                               _controller->hide();
-                       
-                       case Small:
-
-                               controls_table.remove (name_hbox);
-                               if (plugname) {
-                                       if (plugname_packed) {
-                                               controls_table.remove (*plugname);
-                                               plugname_packed = false;
-                                       }
+                       if (plugname) {
+                               if (plugname_packed) {
+                                       controls_table.remove (*plugname);
+                                       plugname_packed = false;
                                }
+                               controls_table.attach (*plugname, 1, 5, 0, 1, Gtk::FILL|Gtk::EXPAND, Gtk::FILL|Gtk::EXPAND);
+                               plugname_packed = true;
+                               controls_table.attach (name_hbox, 1, 5, 1, 2, Gtk::FILL|Gtk::EXPAND, Gtk::FILL|Gtk::EXPAND);
+                       } else {
                                controls_table.attach (name_hbox, 1, 5, 0, 1, Gtk::FILL|Gtk::EXPAND, Gtk::FILL|Gtk::EXPAND);
-                               controls_table.hide_all ();
-                               hide_name_entry ();
-                               show_name_label ();
-                               name_hbox.show_all ();
-
-                               auto_button.hide();
-                               height_button.hide();
-                               clear_button.hide();
-                               hide_button.hide();
-                               break;
+                       }
+                       hide_name_entry ();
+                       show_name_label ();
+                       name_hbox.show_all ();
+
+                       auto_button.show();
+                       hide_button.show_all();
+
+               } else if (h >= hSmall) {
+                       controls_table.remove (name_hbox);
+                       if (plugname) {
+                               if (plugname_packed) {
+                                       controls_table.remove (*plugname);
+                                       plugname_packed = false;
+                               }
+                       }
+                       controls_table.attach (name_hbox, 1, 5, 0, 1, Gtk::FILL|Gtk::EXPAND, Gtk::FILL|Gtk::EXPAND);
+                       controls_table.hide_all ();
+                       hide_name_entry ();
+                       show_name_label ();
+                       name_hbox.show_all ();
+
+                       auto_button.hide();
+                       hide_button.hide();
                }
-
-       //}
+       } else if (h >= hNormal){
+               cerr << "track grown, but neither changed_between_small_and_normal nor first_call_to_set_height set!" << endl;
+       }
 
        if (changed) {
                /* only emit the signal if the height really changed */
-               _route->gui_changed ("track_height", (void *) 0); /* EMIT_SIGNAL */
+               _route->gui_changed ("visible_tracks", (void *) 0); /* EMIT_SIGNAL */
        }
 }
 
 void
 AutomationTimeAxisView::set_samples_per_unit (double spu)
 {
-       TimeAxisView::set_samples_per_unit (editor.get_current_zoom());
+       TimeAxisView::set_samples_per_unit (spu);
+
+       if (_line)
+               _line->reset ();
 
-       _line->reset ();
+       if (_view)
+               _view->set_samples_per_unit (spu);
 }
+
 void
 AutomationTimeAxisView::hide_clicked ()
 {
        // LAME fix for refreshing the hide button
        hide_button.set_sensitive(false);
-       
+
        set_marked_for_display (false);
        hide ();
-       
+
        hide_button.set_sensitive(true);
 }
 
@@ -494,39 +506,78 @@ AutomationTimeAxisView::build_display_menu ()
        items.push_back (MenuElem (_("Clear"), mem_fun(*this, &AutomationTimeAxisView::clear_clicked)));
        items.push_back (SeparatorElem());
 
+       /* state menu */
+
        Menu* auto_state_menu = manage (new Menu);
        auto_state_menu->set_name ("ArdourContextMenu");
        MenuList& as_items = auto_state_menu->items();
-       
-       as_items.push_back (CheckMenuElem (_("Manual"), 
-                                          bind (mem_fun(*this, &AutomationTimeAxisView::set_automation_state), (AutoState) Off)));
+
+       as_items.push_back (CheckMenuElem (_("Manual"), bind (
+                       mem_fun(*this, &AutomationTimeAxisView::set_automation_state),
+                       (AutoState) Off)));
        auto_off_item = dynamic_cast<CheckMenuItem*>(&as_items.back());
 
-       as_items.push_back (CheckMenuElem (_("Play"),
-                                          bind (mem_fun(*this, &AutomationTimeAxisView::set_automation_state), (AutoState) Play)));
+       as_items.push_back (CheckMenuElem (_("Play"), bind (
+                       mem_fun(*this, &AutomationTimeAxisView::set_automation_state),
+                       (AutoState) Play)));
        auto_play_item = dynamic_cast<CheckMenuItem*>(&as_items.back());
 
-       as_items.push_back (CheckMenuElem (_("Write"),
-                                          bind (mem_fun(*this, &AutomationTimeAxisView::set_automation_state), (AutoState) Write)));
+       as_items.push_back (CheckMenuElem (_("Write"), bind (
+                       mem_fun(*this, &AutomationTimeAxisView::set_automation_state),
+                       (AutoState) Write)));
        auto_write_item = dynamic_cast<CheckMenuItem*>(&as_items.back());
 
-       as_items.push_back (CheckMenuElem (_("Touch"),
-                                          bind (mem_fun(*this, &AutomationTimeAxisView::set_automation_state), (AutoState) Touch)));
+       as_items.push_back (CheckMenuElem (_("Touch"), bind (
+                       mem_fun(*this, &AutomationTimeAxisView::set_automation_state),
+                       (AutoState) Touch)));
        auto_touch_item = dynamic_cast<CheckMenuItem*>(&as_items.back());
 
        items.push_back (MenuElem (_("State"), *auto_state_menu));
 
+       /* mode menu */
+
+       if (EventTypeMap::instance().is_midi_parameter(_control->parameter())) {
+
+               Menu* auto_mode_menu = manage (new Menu);
+               auto_mode_menu->set_name ("ArdourContextMenu");
+               MenuList& am_items = auto_mode_menu->items();
+
+               RadioMenuItem::Group group;
+
+               am_items.push_back (RadioMenuElem (group, _("Discrete"), bind (
+                               mem_fun(*this, &AutomationTimeAxisView::set_interpolation),
+                               AutomationList::Discrete)));
+               mode_discrete_item = dynamic_cast<CheckMenuItem*>(&am_items.back());
+               mode_discrete_item->set_active(_control->list()->interpolation() == AutomationList::Discrete);
+
+               am_items.push_back (RadioMenuElem (group, _("Linear"), bind (
+                               mem_fun(*this, &AutomationTimeAxisView::set_interpolation),
+                               AutomationList::Linear)));
+               mode_line_item = dynamic_cast<CheckMenuItem*>(&am_items.back());
+
+               // Set default interpolation type to linear if this isn't a (usually) discrete controller
+               if (EventTypeMap::instance().interpolation_of(_control->parameter()) == Evoral::ControlList::Linear) {
+                       mode_line_item->set_active(_control->list()->interpolation() == AutomationList::Linear);
+               }
+
+               items.push_back (MenuElem (_("Mode"), *auto_mode_menu));
+       }
+
        /* make sure the automation menu state is correct */
 
        automation_state_changed ();
+       interpolation_changed ();
 }
 
 void
-AutomationTimeAxisView::add_automation_event (ArdourCanvas::Item* item, GdkEvent* event, nframes_t when, double y)
+AutomationTimeAxisView::add_automation_event (ArdourCanvas::Item* /*item*/, GdkEvent* /*event*/, nframes_t when, double y)
 {
+       if (!_line)
+               return;
+
        double x = 0;
 
-       canvas_display->w2i (x, y);
+       _canvas_display->w2i (x, y);
 
        /* compute vertical fractional position */
 
@@ -534,30 +585,29 @@ AutomationTimeAxisView::add_automation_event (ArdourCanvas::Item* item, GdkEvent
 
        /* map using line */
 
-       _line->view_to_model_y (y);
+       _line->view_to_model_coord (x, y);
 
        _session.begin_reversible_command (_("add automation event"));
-       XMLNode& before = _control->list()->get_state();
+       XMLNode& before = _control->alist()->get_state();
 
-       _control->list()->add (when, y);
+       _control->alist()->add (when, y);
 
-       XMLNode& after = _control->list()->get_state();
-       _session.commit_reversible_command (new MementoCommand<ARDOUR::AutomationList>(*_control->list().get(), &before, &after));
+       XMLNode& after = _control->alist()->get_state();
+       _session.commit_reversible_command (new MementoCommand<ARDOUR::AutomationList>(*_control->alist(), &before, &after));
 
        _session.set_dirty ();
 }
 
-
 bool
 AutomationTimeAxisView::cut_copy_clear (Selection& selection, CutCopyOp op)
 {
-       return cut_copy_clear_one (*_line, selection, op);
+       return (_line ? cut_copy_clear_one (*_line, selection, op) : false);
 }
 
 bool
 AutomationTimeAxisView::cut_copy_clear_one (AutomationLine& line, Selection& selection, CutCopyOp op)
 {
-       AutomationList* what_we_got = 0;
+       boost::shared_ptr<Evoral::ControlList> what_we_got;
        boost::shared_ptr<AutomationList> alist (line.the_list());
        bool ret = false;
 
@@ -566,22 +616,20 @@ AutomationTimeAxisView::cut_copy_clear_one (AutomationLine& line, Selection& sel
        switch (op) {
        case Cut:
                if ((what_we_got = alist->cut (selection.time.front().start, selection.time.front().end)) != 0) {
-                       editor.get_cut_buffer().add (what_we_got);
+                       _editor.get_cut_buffer().add (what_we_got);
                        _session.add_command(new MementoCommand<AutomationList>(*alist.get(), &before, &alist->get_state()));
                        ret = true;
                }
                break;
        case Copy:
                if ((what_we_got = alist->copy (selection.time.front().start, selection.time.front().end)) != 0) {
-                       editor.get_cut_buffer().add (what_we_got);
+                       _editor.get_cut_buffer().add (what_we_got);
                }
                break;
 
        case Clear:
                if ((what_we_got = alist->cut (selection.time.front().start, selection.time.front().end)) != 0) {
                        _session.add_command(new MementoCommand<AutomationList>(*alist.get(), &before, &alist->get_state()));
-                       delete what_we_got;
-                       what_we_got = 0;
                        ret = true;
                }
                break;
@@ -589,9 +637,11 @@ AutomationTimeAxisView::cut_copy_clear_one (AutomationLine& line, Selection& sel
 
        if (what_we_got) {
                for (AutomationList::iterator x = what_we_got->begin(); x != what_we_got->end(); ++x) {
-                       double foo = (*x)->value;
-                       line.model_to_view_y (foo);
-                       (*x)->value = foo;
+                       double when = (*x)->when;
+                       double val  = (*x)->value;
+                       line.model_to_view_coord (when, val);
+                       (*x)->when = when;
+                       (*x)->value = val;
                }
        }
 
@@ -616,7 +666,7 @@ AutomationTimeAxisView::reset_objects_one (AutomationLine& line, PointSelection&
                if (&(*i).track != this) {
                        continue;
                }
-               
+
                alist->reset_range ((*i).start, (*i).end);
        }
 }
@@ -630,7 +680,7 @@ AutomationTimeAxisView::cut_copy_clear_objects (PointSelection& selection, CutCo
 bool
 AutomationTimeAxisView::cut_copy_clear_objects_one (AutomationLine& line, PointSelection& selection, CutCopyOp op)
 {
-       AutomationList* what_we_got = 0;
+       boost::shared_ptr<Evoral::ControlList> what_we_got;
        boost::shared_ptr<AutomationList> alist(line.the_list());
        bool ret = false;
 
@@ -645,22 +695,20 @@ AutomationTimeAxisView::cut_copy_clear_objects_one (AutomationLine& line, PointS
                switch (op) {
                case Cut:
                        if ((what_we_got = alist->cut ((*i).start, (*i).end)) != 0) {
-                               editor.get_cut_buffer().add (what_we_got);
+                               _editor.get_cut_buffer().add (what_we_got);
                                _session.add_command (new MementoCommand<AutomationList>(*alist.get(), new XMLNode (before), &alist->get_state()));
                                ret = true;
                        }
                        break;
                case Copy:
                        if ((what_we_got = alist->copy ((*i).start, (*i).end)) != 0) {
-                               editor.get_cut_buffer().add (what_we_got);
+                               _editor.get_cut_buffer().add (what_we_got);
                        }
                        break;
-                       
+
                case Clear:
                        if ((what_we_got = alist->cut ((*i).start, (*i).end)) != 0) {
                                _session.add_command (new MementoCommand<AutomationList>(*alist.get(), new XMLNode (before), &alist->get_state()));
-                               delete what_we_got;
-                               what_we_got = 0;
                                ret = true;
                        }
                        break;
@@ -671,9 +719,11 @@ AutomationTimeAxisView::cut_copy_clear_objects_one (AutomationLine& line, PointS
 
        if (what_we_got) {
                for (AutomationList::iterator x = what_we_got->begin(); x != what_we_got->end(); ++x) {
-                       double foo = (*x)->value;
-                       line.model_to_view_y (foo);
-                       (*x)->value = foo;
+                       double when = (*x)->when;
+                       double val  = (*x)->value;
+                       line.model_to_view_coord (when, val);
+                       (*x)->when = when;
+                       (*x)->value = val;
                }
        }
 
@@ -691,8 +741,8 @@ AutomationTimeAxisView::paste_one (AutomationLine& line, nframes_t pos, float ti
 {
        AutomationSelection::iterator p;
        boost::shared_ptr<AutomationList> alist(line.the_list());
-       
-       for (p = selection.lines.begin(); p != selection.lines.end() && nth; ++p, --nth);
+
+       for (p = selection.lines.begin(); p != selection.lines.end() && nth; ++p, --nth) {}
 
        if (p == selection.lines.end()) {
                return false;
@@ -702,13 +752,15 @@ AutomationTimeAxisView::paste_one (AutomationLine& line, nframes_t pos, float ti
           values from view coordinates to model coordinates, and we're
           not supposed to modify the points in the selection.
        */
-          
+
        AutomationList copy (**p);
 
        for (AutomationList::iterator x = copy.begin(); x != copy.end(); ++x) {
-               double foo = (*x)->value;
-               line.view_to_model_y (foo);
-               (*x)->value = foo;
+               double when = (*x)->when;
+               double val  = (*x)->value;
+               line.view_to_model_coord (when, val);
+               (*x)->when = when;
+               (*x)->value = val;
        }
 
        XMLNode &before = alist->get_state();
@@ -718,30 +770,6 @@ AutomationTimeAxisView::paste_one (AutomationLine& line, nframes_t pos, float ti
        return true;
 }
 
-void
-AutomationTimeAxisView::add_ghost (GhostRegion* gr)
-{
-       ghosts.push_back (gr);
-       gr->GoingAway.connect (mem_fun(*this, &AutomationTimeAxisView::remove_ghost));
-}
-
-void
-AutomationTimeAxisView::remove_ghost (GhostRegion* gr)
-{
-       if (in_destructor) {
-               return;
-       }
-
-       list<GhostRegion*>::iterator i;
-
-       for (i = ghosts.begin(); i != ghosts.end(); ++i) {
-               if ((*i) == gr) {
-                       ghosts.erase (i);
-                       break;
-               }
-       }
-}
-
 void
 AutomationTimeAxisView::get_selectables (nframes_t start, nframes_t end, double top, double bot, list<Selectable*>& results)
 {
@@ -750,14 +778,14 @@ AutomationTimeAxisView::get_selectables (nframes_t start, nframes_t end, double
                double botfrac;
 
                /* remember: this is X Window - coordinate space starts in upper left and moves down.
-                  y_position is the "origin" or "top" of the track.
+                  _y_position is the "origin" or "top" of the track.
                */
 
-               double mybot = y_position + height;
+               double mybot = _y_position + height;
 
-               if (y_position >= top && mybot <= bot) {
+               if (_y_position >= top && mybot <= bot) {
 
-                       /* y_position is below top, mybot is above bot, so we're fully
+                       /* _y_position is below top, mybot is above bot, so we're fully
                           covered vertically.
                        */
 
@@ -766,26 +794,29 @@ AutomationTimeAxisView::get_selectables (nframes_t start, nframes_t end, double
 
                } else {
 
-                       /* top and bot are within y_position .. mybot */
+                       /* top and bot are within _y_position .. mybot */
 
-                       topfrac = 1.0 - ((top - y_position) / height);
-                       botfrac = 1.0 - ((bot - y_position) / height);
+                       topfrac = 1.0 - ((top - _y_position) / height);
+                       botfrac = 1.0 - ((bot - _y_position) / height);
                }
 
-               _line->get_selectables (start, end, botfrac, topfrac, results);
+               if (_line)
+                       _line->get_selectables (start, end, botfrac, topfrac, results);
        }
 }
 
 void
 AutomationTimeAxisView::get_inverted_selectables (Selection& sel, list<Selectable*>& result)
 {
-       _line->get_inverted_selectables (sel, result);
+       if (_line)
+               _line->get_inverted_selectables (sel, result);
 }
 
 void
 AutomationTimeAxisView::set_selected_points (PointSelection& points)
 {
-       _line->set_selected_points (points);
+       if (_line)
+               _line->set_selected_points (points);
 }
 
 void
@@ -802,13 +833,13 @@ AutomationTimeAxisView::add_line (boost::shared_ptr<AutomationLine> line)
        assert(!_line);
        assert(line->the_list() == _control->list());
 
-       automation_connection = _control->list()->automation_state_changed.connect
+       automation_connection = _control->alist()->automation_state_changed.connect
                (mem_fun(*this, &AutomationTimeAxisView::automation_state_changed));
 
        _line = line;
        //_controller = AutomationController::create(_session, line->the_list(), _control);
 
-       line->set_y_position_and_height (0, height);
+       line->set_height (height);
 
        /* pick up the current state */
        automation_state_changed ();
@@ -816,75 +847,47 @@ AutomationTimeAxisView::add_line (boost::shared_ptr<AutomationLine> line)
        line->show();
 }
 
-void
-AutomationTimeAxisView::show_all_control_points ()
-{
-       _line->show_all_control_points ();
-}
-
-void
-AutomationTimeAxisView::hide_all_but_selected_control_points ()
-{
-       _line->hide_all_but_selected_control_points ();
-}
-
 void
 AutomationTimeAxisView::entered()
 {
-       show_all_control_points ();
+       if (_line)
+               _line->track_entered();
 }
 
 void
 AutomationTimeAxisView::exited ()
 {
-       hide_all_but_selected_control_points ();
+       if (_line)
+               _line->track_exited();
 }
 
 void
-AutomationTimeAxisView::set_colors ()
+AutomationTimeAxisView::color_handler ()
 {
-    for (list<GhostRegion*>::iterator i=ghosts.begin(); i != ghosts.end(); i++ ) {
-               (*i)->set_colors();
-    }
-    
-       _line->set_colors();
+       if (_line) {
+               _line->set_colors();
+       }
 }
 
-void
-AutomationTimeAxisView::color_handler () 
+int
+AutomationTimeAxisView::set_state (const XMLNode& node, int version)
 {
-       set_colors ();
-}
+       TimeAxisView::set_state (node, version);
 
-void
-AutomationTimeAxisView::set_state (const XMLNode& node)
-{
-       TimeAxisView::set_state (node);
-       
-       XMLNodeList kids;
-       XMLNodeConstIterator iter;
-
-       kids = node.children ();
-
-       //snprintf (buf, sizeof(buf), "Port_%" PRIu32, param.id());
-               
-       for (iter = kids.begin(); iter != kids.end(); ++iter) {
-               if ((*iter)->name() == _control->list()->param_id().to_string()) {
-               
-                       XMLProperty *shown = (*iter)->property("shown_editor");
-                       
-                       if (shown && shown->value() == "yes") {
-                               set_marked_for_display(true);
-                               canvas_display->show(); /* FIXME: necessary? show_at? */
-                       }
-                       break;
+       XMLProperty const * type = node.property ("automation-id");
+       if (type && type->value () == ARDOUR::EventTypeMap::instance().to_symbol (_control->parameter())) {
+               XMLProperty const * shown = node.property ("shown");
+               if (shown && shown->value () == "yes") {
+                       set_marked_for_display (true);
+                       _canvas_display->show (); /* FIXME: necessary? show_at? */
                }
        }
 
-       if (!_marked_for_display)
+       if (!_marked_for_display) {
                hide();
+       }
 
-       // FIXME: _xml_node = &node?
+       return 0;
 }
 
 XMLNode*
@@ -893,61 +896,26 @@ AutomationTimeAxisView::get_state_node ()
        TimeAxisView* state_parent = get_parent_with_state ();
 
        if (state_parent) {
-               return state_parent->get_child_xml_node (_state_name);
+               return state_parent->get_automation_child_xml_node (_control->parameter());
        } else {
                return 0;
        }
 }
 
-void
-AutomationTimeAxisView::ensure_xml_node ()
-{
-       if ((_automatable != _route) && _xml_node == 0) {
-               if ((_xml_node = _automatable->extra_xml ("GUI")) == 0) {
-                       _xml_node = new XMLNode ("GUI");
-                       _automatable->add_extra_xml (*_xml_node);
-               }
-       }
-}
-
 void
 AutomationTimeAxisView::update_extra_xml_shown (bool editor_shown)
 {
-       if (_automatable == _route)
-               return;
-
-       char buf[32];
-       
-       ensure_xml_node ();
-
-       XMLNodeList nlist = _xml_node->children ();
-       XMLNodeConstIterator i;
-       XMLNode * port_node = 0;
-
-       /* FIXME: these parsed XML node names need to go */
-       //snprintf (buf, sizeof(buf), "Port_%" PRIu32, _param.id());
-
-       for (i = nlist.begin(); i != nlist.end(); ++i) {
-               /* FIXME: legacy session loading */
-               if ((*i)->name() == _control->list()->param_id().to_string()) {
-                       port_node = (*i);
-                       break;
-               }
-       }
-
-       if (!port_node) {
-               port_node = new XMLNode(buf);
-               _xml_node->add_child_nocopy(*port_node);
+       XMLNode* xml_node = get_state_node();
+       if (xml_node) {
+               xml_node->add_property ("shown", editor_shown ? "yes" : "no");
        }
-       
-       port_node->add_property ("shown_editor", editor_shown ? "yes": "no");
 }
 
 guint32
 AutomationTimeAxisView::show_at (double y, int& nth, Gtk::VBox *parent)
 {
        update_extra_xml_shown (true);
-       
+
        return TimeAxisView::show_at (y, nth, parent);
 }