an awful lot of tweaks to drawing details
[ardour.git] / gtk2_ardour / audio_region_view.cc
index a3b99de3f168c78c64778a216ed88a40cc0d6c5d..077755ff83a3bba949f544342deac35dd450eab0 100644 (file)
@@ -20,7 +20,7 @@
 #include <cassert>
 #include <algorithm>
 
-#include <boost/scoped_ptr.hpp>
+#include <boost/scoped_array.hpp>
 
 #include <gtkmm.h>
 
 
 #include "evoral/Curve.hpp"
 
+#include "canvas/rectangle.h"
+#include "canvas/polygon.h"
+#include "canvas/poly_line.h"
+#include "canvas/line.h"
+#include "canvas/text.h"
+#include "canvas/debug.h"
+#include "canvas/utils.h"
+
 #include "streamview.h"
 #include "audio_region_view.h"
 #include "audio_time_axis.h"
-#include "simplerect.h"
-#include "simpleline.h"
-#include "waveview.h"
 #include "public_editor.h"
 #include "audio_region_editor.h"
+#include "audio_streamview.h"
 #include "region_gain_line.h"
 #include "control_point.h"
 #include "ghostregion.h"
@@ -65,6 +71,7 @@ using namespace Editing;
 using namespace ArdourCanvas;
 
 static const int32_t sync_mark_width = 9;
+static double const handle_size = 15; /* height of fade handles */
 
 AudioRegionView::AudioRegionView (ArdourCanvas::Group *parent, RouteTimeAxisView &tv, boost::shared_ptr<AudioRegion> r, double spu,
                                  Gdk::Color const & basic_color)
@@ -74,17 +81,18 @@ AudioRegionView::AudioRegionView (ArdourCanvas::Group *parent, RouteTimeAxisView
        , fade_out_shape(0)
        , fade_in_handle(0)
        , fade_out_handle(0)
-       , fade_position_line(0)
        , start_xfade_in (0)
        , start_xfade_out (0)
        , start_xfade_rect (0)
+       , _start_xfade_visible (false)
        , end_xfade_in (0)
        , end_xfade_out (0)
        , end_xfade_rect (0)
+       , _end_xfade_visible (false)
        , _amplitude_above_axis(1.0)
-       , _flags(0)
        , fade_color(0)
 {
+       Config->ParameterChanged.connect (*this, invalidator (*this), boost::bind (&AudioRegionView::parameter_changed, this, _1), gui_context());
 }
 
 AudioRegionView::AudioRegionView (ArdourCanvas::Group *parent, RouteTimeAxisView &tv, boost::shared_ptr<AudioRegion> r, double spu,
@@ -95,17 +103,18 @@ AudioRegionView::AudioRegionView (ArdourCanvas::Group *parent, RouteTimeAxisView
        , fade_out_shape(0)
        , fade_in_handle(0)
        , fade_out_handle(0)
-       , fade_position_line(0)
        , start_xfade_in (0)
        , start_xfade_out (0)
        , start_xfade_rect (0)
+       , _start_xfade_visible (false)
        , end_xfade_in (0)
        , end_xfade_out (0)
        , end_xfade_rect (0)
+       , _end_xfade_visible (false)
        , _amplitude_above_axis(1.0)
-       , _flags(0)
        , fade_color(0)
 {
+       Config->ParameterChanged.connect (*this, invalidator (*this), boost::bind (&AudioRegionView::parameter_changed, this, _1), gui_context());
 }
 
 AudioRegionView::AudioRegionView (const AudioRegionView& other, boost::shared_ptr<AudioRegion> other_region)
@@ -114,15 +123,15 @@ AudioRegionView::AudioRegionView (const AudioRegionView& other, boost::shared_pt
        , fade_out_shape(0)
        , fade_in_handle(0)
        , fade_out_handle(0)
-       , fade_position_line(0)
        , start_xfade_in (0)
        , start_xfade_out (0)
        , start_xfade_rect (0)
+       , _start_xfade_visible (false)
        , end_xfade_in (0)
        , end_xfade_out (0)
        , end_xfade_rect (0)
+       , _end_xfade_visible (false)
        , _amplitude_above_axis (other._amplitude_above_axis)
-       , _flags (other._flags)
        , fade_color(0)
 {
        Gdk::Color c;
@@ -132,6 +141,8 @@ AudioRegionView::AudioRegionView (const AudioRegionView& other, boost::shared_pt
        c.set_rgb_p (r/255.0, g/255.0, b/255.0);
 
        init (c, true);
+
+       Config->ParameterChanged.connect (*this, invalidator (*this), boost::bind (&AudioRegionView::parameter_changed, this, _1), gui_context());
 }
 
 void
@@ -142,49 +153,36 @@ AudioRegionView::init (Gdk::Color const & basic_color, bool wfd)
 
        RegionView::init (basic_color, wfd);
 
-       XMLNode *node;
-
        _amplitude_above_axis = 1.0;
-       _flags                = 0;
-
-       if ((node = _region->extra_xml ("GUI")) != 0) {
-               set_flags (node);
-       } else {
-               _flags = WaveformVisible;
-               store_flags ();
-       }
 
        compute_colors (basic_color);
 
        create_waves ();
 
-       fade_in_shape = new ArdourCanvas::Polygon (*group);
-       fade_in_shape->property_fill_color_rgba() = fade_color;
+       fade_in_shape = new ArdourCanvas::Polygon (group);
+       CANVAS_DEBUG_NAME (fade_in_shape, string_compose ("fade in shape for %1", region()->name()));
+       fade_in_shape->set_fill_color (fade_color);
        fade_in_shape->set_data ("regionview", this);
 
-       fade_out_shape = new ArdourCanvas::Polygon (*group);
-       fade_out_shape->property_fill_color_rgba() = fade_color;
+       fade_out_shape = new ArdourCanvas::Polygon (group);
+       CANVAS_DEBUG_NAME (fade_out_shape, string_compose ("fade out shape for %1", region()->name()));
+       fade_out_shape->set_fill_color (fade_color);
        fade_out_shape->set_data ("regionview", this);
 
        if (!_recregion) {
-               fade_in_handle = new ArdourCanvas::SimpleRect (*group);
-               fade_in_handle->property_fill_color_rgba() = UINT_RGBA_CHANGE_A (fill_color, 0);
-               fade_in_handle->property_outline_pixels() = 0;
+               fade_in_handle = new ArdourCanvas::Rectangle (group);
+               CANVAS_DEBUG_NAME (fade_in_handle, string_compose ("fade in handle for %1", region()->name()));
+               fade_in_handle->set_fill_color (UINT_RGBA_CHANGE_A (fill_color, 0));
+               fade_in_handle->set_outline_color (RGBA_TO_UINT (0, 0, 0, 0));
 
                fade_in_handle->set_data ("regionview", this);
 
-               fade_out_handle = new ArdourCanvas::SimpleRect (*group);
-               fade_out_handle->property_fill_color_rgba() = UINT_RGBA_CHANGE_A (fill_color, 0);
-               fade_out_handle->property_outline_pixels() = 0;
+               fade_out_handle = new ArdourCanvas::Rectangle (group);
+               CANVAS_DEBUG_NAME (fade_out_handle, string_compose ("fade out handle for %1", region()->name()));
+               fade_out_handle->set_fill_color (UINT_RGBA_CHANGE_A (fill_color, 0));
+               fade_out_handle->set_outline_color (RGBA_TO_UINT (0, 0, 0, 0));
 
                fade_out_handle->set_data ("regionview", this);
-
-               fade_position_line = new ArdourCanvas::SimpleLine (*group);
-               fade_position_line->property_color_rgba() = 0xBBBBBBAA;
-               fade_position_line->property_y1() = 7;
-               fade_position_line->property_y2() = _height - TimeAxisViewItem::NAME_HIGHLIGHT_SIZE - 1;
-
-               fade_position_line->hide();
        }
 
        setup_fade_handle_positions ();
@@ -198,13 +196,8 @@ AudioRegionView::init (Gdk::Color const & basic_color, bool wfd)
        if (!Profile->get_sae()) {
                gain_line.reset (new AudioRegionGainLine (line_name, *this, *group, audio_region()->envelope()));
        }
-
-       if (Config->get_show_region_gain()) {
-               gain_line->show ();
-       } else {
-               gain_line->hide ();
-       }
-
+       
+       update_envelope_visibility ();
        gain_line->reset ();
 
        set_height (trackview.current_height());
@@ -213,7 +206,11 @@ AudioRegionView::init (Gdk::Color const & basic_color, bool wfd)
        region_sync_changed ();
 
        region_resized (ARDOUR::bounds_change);
-       set_waveview_data_src();
+
+       for (vector<GhostRegion*>::iterator i = ghosts.begin(); i != ghosts.end(); ++i) {
+               (*i)->set_duration (_region->length() / samples_per_pixel);
+       }
+
        region_locked ();
        envelope_active_changed ();
        fade_in_active_changed ();
@@ -221,19 +218,30 @@ AudioRegionView::init (Gdk::Color const & basic_color, bool wfd)
 
        reset_width_dependent_items (_pixel_width);
 
-       fade_in_shape->signal_event().connect (sigc::bind (sigc::mem_fun (PublicEditor::instance(), &PublicEditor::canvas_fade_in_event), fade_in_shape, this));
+       fade_in_shape->Event.connect (sigc::bind (sigc::mem_fun (PublicEditor::instance(), &PublicEditor::canvas_fade_in_event), fade_in_shape, this));
        if (fade_in_handle) {
-               fade_in_handle->signal_event().connect (sigc::bind (sigc::mem_fun (PublicEditor::instance(), &PublicEditor::canvas_fade_in_handle_event), fade_in_handle, this));
+               fade_in_handle->Event.connect (sigc::bind (sigc::mem_fun (PublicEditor::instance(), &PublicEditor::canvas_fade_in_handle_event), fade_in_handle, this));
        }
 
-       fade_out_shape->signal_event().connect (sigc::bind (sigc::mem_fun (PublicEditor::instance(), &PublicEditor::canvas_fade_out_event), fade_out_shape, this));
+       fade_out_shape->Event.connect (sigc::bind (sigc::mem_fun (PublicEditor::instance(), &PublicEditor::canvas_fade_out_event), fade_out_shape, this));
 
        if (fade_out_handle) {
-               fade_out_handle->signal_event().connect (sigc::bind (sigc::mem_fun (PublicEditor::instance(), &PublicEditor::canvas_fade_out_handle_event), fade_out_handle, this));
+               fade_out_handle->Event.connect (sigc::bind (sigc::mem_fun (PublicEditor::instance(), &PublicEditor::canvas_fade_out_handle_event), fade_out_handle, this));
        }
 
        set_colors ();
 
+       setup_waveform_visibility ();
+       setup_waveform_shape ();
+       setup_waveform_scale ();
+
+       if (frame_handle_start) {
+               frame_handle_start->raise_to_top ();
+       }
+       if (frame_handle_end) {
+               frame_handle_end->raise_to_top ();
+       }
+
        /* XXX sync mark drag? */
 }
 
@@ -243,10 +251,6 @@ AudioRegionView::~AudioRegionView ()
 
        RegionViewGoingAway (this); /* EMIT_SIGNAL */
 
-       for (vector<GnomeCanvasWaveViewCache *>::iterator cache = wave_caches.begin(); cache != wave_caches.end() ; ++cache) {
-               gnome_canvas_waveview_cache_destroy (*cache);
-       }
-
        for (vector<ScopedConnection*>::iterator i = _data_ready_connections.begin(); i != _data_ready_connections.end(); ++i) {
                delete *i;
        }
@@ -269,7 +273,6 @@ void
 AudioRegionView::region_changed (const PropertyChange& what_changed)
 {
        ENSURE_GUI_THREAD (*this, &AudioRegionView::region_changed, what_changed);
-       // cerr << "AudioRegionView::region_changed() called" << endl;
 
        RegionView::region_changed (what_changed);
 
@@ -307,15 +310,16 @@ AudioRegionView::fade_out_changed ()
 {
        reset_fade_out_shape ();
 }
+
 void
 AudioRegionView::fade_in_active_changed ()
 {
        if (audio_region()->fade_in_active()) {
-               fade_in_shape->property_fill_color_rgba() = RGBA_TO_UINT(45,45,45,90);                          // FIXME make a themeable colour
-               fade_in_shape->property_width_pixels() = 1;
+               /* XXX: make a themable colour */
+               fade_in_shape->set_fill_color (RGBA_TO_UINT (45, 45, 45, 90));
        } else {
-               fade_in_shape->property_fill_color_rgba() = RGBA_TO_UINT(45,45,45,20);                          // FIXME make a themeable colour
-               fade_in_shape->property_width_pixels() = 1;
+               /* XXX: make a themable colour */
+               fade_in_shape->set_fill_color (RGBA_TO_UINT (45, 45, 45, 20));
        }
 }
 
@@ -323,11 +327,11 @@ void
 AudioRegionView::fade_out_active_changed ()
 {
        if (audio_region()->fade_out_active()) {
-               fade_out_shape->property_fill_color_rgba() = RGBA_TO_UINT(45,45,45,90);                         // FIXME make a themeable colour
-               fade_out_shape->property_width_pixels() = 1;
+               /* XXX: make a themable colour */
+               fade_out_shape->set_fill_color (RGBA_TO_UINT (45, 45, 45, 90));
        } else {
-               fade_out_shape->property_fill_color_rgba() = RGBA_TO_UINT(45,45,45,20);                         // FIXME make a themeable colour
-               fade_out_shape->property_width_pixels() = 1;
+               /* XXX: make a themable colour */
+               fade_out_shape->set_fill_color (RGBA_TO_UINT (45, 45, 45, 20));
        }
 }
 
@@ -335,11 +339,8 @@ AudioRegionView::fade_out_active_changed ()
 void
 AudioRegionView::region_scale_amplitude_changed ()
 {
-       ENSURE_GUI_THREAD (*this, &AudioRegionView::region_scale_amplitude_changed)
-
        for (uint32_t n = 0; n < waves.size(); ++n) {
-               // force a reload of the cache
-               waves[n]->property_data_src() = _region.get();
+               waves[n]->gain_changed ();
        }
 }
 
@@ -372,16 +373,16 @@ AudioRegionView::region_resized (const PropertyChange& what_changed)
        interesting_stuff.add (ARDOUR::Properties::length);
 
        if (what_changed.contains (interesting_stuff)) {
-
+               
                for (uint32_t n = 0; n < waves.size(); ++n) {
-                       waves[n]->property_region_start() = _region->start();
+                       waves[n]->region_resized ();
                }
 
                for (vector<GhostRegion*>::iterator i = ghosts.begin(); i != ghosts.end(); ++i) {
                        if ((agr = dynamic_cast<AudioGhostRegion*>(*i)) != 0) {
 
                                for (vector<WaveView*>::iterator w = agr->waves.begin(); w != agr->waves.end(); ++w) {
-                                       (*w)->property_region_start() = _region->start();
+                                       (*w)->region_resized ();
                                }
                        }
                }
@@ -424,14 +425,15 @@ AudioRegionView::reset_width_dependent_items (double pixel_width)
 
        for (i = analysis_features.begin(), l = feature_lines.begin(); i != analysis_features.end() && l != feature_lines.end(); ++i, ++l) {
 
-               float x_pos = trackview.editor().frame_to_pixel (*i);
+               float x_pos = trackview.editor().sample_to_pixel (*i);
 
-               ArdourCanvas::Points points;
-               points.push_back(Gnome::Art::Point(x_pos, 2.0)); // first x-coord needs to be a non-normal value
-               points.push_back(Gnome::Art::Point(x_pos, _height - TimeAxisViewItem::NAME_HIGHLIGHT_SIZE - 1));
+               (*l).second->set (ArdourCanvas::Duple (x_pos, 2.0),
+                                 ArdourCanvas::Duple (x_pos, _height - TimeAxisViewItem::NAME_HIGHLIGHT_SIZE - 1));
 
                (*l).first = *i;
-               (*l).second->property_points() = points;
+
+               (*l).second->set (ArdourCanvas::Duple (x_pos, 2.0),
+                                 ArdourCanvas::Duple (x_pos, _height - TimeAxisViewItem::NAME_HIGHLIGHT_SIZE - 1));
        }
 
        reset_fade_shapes ();
@@ -441,14 +443,7 @@ void
 AudioRegionView::region_muted ()
 {
        RegionView::region_muted();
-
-       for (uint32_t n=0; n < waves.size(); ++n) {
-               if (_region->muted()) {
-                       waves[n]->property_wave_color() = UINT_RGBA_CHANGE_A(ARDOUR_UI::config()->canvasvar_WaveForm.get(), MUTED_ALPHA);
-               } else {
-                       waves[n]->property_wave_color() = ARDOUR_UI::config()->canvasvar_WaveForm.get();
-               }
-       }
+       set_waveform_colors ();
 }
 
 void
@@ -456,17 +451,15 @@ AudioRegionView::setup_fade_handle_positions()
 {
        /* position of fade handle offset from the top of the region view */
        double const handle_pos = 2;
-       /* height of fade handles */
-       double const handle_height = 5;
 
        if (fade_in_handle) {
-               fade_in_handle->property_y1() = handle_pos;
-               fade_in_handle->property_y2() = handle_pos + handle_height;
+               fade_in_handle->set_y0 (handle_pos);
+               fade_in_handle->set_y1 (handle_pos + handle_size);
        }
 
        if (fade_out_handle) {
-               fade_out_handle->property_y1() = handle_pos;
-               fade_out_handle->property_y2() = handle_pos + handle_height;
+               fade_out_handle->set_y0 (handle_pos);
+               fade_out_handle->set_y1 (handle_pos + handle_size);
        }
 }
 
@@ -488,8 +481,8 @@ AudioRegionView::set_height (gdouble height)
 
                gdouble yoff = n * (ht + 1);
 
-               waves[n]->property_height() = ht;
-               waves[n]->property_y() = yoff + 2;
+               waves[n]->set_height (ht);
+               waves[n]->set_y_position (yoff + 2);
        }
 
        if (gain_line) {
@@ -497,9 +490,7 @@ AudioRegionView::set_height (gdouble height)
                if ((height/wcnt) < NAME_HIGHLIGHT_THRESH) {
                        gain_line->hide ();
                } else {
-                       if (Config->get_show_region_gain ()) {
-                               gain_line->show ();
-                       }
+                       update_envelope_visibility ();
                }
 
                gain_line->set_height ((uint32_t) rint (height - NAME_HIGHLIGHT_SIZE) - 2);
@@ -512,28 +503,14 @@ AudioRegionView::set_height (gdouble height)
 
        for (l = feature_lines.begin(); l != feature_lines.end(); ++l) {
 
-               float pos_x = trackview.editor().frame_to_pixel((*l).first);
-
-               ArdourCanvas::Points points;
-
-               points.push_back(Gnome::Art::Point(pos_x, 2.0)); // first x-coord needs to be a non-normal value
-               points.push_back(Gnome::Art::Point(pos_x, _height - TimeAxisViewItem::NAME_HIGHLIGHT_SIZE - 1));
-
-               (*l).second->property_points() = points;
-       }
-
-       if (fade_position_line) {
+               float pos_x = trackview.editor().sample_to_pixel((*l).first);
 
-               if (height < NAME_HIGHLIGHT_THRESH) {
-                       fade_position_line->property_y2() = _height - 1;
-               }
-               else {
-                       fade_position_line->property_y2() = _height - TimeAxisViewItem::NAME_HIGHLIGHT_SIZE - 1;
-               }
+               (*l).second->set (ArdourCanvas::Duple (pos_x, 2.0),
+                                 ArdourCanvas::Duple (pos_x, _height - TimeAxisViewItem::NAME_HIGHLIGHT_SIZE - 1));
        }
 
-       if (name_pixbuf) {
-               name_pixbuf->raise_to_top();
+       if (name_text) {
+               name_text->raise_to_top();
        }
 }
 
@@ -547,31 +524,12 @@ AudioRegionView::reset_fade_shapes ()
 void
 AudioRegionView::reset_fade_in_shape ()
 {
-       reset_fade_in_shape_width ((framecnt_t) audio_region()->fade_in()->back()->when);
+       reset_fade_in_shape_width (audio_region(), (framecnt_t) audio_region()->fade_in()->back()->when);
 }
 
 void
-AudioRegionView::reset_fade_in_shape_width (framecnt_t width)
+AudioRegionView::reset_fade_in_shape_width (boost::shared_ptr<AudioRegion> ar, framecnt_t width)
 {
-       if (dragging()) {
-               return;
-       }
-
-       if (audio_region()->fade_in_is_xfade()) {
-               if (fade_in_handle) {
-                       fade_in_handle->hide ();
-                       fade_in_shape->hide ();
-               }
-               redraw_start_xfade ();
-               return;
-       } else {
-               if (start_xfade_in) {
-                       start_xfade_in->hide ();
-                       start_xfade_out->hide ();
-                       start_xfade_rect->hide ();
-               }
-       }
-
        if (fade_in_handle == 0) {
                return;
        }
@@ -582,26 +540,16 @@ AudioRegionView::reset_fade_in_shape_width (framecnt_t width)
 
        width = std::max ((framecnt_t) 64, width);
 
-       Points* points;
-
        /* round here to prevent little visual glitches with sub-pixel placement */
-       double const pwidth = rint (width / samples_per_unit);
-       uint32_t npoints = std::min (gdk_screen_width(), (int) pwidth);
-       double h;
-
-       if (_height < 5) {
-               fade_in_shape->hide();
-               fade_in_handle->hide();
-               return;
-       }
-
-       double const handle_center = pwidth;
+       double const pwidth = rint (width / samples_per_pixel);
+       double const handle_left = pwidth;
 
        /* Put the fade in handle so that its left side is at the end-of-fade line */
-       fade_in_handle->property_x1() = handle_center;
-       fade_in_handle->property_x2() = handle_center + 6;
+       fade_in_handle->set_x0 (handle_left);
+       fade_in_handle->set_x1 (handle_left + handle_size);
 
        if (pwidth < 5) {
+               hide_start_xfade();
                fade_in_shape->hide();
                return;
        }
@@ -610,76 +558,74 @@ AudioRegionView::reset_fade_in_shape_width (framecnt_t width)
                fade_in_shape->show();
        }
 
+       uint32_t npoints = std::min (gdk_screen_width(), (int) pwidth);
+       double effective_height;
        float curve[npoints];
-       audio_region()->fade_in()->curve().get_vector (0, audio_region()->fade_in()->back()->when, curve, npoints);
 
-       points = get_canvas_points ("fade in shape", npoints + 3);
+       audio_region()->fade_in()->curve().get_vector (0, audio_region()->fade_in()->back()->when, curve, npoints);
 
        if (_height >= NAME_HIGHLIGHT_THRESH) {
-               h = _height - NAME_HIGHLIGHT_SIZE - 2;
+               effective_height = _height - NAME_HIGHLIGHT_SIZE - 2;
        } else {
-               h = _height;
+               effective_height = _height - 2;
        }
 
+       Points points;
+
+       points.assign (npoints, Duple());
+
        /* points *MUST* be in anti-clockwise order */
 
        uint32_t pi, pc;
        double xdelta = pwidth/npoints;
 
        for (pi = 0, pc = 0; pc < npoints; ++pc) {
-               (*points)[pi].set_x(1 + (pc * xdelta));
-               (*points)[pi++].set_y(2 + (h - (curve[pc] * h)));
+               points[pi].x = 1 + (pc * xdelta);
+               points[pi++].y = 2 + (effective_height - (curve[pc] * effective_height));
        }
 
+       /* draw the line */
+
+       redraw_start_xfade_to (ar, width, points, effective_height);
+
+       /* add 3 more points */
+
+       points.push_back (Duple());
+       points.push_back (Duple());
+       points.push_back (Duple());
+
        /* fold back */
 
-       (*points)[pi].set_x(pwidth);
-       (*points)[pi++].set_y(2);
+       points[pi].x = pwidth;
+       points[pi].y = 2;
+       pi++;
 
-       (*points)[pi].set_x(1);
-       (*points)[pi++].set_y(2);
+       points[pi].x = 1;
+       points[pi].y = 2;
+       pi++;
 
        /* connect the dots ... */
 
-       (*points)[pi] = (*points)[0];
+       points[pi] = points[0];
 
-       fade_in_shape->property_points() = *points;
-       delete points;
+       fade_in_shape->set (points);
 
        /* ensure trim handle stays on top */
        if (frame_handle_start) {
                frame_handle_start->raise_to_top();
        }
+
 }
 
 void
 AudioRegionView::reset_fade_out_shape ()
 {
-       reset_fade_out_shape_width ((framecnt_t) audio_region()->fade_out()->back()->when);
+       reset_fade_out_shape_width (audio_region(), (framecnt_t) audio_region()->fade_out()->back()->when);
 }
 
 void
-AudioRegionView::reset_fade_out_shape_width (framecnt_t width)
+AudioRegionView::reset_fade_out_shape_width (boost::shared_ptr<AudioRegion> ar, framecnt_t width)
 {
-       if (dragging()) {
-               return;
-       }
-
-       if (audio_region()->fade_out_is_xfade()) {
-               if (fade_out_handle) {
-                       fade_out_handle->hide ();
-                       fade_out_shape->hide ();
-               }
-               redraw_end_xfade ();
-               return;
-       } else {
-               if (end_xfade_in) {
-                       end_xfade_in->hide ();
-                       end_xfade_out->hide ();
-                       end_xfade_rect->hide ();
-               }
-       }
-
        if (fade_out_handle == 0) {
                return;
        }
@@ -690,30 +636,20 @@ AudioRegionView::reset_fade_out_shape_width (framecnt_t width)
 
        width = std::max ((framecnt_t) 64, width);
 
-       Points* points;
-
        /* round here to prevent little visual glitches with sub-pixel placement */
-       double const pwidth = rint (width / samples_per_unit);
-       uint32_t npoints = std::min (gdk_screen_width(), (int) pwidth);
-       double h;
+       double const pwidth = rint (width / samples_per_pixel);
 
-       if (_height < 5) {
-               fade_out_shape->hide();
-               fade_out_handle->hide();
-               return;
-       }
-
-       double const handle_center = (_region->length() - width) / samples_per_unit;
+       double const handle_right = (_region->length() / samples_per_pixel) - pwidth;
 
        /* Put the fade out handle so that its right side is at the end-of-fade line;
-        * it's `one out' for precise pixel accuracy.
         */
-       fade_out_handle->property_x1() = handle_center - 5;
-       fade_out_handle->property_x2() = handle_center + 1;
+       fade_out_handle->set_x0 (handle_right - handle_size);
+       fade_out_handle->set_x1 (handle_right);
 
        /* don't show shape if its too small */
 
        if (pwidth < 5) {
+               hide_end_xfade();
                fade_out_shape->hide();
                return;
        }
@@ -722,263 +658,575 @@ AudioRegionView::reset_fade_out_shape_width (framecnt_t width)
                fade_out_shape->show();
        }
 
+       uint32_t npoints = std::min (gdk_screen_width(), (int) pwidth);
+       double effective_height;
        float curve[npoints];
+
        audio_region()->fade_out()->curve().get_vector (0, audio_region()->fade_out()->back()->when, curve, npoints);
 
        if (_height >= NAME_HIGHLIGHT_THRESH) {
-               h = _height - NAME_HIGHLIGHT_SIZE - 2;
+               effective_height = _height - NAME_HIGHLIGHT_SIZE - 2;
        } else {
-               h = _height;
+               effective_height = _height - 2;
        }
 
        /* points *MUST* be in anti-clockwise order */
 
-       points = get_canvas_points ("fade out shape", npoints + 3);
+       Points points;
 
        uint32_t pi, pc;
        double xdelta = pwidth/npoints;
 
-       for (pi = 0, pc = 0; pc < npoints; ++pc) {
-               (*points)[pi].set_x(_pixel_width - pwidth + (pc * xdelta));
-               (*points)[pi++].set_y(2 + (h - (curve[pc] * h)));
+       points.assign (npoints, Duple ());
+
+       for (pi = 0, pc = 0; pc < npoints; ++pc, ++pi) {
+               points[pi].x = _pixel_width - pwidth + (pc * xdelta);
+               points[pi].y = 2 + (effective_height - (curve[pc] * effective_height));
        }
 
+       /* draw the line */
+
+       redraw_end_xfade_to (ar, width, points, effective_height);
+
+       /* fill the polygon*/
+
+       /* add 3 more points */
+
+       points.push_back (Duple());
+       points.push_back (Duple());
+       points.push_back (Duple());
+
        /* fold back */
 
-       (*points)[pi].set_x(_pixel_width);
-       (*points)[pi++].set_y(h);
+       points[pi].x = _pixel_width;
+       points[pi].y = effective_height;
+       pi++;
 
-       (*points)[pi].set_x(_pixel_width);
-       (*points)[pi++].set_y(2);
+       points[pi].x = _pixel_width;
+       points[pi].y = 2;
+       pi++;
 
        /* connect the dots ... */
 
-       (*points)[pi] = (*points)[0];
+       points[pi] = points[0];
 
-       fade_out_shape->property_points() = *points;
-       delete points;
+       fade_out_shape->set (points);
 
        /* ensure trim handle stays on top */
        if (frame_handle_end) {
                frame_handle_end->raise_to_top();
        }
+
 }
 
-void
-AudioRegionView::set_samples_per_unit (gdouble spu)
+framepos_t
+AudioRegionView::get_fade_in_shape_width ()
 {
-       RegionView::set_samples_per_unit (spu);
-
-       if (_flags & WaveformVisible) {
-               for (uint32_t n=0; n < waves.size(); ++n) {
-                       waves[n]->property_samples_per_unit() = spu;
-               }
-       }
-
-       if (gain_line) {
-               gain_line->reset ();
-       }
-
-       reset_fade_shapes ();
+       return audio_region()->fade_in()->back()->when;
 }
 
-void
-AudioRegionView::set_amplitude_above_axis (gdouble spp)
+framepos_t
+AudioRegionView::get_fade_out_shape_width ()
 {
-       for (uint32_t n=0; n < waves.size(); ++n) {
-               waves[n]->property_amplitude_above_axis() = spp;
-       }
+       return audio_region()->fade_out()->back()->when;
 }
 
+
 void
-AudioRegionView::compute_colors (Gdk::Color const & basic_color)
+AudioRegionView::redraw_start_xfade ()
 {
-       RegionView::compute_colors (basic_color);
+       boost::shared_ptr<AudioRegion> ar (audio_region());
 
-       /* gain color computed in envelope_active_changed() */
+       if (!ar->fade_in() || ar->fade_in()->empty()) {
+               return;
+       }
 
-       fade_color = UINT_RGBA_CHANGE_A (fill_color, 120);
+       show_start_xfade();
+       reset_fade_in_shape_width (ar, ar->fade_in()->back()->when);
 }
 
 void
-AudioRegionView::set_colors ()
+AudioRegionView::redraw_start_xfade_to (boost::shared_ptr<AudioRegion> ar, framecnt_t /*width*/, Points& points, double effective_height)
 {
-       RegionView::set_colors();
+       if (points.size() < 3) {
+               return;
+       }
 
-       if (gain_line) {
-               gain_line->set_line_color (audio_region()->envelope_active() ? ARDOUR_UI::config()->canvasvar_GainLine.get() : ARDOUR_UI::config()->canvasvar_GainLineInactive.get());
+       if (!start_xfade_in) {
+               start_xfade_in = new ArdourCanvas::PolyLine (group);
+               CANVAS_DEBUG_NAME (start_xfade_in, string_compose ("xfade start in line for %1", region()->name()));
+               start_xfade_in->set_outline_color (ARDOUR_UI::config()->get_canvasvar_CrossfadeLine());
+               start_xfade_in->set_outline_width (1.5);
        }
 
-       for (uint32_t n=0; n < waves.size(); ++n) {
-               if (_region->muted()) {
-                       waves[n]->property_wave_color() = UINT_RGBA_CHANGE_A(ARDOUR_UI::config()->canvasvar_WaveForm.get(), MUTED_ALPHA);
-               } else {
-                       waves[n]->property_wave_color() = ARDOUR_UI::config()->canvasvar_WaveForm.get();
-               }
+       if (!start_xfade_out) {
+               start_xfade_out = new ArdourCanvas::PolyLine (group);
+               CANVAS_DEBUG_NAME (start_xfade_out, string_compose ("xfade start out line for %1", region()->name()));
+               uint32_t col = UINT_RGBA_CHANGE_A (ARDOUR_UI::config()->get_canvasvar_CrossfadeLine(), 128);
+               start_xfade_out->set_outline_color (col);
+               start_xfade_out->set_outline_width (2.0);
+       }
 
-               waves[n]->property_clip_color() = ARDOUR_UI::config()->canvasvar_WaveFormClip.get();
-               waves[n]->property_zero_color() = ARDOUR_UI::config()->canvasvar_ZeroLine.get();
+       if (!start_xfade_rect) {
+               start_xfade_rect = new ArdourCanvas::Rectangle (group);
+               CANVAS_DEBUG_NAME (start_xfade_rect, string_compose ("xfade start rect for %1", region()->name()));
+               start_xfade_rect->set_fill (true);
+               start_xfade_rect->set_fill_color (ARDOUR_UI::config()->get_canvasvar_ActiveCrossfade());
+               start_xfade_rect->set_outline (false);
+               start_xfade_rect->Event.connect (sigc::bind (sigc::mem_fun (PublicEditor::instance(), &PublicEditor::canvas_start_xfade_event), start_xfade_rect, this));
+               start_xfade_rect->set_data ("regionview", this);
        }
-}
 
-void
-AudioRegionView::set_waveform_visible (bool yn)
-{
-       if (((_flags & WaveformVisible) != yn)) {
-               if (yn) {
-                       for (uint32_t n=0; n < waves.size(); ++n) {
-                               /* make sure the zoom level is correct, since we don't update
-                                  this when waveforms are hidden.
-                               */
-                               waves[n]->property_samples_per_unit() = samples_per_unit;
-                               waves[n]->show();
-                       }
-                       _flags |= WaveformVisible;
-               } else {
-                       for (uint32_t n=0; n < waves.size(); ++n) {
-                               waves[n]->hide();
-                       }
-                       _flags &= ~WaveformVisible;
+       start_xfade_rect->set (ArdourCanvas::Rect (points.front().x, 1.0, points.back().x, effective_height));
+       start_xfade_rect->show ();
+
+       start_xfade_in->set (points);
+       start_xfade_in->show ();
+
+       /* fade out line */
+
+       boost::shared_ptr<AutomationList> inverse = ar->inverse_fade_in();
+       Points ipoints;
+       Points::size_type npoints = points.size();
+
+       ipoints.assign (npoints, Duple());
+
+       if (!inverse) {
+
+               for (Points::size_type i = 0, pci = 0; i < npoints; ++i, ++pci) {
+                       ArdourCanvas::Duple &p (ipoints[pci]);
+                       p.x = i;
+                       p.y = effective_height - points[pci].y;
+               }
+
+       } else {
+
+               float vec[npoints];
+               inverse->curve().get_vector (0, inverse->back()->when, vec, npoints);
+               
+               for (Points::size_type i = 0, pci = 0; i < npoints; ++i, ++pci) {
+                       ArdourCanvas::Duple &p (ipoints[pci]);
+                       p.x = i;
+                       p.y = 1.0 + effective_height - (effective_height * vec[i]);
                }
-               store_flags ();
        }
+
+       start_xfade_out->set (ipoints);
+       start_xfade_out->show ();
+
+       show_start_xfade();
 }
 
 void
-AudioRegionView::temporarily_hide_envelope ()
+AudioRegionView::redraw_end_xfade ()
 {
-       if (gain_line) {
-               gain_line->hide ();
+       boost::shared_ptr<AudioRegion> ar (audio_region());
+
+       if (!ar->fade_out() || ar->fade_out()->empty()) {
+               return;
        }
+
+       show_end_xfade();
+       
+       reset_fade_out_shape_width (ar, ar->fade_out()->back()->when);
 }
 
 void
-AudioRegionView::unhide_envelope ()
+AudioRegionView::redraw_end_xfade_to (boost::shared_ptr<AudioRegion> ar, framecnt_t width, Points& points, double effective_height)
 {
-       if (gain_line) {
-               gain_line->show ();
+       if (points.size() < 3) {
+               return;
        }
-}
 
-void
-AudioRegionView::set_envelope_visible (bool yn)
-{
-       if (gain_line) {
-               if (yn) {
-                       gain_line->show ();
-               } else {
-                       gain_line->hide ();
-               }
+       if (!end_xfade_in) {
+               end_xfade_in = new ArdourCanvas::PolyLine (group);
+               CANVAS_DEBUG_NAME (end_xfade_in, string_compose ("xfade end in line for %1", region()->name()));
+               uint32_t col UINT_RGBA_CHANGE_A (ARDOUR_UI::config()->get_canvasvar_CrossfadeLine(), 128);
+               end_xfade_in->set_outline_color (col);
+               end_xfade_in->set_outline_width (1.5);
        }
-}
 
-void
-AudioRegionView::create_waves ()
-{
-       // cerr << "AudioRegionView::create_waves() called on " << this << endl;//DEBUG
-       RouteTimeAxisView& atv (*(dynamic_cast<RouteTimeAxisView*>(&trackview))); // ick
+       if (!end_xfade_out) {
+               end_xfade_out = new ArdourCanvas::PolyLine (group);
+               CANVAS_DEBUG_NAME (end_xfade_out, string_compose ("xfade end out line for %1", region()->name()));
+               end_xfade_out->set_outline_color (ARDOUR_UI::config()->get_canvasvar_CrossfadeLine());
+               end_xfade_out->set_outline_width (2.0);
+       }
 
-       if (!atv.track()) {
-               return;
+       if (!end_xfade_rect) {
+               end_xfade_rect = new ArdourCanvas::Rectangle (group);
+               CANVAS_DEBUG_NAME (end_xfade_rect, string_compose ("xfade end rect for %1", region()->name()));
+               end_xfade_rect->set_fill (true);
+               end_xfade_rect->set_fill_color (ARDOUR_UI::config()->get_canvasvar_ActiveCrossfade());
+               end_xfade_rect->set_outline (0);
+               end_xfade_rect->Event.connect (sigc::bind (sigc::mem_fun (PublicEditor::instance(), &PublicEditor::canvas_end_xfade_event), end_xfade_rect, this));
+               end_xfade_rect->set_data ("regionview", this);
        }
 
-       ChanCount nchans = atv.track()->n_channels();
+       end_xfade_rect->set (ArdourCanvas::Rect (points.front().x, 1.0, points.back().x, effective_height));
+       end_xfade_rect->show ();
 
-       // cerr << "creating waves for " << _region->name() << " with wfd = " << wait_for_data
-       //              << " and channels = " << nchans.n_audio() << endl;
+       end_xfade_in->set (points);
+       end_xfade_in->show ();
+       end_xfade_in->raise_to_top ();
 
-       /* in tmp_waves, set up null pointers for each channel so the vector is allocated */
-       for (uint32_t n = 0; n < nchans.n_audio(); ++n) {
-               tmp_waves.push_back (0);
-       }
+       /* fade in line */
 
-       for (vector<ScopedConnection*>::iterator i = _data_ready_connections.begin(); i != _data_ready_connections.end(); ++i) {
-               delete *i;
-       }
+       boost::shared_ptr<AutomationList> inverse = ar->inverse_fade_out ();
+       Points ipoints;
+       Points::size_type npoints = points.size();
 
-       _data_ready_connections.clear ();
+       ipoints.assign (npoints, Duple());
 
-       for (uint32_t i = 0; i < nchans.n_audio(); ++i) {
-               _data_ready_connections.push_back (0);
-       }
+       if (!inverse) {
 
-       for (uint32_t n = 0; n < nchans.n_audio(); ++n) {
+               const double rend = trackview.editor().sample_to_pixel (_region->length() - points.back().y);
 
-               if (n >= audio_region()->n_channels()) {
-                       break;
+               for (Points::size_type i = 0, pci = 0; i < npoints; ++i, ++pci) {
+                       ArdourCanvas::Duple &p (ipoints[pci]);
+                       p.x = rend + i;
+                       p.y = effective_height - points[pci].y;
                }
 
-               wave_caches.push_back (WaveView::create_cache ());
+       } else {
 
-               // cerr << "\tchannel " << n << endl;
+               boost::scoped_array<float> vec (new float[npoints]);
+               inverse->curve().get_vector (inverse->front()->when, inverse->back()->when, vec.get(), npoints);
 
-               if (wait_for_data) {
-                       if (audio_region()->audio_source(n)->peaks_ready (boost::bind (&AudioRegionView::peaks_ready_handler, this, n), &_data_ready_connections[n], gui_context())) {
-                               // cerr << "\tData is ready\n";
-                               create_one_wave (n, true);
-                       } else {
-                               // cerr << "\tdata is not ready\n";
-                               // we'll get a PeaksReady signal from the source in the future
-                               // and will call create_one_wave(n) then.
-                       }
+               const double rend = trackview.editor().sample_to_pixel (_region->length() - width);
 
-               } else {
-                       // cerr << "\tdon't delay, display today!\n";
-                       create_one_wave (n, true);
-               }
+               float* vp = vec.get();
 
+               for (Points::size_type i = 0, pci = 0; i < npoints; ++i) {
+                       ArdourCanvas::Duple& p (ipoints[pci++]);
+                       p.x = rend + i;
+                       p.y = 1.0 + effective_height - (effective_height * vp[i]);
+               }
        }
+
+       end_xfade_out->set (ipoints);
+       end_xfade_out->show ();
+       end_xfade_out->raise_to_top ();
+
+       end_xfade_rect->raise_to_top ();  //this needs to be topmost so the lines don't steal mouse focus
+
+       show_end_xfade();
 }
 
 void
-AudioRegionView::create_one_wave (uint32_t which, bool /*direct*/)
+AudioRegionView::hide_xfades ()
 {
-       //cerr << "AudioRegionView::create_one_wave() called which: " << which << " this: " << this << endl;//DEBUG
-       RouteTimeAxisView& atv (*(dynamic_cast<RouteTimeAxisView*>(&trackview))); // ick
-       uint32_t nchans = atv.track()->n_channels().n_audio();
-       uint32_t n;
-       uint32_t nwaves = std::min (nchans, audio_region()->n_channels());
-       gdouble ht;
+       hide_start_xfade ();
+       hide_end_xfade ();
+}
 
-       if (trackview.current_height() < NAME_HIGHLIGHT_THRESH) {
-               ht = ((trackview.current_height()) / (double) nchans);
-       } else {
-               ht = ((trackview.current_height() - NAME_HIGHLIGHT_SIZE) / (double) nchans);
+void
+AudioRegionView::hide_start_xfade ()
+{
+       if (start_xfade_in) {
+               start_xfade_in->hide();
+       }
+       if (start_xfade_out) {
+               start_xfade_out->hide();
+       }
+       if (start_xfade_rect) {
+               start_xfade_rect->hide ();
+       }
+
+       _start_xfade_visible = false;
+}
+
+void
+AudioRegionView::hide_end_xfade ()
+{
+       if (end_xfade_in) {
+               end_xfade_in->hide();
+       }
+       if (end_xfade_out) {
+               end_xfade_out->hide();
+       }
+       if (end_xfade_rect) {
+               end_xfade_rect->hide ();
+       }
+
+       _end_xfade_visible = false;
+}
+
+void
+AudioRegionView::show_start_xfade ()
+{
+       if (start_xfade_in) {
+               start_xfade_in->show();
+       }
+       if (start_xfade_out) {
+               start_xfade_out->show();
+       }
+       if (start_xfade_rect) {
+               start_xfade_rect->show ();
+       }
+
+       _start_xfade_visible = true;
+}
+
+void
+AudioRegionView::show_end_xfade ()
+{
+       if (end_xfade_in) {
+               end_xfade_in->show();
+       }
+       if (end_xfade_out) {
+               end_xfade_out->show();
+       }
+       if (end_xfade_rect) {
+               end_xfade_rect->show ();
+       }
+
+       _end_xfade_visible = true;
+}
+
+void
+AudioRegionView::set_samples_per_pixel (gdouble fpp)
+{
+       RegionView::set_samples_per_pixel (fpp);
+
+       if (Config->get_show_waveforms ()) {
+               for (uint32_t n = 0; n < waves.size(); ++n) {
+                       waves[n]->set_samples_per_pixel (fpp);
+               }
+       }
+
+       if (gain_line) {
+               gain_line->reset ();
+       }
+
+       reset_fade_shapes ();
+}
+
+void
+AudioRegionView::set_amplitude_above_axis (gdouble a)
+{
+       for (uint32_t n=0; n < waves.size(); ++n) {
+               waves[n]->set_amplitude_above_axis (a);
+       }
+}
+
+void
+AudioRegionView::compute_colors (Gdk::Color const & basic_color)
+{
+       RegionView::compute_colors (basic_color);
+
+       /* gain color computed in envelope_active_changed() */
+
+       fade_color = UINT_RGBA_CHANGE_A (fill_color, 120);
+}
+
+void
+AudioRegionView::set_colors ()
+{
+       RegionView::set_colors();
+
+       if (gain_line) {
+               gain_line->set_line_color (audio_region()->envelope_active() ? 
+                                          ARDOUR_UI::config()->get_canvasvar_GainLine() : 
+                                          ARDOUR_UI::config()->get_canvasvar_GainLineInactive());
+       }
+
+       set_waveform_colors ();
+
+       if (start_xfade_in) {
+               start_xfade_in->set_outline_color (ARDOUR_UI::config()->get_canvasvar_CrossfadeLine());
+       }
+       if (start_xfade_out) {
+               uint32_t col UINT_RGBA_CHANGE_A (ARDOUR_UI::config()->get_canvasvar_CrossfadeLine(), 128);
+               start_xfade_out->set_outline_color (col);
+       }
+       if (end_xfade_in) {
+               end_xfade_in->set_outline_color (ARDOUR_UI::config()->get_canvasvar_CrossfadeLine());
+       }
+       if (end_xfade_out) {
+               uint32_t col UINT_RGBA_CHANGE_A (ARDOUR_UI::config()->get_canvasvar_CrossfadeLine(), 128);
+               end_xfade_out->set_outline_color (col);
+       }
+
+       if (start_xfade_rect) {
+               start_xfade_rect->set_fill_color (ARDOUR_UI::config()->get_canvasvar_ActiveCrossfade());
+       }
+       if (end_xfade_rect) {
+               end_xfade_rect->set_fill_color (ARDOUR_UI::config()->get_canvasvar_ActiveCrossfade());
+       }
+}
+
+void
+AudioRegionView::setup_waveform_visibility ()
+{
+       if (Config->get_show_waveforms ()) {
+               for (uint32_t n = 0; n < waves.size(); ++n) {
+                       /* make sure the zoom level is correct, since we don't update
+                          this when waveforms are hidden.
+                       */
+                       // CAIROCANVAS
+                       // waves[n]->set_samples_per_pixel (_samples_per_pixel);
+                       waves[n]->show();
+               }
+       } else {
+               for (uint32_t n = 0; n < waves.size(); ++n) {
+                       waves[n]->hide();
+               }
+       }
+}
+
+void
+AudioRegionView::temporarily_hide_envelope ()
+{
+       if (gain_line) {
+               gain_line->hide ();
+       }
+}
+
+void
+AudioRegionView::unhide_envelope ()
+{
+       update_envelope_visibility ();
+}
+
+void
+AudioRegionView::update_envelope_visibility ()
+{
+       if (!gain_line) {
+               return;
+       }
+
+       if (Config->get_show_region_gain() || trackview.editor().current_mouse_mode() == Editing::MouseGain) {
+               gain_line->add_visibility (AutomationLine::Line);
+       } else {
+               gain_line->hide ();
+       }
+}
+
+void
+AudioRegionView::create_waves ()
+{
+       // cerr << "AudioRegionView::create_waves() called on " << this << endl;//DEBUG
+       RouteTimeAxisView& atv (*(dynamic_cast<RouteTimeAxisView*>(&trackview))); // ick
+
+       if (!atv.track()) {
+               return;
+       }
+
+       ChanCount nchans = atv.track()->n_channels();
+
+       // cerr << "creating waves for " << _region->name() << " with wfd = " << wait_for_data
+       //              << " and channels = " << nchans.n_audio() << endl;
+
+       /* in tmp_waves, set up null pointers for each channel so the vector is allocated */
+       for (uint32_t n = 0; n < nchans.n_audio(); ++n) {
+               tmp_waves.push_back (0);
+       }
+
+       for (vector<ScopedConnection*>::iterator i = _data_ready_connections.begin(); i != _data_ready_connections.end(); ++i) {
+               delete *i;
+       }
+
+       _data_ready_connections.clear ();
+
+       for (uint32_t i = 0; i < nchans.n_audio(); ++i) {
+               _data_ready_connections.push_back (0);
+       }
+
+       for (uint32_t n = 0; n < nchans.n_audio(); ++n) {
+
+               if (n >= audio_region()->n_channels()) {
+                       break;
+               }
+
+               // cerr << "\tchannel " << n << endl;
+
+               if (wait_for_data) {
+                       if (audio_region()->audio_source(n)->peaks_ready (boost::bind (&AudioRegionView::peaks_ready_handler, this, n), &_data_ready_connections[n], gui_context())) {
+                               // cerr << "\tData is ready\n";
+                               create_one_wave (n, true);
+                       } else {
+                               // cerr << "\tdata is not ready\n";
+                               // we'll get a PeaksReady signal from the source in the future
+                               // and will call create_one_wave(n) then.
+                       }
+
+               } else {
+                       // cerr << "\tdon't delay, display today!\n";
+                       create_one_wave (n, true);
+               }
+
+       }
+}
+
+void
+AudioRegionView::create_one_wave (uint32_t which, bool /*direct*/)
+{
+       //cerr << "AudioRegionView::create_one_wave() called which: " << which << " this: " << this << endl;//DEBUG
+       RouteTimeAxisView& atv (*(dynamic_cast<RouteTimeAxisView*>(&trackview))); // ick
+       uint32_t nchans = atv.track()->n_channels().n_audio();
+       uint32_t n;
+       uint32_t nwaves = std::min (nchans, audio_region()->n_channels());
+       gdouble ht;
+
+       if (trackview.current_height() < NAME_HIGHLIGHT_THRESH) {
+               ht = ((trackview.current_height()) / (double) nchans);
+       } else {
+               ht = ((trackview.current_height() - NAME_HIGHLIGHT_SIZE) / (double) nchans);
        }
 
        gdouble yoff = which * ht;
 
-       WaveView *wave = new WaveView(*group);
-
-       wave->property_data_src() = (gpointer) _region.get();
-       wave->property_cache() =  wave_caches[which];
-       wave->property_cache_updater() = true;
-       wave->property_channel() =  which;
-       wave->property_length_function() = (gpointer) region_length_from_c;
-       wave->property_sourcefile_length_function() = (gpointer) sourcefile_length_from_c;
-       wave->property_peak_function() =  (gpointer) region_read_peaks_from_c;
-       wave->property_x() =  0.0;
-       wave->property_y() =  yoff;
-       wave->property_height() =  (double) ht;
-       wave->property_samples_per_unit() =  samples_per_unit;
-       wave->property_amplitude_above_axis() =  _amplitude_above_axis;
+       WaveView *wave = new WaveView (group, audio_region ());
+       CANVAS_DEBUG_NAME (wave, string_compose ("wave view for chn %1 of %2", which, get_item_name()));
+       
+       wave->set_channel (which);
+       wave->set_y_position (yoff);
+       wave->set_height (ht);
+       wave->set_samples_per_pixel (samples_per_pixel);
 
        if (_recregion) {
-               wave->property_wave_color() = _region->muted() ? UINT_RGBA_CHANGE_A(ARDOUR_UI::config()->canvasvar_RecWaveForm.get(), MUTED_ALPHA) : ARDOUR_UI::config()->canvasvar_RecWaveForm.get();
-               wave->property_fill_color() = ARDOUR_UI::config()->canvasvar_RecWaveFormFill.get();
+               wave->set_outline_color (_region->muted() ? UINT_RGBA_CHANGE_A(ARDOUR_UI::config()->get_canvasvar_RecWaveForm(), MUTED_ALPHA) : ARDOUR_UI::config()->get_canvasvar_RecWaveForm());
+
+
+
+               wave->set_fill_color (ARDOUR_UI::config()->get_canvasvar_RecWaveFormFill());
        } else {
-               wave->property_wave_color() = _region->muted() ? UINT_RGBA_CHANGE_A(ARDOUR_UI::config()->canvasvar_WaveForm.get(), MUTED_ALPHA) : ARDOUR_UI::config()->canvasvar_WaveForm.get();
-               wave->property_fill_color() = ARDOUR_UI::config()->canvasvar_WaveFormFill.get();
+               wave->set_outline_color (_region->muted() ? UINT_RGBA_CHANGE_A(ARDOUR_UI::config()->get_canvasvar_WaveForm(), MUTED_ALPHA) : ARDOUR_UI::config()->get_canvasvar_WaveForm());
+
+               /* wave color is a saturated, whiter version of the frame's
+                * fill color 
+                */
+
+               ArdourCanvas::Color c = frame->fill_color ();
+               double h, s, v;
+               ArdourCanvas::color_to_hsv (c, h, s, v);
+
+               /* full saturate */
+               s = 1.0;
+               /* head towards white */
+               v = min (1.0, v * 3.0);
+               
+               c = ArdourCanvas::hsv_to_color (h, s, v, _region->muted() ? MUTED_ALPHA : 1.0);
+               
+               wave->set_fill_color (c);
        }
 
-       wave->property_clip_color() = ARDOUR_UI::config()->canvasvar_WaveFormClip.get();
-       wave->property_zero_color() = ARDOUR_UI::config()->canvasvar_ZeroLine.get();
-       wave->property_zero_line() = true;
-       wave->property_region_start() = _region->start();
-       wave->property_rectified() = (bool) (_flags & WaveformRectified);
-       wave->property_logscaled() = (bool) (_flags & WaveformLogScaled);
+       wave->set_clip_color (ARDOUR_UI::config()->get_canvasvar_WaveFormClip());
+       wave->set_zero_color (ARDOUR_UI::config()->get_canvasvar_ZeroLine());
+       wave->set_show_zero_line (true);
 
-       if (!(_flags & WaveformVisible)) {
+       switch (Config->get_waveform_shape()) {
+       case Rectified:
+               wave->set_shape (WaveView::Rectified);
+               break;
+       default:
+               wave->set_shape (WaveView::Normal);
+       }
+               
+       wave->set_logscaled (Config->get_waveform_scale() == Logarithmic);
+
+       if (!Config->get_show_waveforms ()) {
                wave->hide();
        }
 
@@ -1033,14 +1281,14 @@ AudioRegionView::add_gain_point_event (ArdourCanvas::Item *item, GdkEvent *ev)
 
        /* don't create points that can't be seen */
 
-       set_envelope_visible (true);
+       update_envelope_visibility ();
 
        x = ev->button.x;
        y = ev->button.y;
 
-       item->w2i (x, y);
+       item->canvas_to_item (x, y);
 
-       framepos_t fx = trackview.editor().pixel_to_frame (x);
+       framepos_t fx = trackview.editor().pixel_to_sample (x);
 
        if (fx > _region->length()) {
                return;
@@ -1083,101 +1331,35 @@ AudioRegionView::remove_gain_point_event (ArdourCanvas::Item *item, GdkEvent */*
 }
 
 void
-AudioRegionView::store_flags()
-{
-       XMLNode *node = new XMLNode ("GUI");
-
-       node->add_property ("waveform-visible", (_flags & WaveformVisible) ? "yes" : "no");
-       node->add_property ("waveform-rectified", (_flags & WaveformRectified) ? "yes" : "no");
-       node->add_property ("waveform-logscaled", (_flags & WaveformLogScaled) ? "yes" : "no");
-
-       _region->add_extra_xml (*node);
-}
-
-void
-AudioRegionView::set_flags (XMLNode* node)
-{
-       XMLProperty *prop;
-
-       if ((prop = node->property ("waveform-visible")) != 0) {
-               if (string_is_affirmative (prop->value())) {
-                       _flags |= WaveformVisible;
-               }
-       }
-
-       if ((prop = node->property ("waveform-rectified")) != 0) {
-               if (string_is_affirmative (prop->value())) {
-                       _flags |= WaveformRectified;
-               }
-       }
-
-       if ((prop = node->property ("waveform-logscaled")) != 0) {
-               if (string_is_affirmative (prop->value())) {
-                       _flags |= WaveformLogScaled;
-               }
-       }
-}
-
-void
-AudioRegionView::set_waveform_shape (WaveformShape shape)
+AudioRegionView::setup_waveform_shape ()
 {
-       bool yn;
-
-       /* this slightly odd approach is to leave the door open to
-          other "shapes" such as spectral displays, etc.
-       */
+       WaveView::Shape shape;
 
-       switch (shape) {
+       switch (Config->get_waveform_shape()) {
        case Rectified:
-               yn = true;
+               shape = WaveView::Rectified;
                break;
-
        default:
-               yn = false;
-               break;
+               shape = WaveView::Normal;
        }
-
-       if (yn != (bool) (_flags & WaveformRectified)) {
-               for (vector<WaveView *>::iterator wave = waves.begin(); wave != waves.end() ; ++wave) {
-                       (*wave)->property_rectified() = yn;
-               }
-
-               if (yn) {
-                       _flags |= WaveformRectified;
-               } else {
-                       _flags &= ~WaveformRectified;
-               }
-               store_flags ();
+       for (vector<WaveView *>::iterator wave = waves.begin(); wave != waves.end() ; ++wave) {
+               (*wave)->set_shape (shape);
        }
 }
 
 void
-AudioRegionView::set_waveform_scale (WaveformScale scale)
+AudioRegionView::setup_waveform_scale ()
 {
-       bool yn = (scale == Logarithmic);
-
-       if (yn != (bool) (_flags & WaveformLogScaled)) {
-               for (vector<WaveView *>::iterator wave = waves.begin(); wave != waves.end() ; ++wave) {
-                       (*wave)->property_logscaled() = yn;
-               }
-
-               if (yn) {
-                       _flags |= WaveformLogScaled;
-               } else {
-                       _flags &= ~WaveformLogScaled;
-               }
-               store_flags ();
-       }
+       WaveView::set_global_logscaled (Config->get_waveform_scale() == Logarithmic);
 }
 
-
 GhostRegion*
 AudioRegionView::add_ghost (TimeAxisView& tv)
 {
        RouteTimeAxisView* rtv = dynamic_cast<RouteTimeAxisView*>(&trackview);
        assert(rtv);
 
-       double unit_position = _region->position () / samples_per_unit;
+       double unit_position = _region->position () / samples_per_pixel;
        AudioGhostRegion* ghost = new AudioGhostRegion (tv, trackview, unit_position);
        uint32_t nchans;
 
@@ -1189,26 +1371,18 @@ AudioRegionView::add_ghost (TimeAxisView& tv)
                        break;
                }
 
-               WaveView *wave = new WaveView(*ghost->group);
-
-               wave->property_data_src() = _region.get();
-               wave->property_cache() =  wave_caches[n];
-               wave->property_cache_updater() = false;
-               wave->property_channel() = n;
-               wave->property_length_function() = (gpointer)region_length_from_c;
-               wave->property_sourcefile_length_function() = (gpointer) sourcefile_length_from_c;
-               wave->property_peak_function() =  (gpointer) region_read_peaks_from_c;
-               wave->property_x() =  0.0;
-               wave->property_samples_per_unit() =  samples_per_unit;
-               wave->property_amplitude_above_axis() =  _amplitude_above_axis;
+               WaveView *wave = new WaveView (ghost->group, audio_region());
+               CANVAS_DEBUG_NAME (wave, string_compose ("ghost wave for %1", get_item_name()));
 
-               wave->property_region_start() = _region->start();
+               wave->set_channel (n);
+               wave->set_samples_per_pixel (samples_per_pixel);
+               wave->set_amplitude_above_axis (_amplitude_above_axis);
 
                ghost->waves.push_back(wave);
        }
 
        ghost->set_height ();
-       ghost->set_duration (_region->length() / samples_per_unit);
+       ghost->set_duration (_region->length() / samples_per_pixel);
        ghost->set_colors();
        ghosts.push_back (ghost);
 
@@ -1220,14 +1394,16 @@ AudioRegionView::entered (bool internal_editing)
 {
        trackview.editor().set_current_trimmable (_region);
        trackview.editor().set_current_movable (_region);
-
-       if (gain_line && Config->get_show_region_gain ()) {
-               gain_line->show_all_control_points ();
+       
+       if (gain_line && trackview.editor().current_mouse_mode() == Editing::MouseGain) {
+               gain_line->add_visibility (AutomationLine::ControlPoints);
        }
 
        if (fade_in_handle && !internal_editing) {
-               fade_in_handle->property_fill_color_rgba() = UINT_RGBA_CHANGE_A (fade_color, 255);
-               fade_out_handle->property_fill_color_rgba() = UINT_RGBA_CHANGE_A (fade_color, 255);
+               fade_in_handle->set_outline_color (RGBA_TO_UINT (0, 0, 0, 255));
+               fade_in_handle->set_fill_color (UINT_RGBA_CHANGE_A (fade_color, 255));
+               fade_out_handle->set_outline_color (RGBA_TO_UINT (0, 0, 0, 255));
+               fade_out_handle->set_fill_color (UINT_RGBA_CHANGE_A (fade_color, 255));
        }
 }
 
@@ -1237,13 +1413,15 @@ AudioRegionView::exited ()
        trackview.editor().set_current_trimmable (boost::shared_ptr<Trimmable>());
        trackview.editor().set_current_movable (boost::shared_ptr<Movable>());
 
-       if (gain_line) {
-               gain_line->hide_all_but_selected_control_points ();
+       if (gain_line && trackview.editor().current_mouse_mode() == Editing::MouseGain) {
+               gain_line->remove_visibility (AutomationLine::ControlPoints);
        }
 
        if (fade_in_handle) {
-               fade_in_handle->property_fill_color_rgba() = UINT_RGBA_CHANGE_A (fade_color, 0);
-               fade_out_handle->property_fill_color_rgba() = UINT_RGBA_CHANGE_A (fade_color, 0);
+               fade_in_handle->set_outline_color (RGBA_TO_UINT (0, 0, 0, 0));
+               fade_in_handle->set_fill_color (UINT_RGBA_CHANGE_A (fade_color, 0));
+               fade_out_handle->set_outline_color (RGBA_TO_UINT (0, 0, 0, 0));
+               fade_out_handle->set_fill_color (UINT_RGBA_CHANGE_A (fade_color, 0));
        }
 }
 
@@ -1251,32 +1429,10 @@ void
 AudioRegionView::envelope_active_changed ()
 {
        if (gain_line) {
-               gain_line->set_line_color (audio_region()->envelope_active() ? ARDOUR_UI::config()->canvasvar_GainLine.get() : ARDOUR_UI::config()->canvasvar_GainLineInactive.get());
-       }
-}
-
-void
-AudioRegionView::set_waveview_data_src()
-{
-       AudioGhostRegion* agr;
-       double unit_length= _region->length() / samples_per_unit;
-
-       for (uint32_t n = 0; n < waves.size(); ++n) {
-               // TODO: something else to let it know the channel
-               waves[n]->property_data_src() = _region.get();
+               gain_line->set_line_color (audio_region()->envelope_active() ? 
+                                          ARDOUR_UI::config()->get_canvasvar_GainLine() : 
+                                          ARDOUR_UI::config()->get_canvasvar_GainLineInactive());
        }
-
-       for (vector<GhostRegion*>::iterator i = ghosts.begin(); i != ghosts.end(); ++i) {
-
-               (*i)->set_duration (unit_length);
-
-               if((agr = dynamic_cast<AudioGhostRegion*>(*i)) != 0) {
-                       for (vector<WaveView*>::iterator w = agr->waves.begin(); w != agr->waves.end(); ++w) {
-                               (*w)->property_data_src() = _region.get();
-                       }
-               }
-       }
-
 }
 
 void
@@ -1295,56 +1451,62 @@ AudioRegionView::color_handler ()
 }
 
 void
-AudioRegionView::set_frame_color ()
+AudioRegionView::set_waveform_colors ()
 {
-       if (!frame) {
-               return;
-       }
-
-       if (_region->opaque()) {
-               fill_opacity = 130;
-       } else {
-               fill_opacity = 0;
-       }
-
-       TimeAxisViewItem::set_frame_color ();
-
-        uint32_t wc;
-        uint32_t fc;
-
+       ArdourCanvas::Color fill, outline, clip, zero;
+       
        if (_selected) {
                 if (_region->muted()) {
-                        wc = UINT_RGBA_CHANGE_A(ARDOUR_UI::config()->canvasvar_SelectedWaveForm.get(), MUTED_ALPHA);
+                        outline = UINT_RGBA_CHANGE_A(ARDOUR_UI::config()->get_canvasvar_SelectedWaveForm(), MUTED_ALPHA);
                 } else {
-                        wc = ARDOUR_UI::config()->canvasvar_SelectedWaveForm.get();
+                        outline = ARDOUR_UI::config()->get_canvasvar_SelectedWaveForm();
                 }
-                fc = ARDOUR_UI::config()->canvasvar_SelectedWaveFormFill.get();
+                fill = ARDOUR_UI::config()->get_canvasvar_SelectedWaveFormFill();
        } else {
                if (_recregion) {
                         if (_region->muted()) {
-                                wc = UINT_RGBA_CHANGE_A(ARDOUR_UI::config()->canvasvar_RecWaveForm.get(), MUTED_ALPHA);
+                                outline = UINT_RGBA_CHANGE_A(ARDOUR_UI::config()->get_canvasvar_RecWaveForm(), MUTED_ALPHA);
                         } else {
-                                wc = ARDOUR_UI::config()->canvasvar_RecWaveForm.get();
+                                outline = ARDOUR_UI::config()->get_canvasvar_RecWaveForm();
                         }
-                        fc = ARDOUR_UI::config()->canvasvar_RecWaveFormFill.get();
+                        fill = ARDOUR_UI::config()->get_canvasvar_RecWaveFormFill();
                } else {
                         if (_region->muted()) {
-                                wc = UINT_RGBA_CHANGE_A(ARDOUR_UI::config()->canvasvar_WaveForm.get(), MUTED_ALPHA);
+                                outline = UINT_RGBA_CHANGE_A(ARDOUR_UI::config()->get_canvasvar_WaveForm(), MUTED_ALPHA);
                         } else {
-                                wc = ARDOUR_UI::config()->canvasvar_WaveForm.get();
+                                outline = ARDOUR_UI::config()->get_canvasvar_WaveForm();
                         }
-                        fc = ARDOUR_UI::config()->canvasvar_WaveFormFill.get();
+                        fill = ARDOUR_UI::config()->get_canvasvar_WaveFormFill();
                }
        }
 
+       clip = ARDOUR_UI::config()->get_canvasvar_WaveFormClip();
+       zero = ARDOUR_UI::config()->get_canvasvar_ZeroLine();
+
         for (vector<ArdourCanvas::WaveView*>::iterator w = waves.begin(); w != waves.end(); ++w) {
-                if (_region->muted()) {
-                        (*w)->property_wave_color() = wc;
-                } else {
-                        (*w)->property_wave_color() = wc;
-                        (*w)->property_fill_color() = fc;
-                }
-        }
+               (*w)->set_outline_color (outline);
+               (*w)->set_fill_color (fill);
+               (*w)->set_clip_color (clip);
+               (*w)->set_zero_color (zero);
+       }
+}
+
+void
+AudioRegionView::set_frame_color ()
+{
+       if (!frame) {
+               return;
+       }
+
+       if (_region->opaque()) {
+               fill_opacity = 130;
+       } else {
+               fill_opacity = 0;
+       }
+
+       TimeAxisViewItem::set_frame_color ();
+
+       set_waveform_colors ();
 }
 
 void
@@ -1398,27 +1560,9 @@ AudioRegionView::show_region_editor ()
        }
 
        editor->present ();
-       editor->set_position (Gtk::WIN_POS_MOUSE);
        editor->show_all();
 }
 
-
-void
-AudioRegionView::show_fade_line (framepos_t pos)
-{
-       fade_position_line->property_x1() = trackview.editor().frame_to_pixel (pos);
-       fade_position_line->property_x2() = trackview.editor().frame_to_pixel (pos);
-       fade_position_line->show ();
-       fade_position_line->raise_to_top ();
-}
-
-void
-AudioRegionView::hide_fade_line ()
-{
-       fade_position_line->hide ();
-}
-
-
 void
 AudioRegionView::transients_changed ()
 {
@@ -1426,16 +1570,12 @@ AudioRegionView::transients_changed ()
 
        while (feature_lines.size() < analysis_features.size()) {
 
-               ArdourCanvas::Line* canvas_item = new ArdourCanvas::Line(*group);
-
-               ArdourCanvas::Points points;
+               ArdourCanvas::Line* canvas_item = new ArdourCanvas::Line(group);
+               CANVAS_DEBUG_NAME (canvas_item, string_compose ("transient group for %1", region()->name()));
 
-               points.push_back(Gnome::Art::Point(-1.0, 2.0)); // first x-coord needs to be a non-normal value
-               points.push_back(Gnome::Art::Point(1.0, _height - TimeAxisViewItem::NAME_HIGHLIGHT_SIZE - 1));
+               canvas_item->set (ArdourCanvas::Duple (-1.0, 2.0),
+                                 ArdourCanvas::Duple (1.0, _height - TimeAxisViewItem::NAME_HIGHLIGHT_SIZE - 1));
 
-               canvas_item->property_points() = points;
-               canvas_item->property_width_pixels() = 1;
-               canvas_item->property_fill_color_rgba() = ARDOUR_UI::config()->canvasvar_ZeroLine.get();
                canvas_item->property_first_arrowhead() = TRUE;
                canvas_item->property_last_arrowhead() = TRUE;
                canvas_item->property_arrow_shape_a() = 11.0;
@@ -1446,7 +1586,7 @@ AudioRegionView::transients_changed ()
                canvas_item->show ();
 
                canvas_item->set_data ("regionview", this);
-               canvas_item->signal_event().connect (sigc::bind (sigc::mem_fun (PublicEditor::instance(), &PublicEditor::canvas_feature_line_event), canvas_item, this));
+               canvas_item->Event.connect (sigc::bind (sigc::mem_fun (PublicEditor::instance(), &PublicEditor::canvas_feature_line_event), canvas_item, this));
 
                feature_lines.push_back (make_pair(0, canvas_item));
        }
@@ -1462,17 +1602,15 @@ AudioRegionView::transients_changed ()
 
        for (i = analysis_features.begin(), l = feature_lines.begin(); i != analysis_features.end() && l != feature_lines.end(); ++i, ++l) {
 
-               ArdourCanvas::Points points;
-
                float *pos = new float;
-               *pos = trackview.editor().frame_to_pixel (*i);
+               *pos = trackview.editor().sample_to_pixel (*i);
 
-               points.push_back(Gnome::Art::Point(*pos, 2.0)); // first x-coord needs to be a non-normal value
-               points.push_back(Gnome::Art::Point(*pos, _height - TimeAxisViewItem::NAME_HIGHLIGHT_SIZE - 1));
+               (*l).second->set (
+                       ArdourCanvas::Duple (*pos, 2.0),
+                       ArdourCanvas::Duple (*pos, _height - TimeAxisViewItem::NAME_HIGHLIGHT_SIZE - 1)
+                       );
 
-               (*l).second->property_points() = points;
                (*l).second->set_data ("position", pos);
-
                (*l).first = *i;
        }
 }
@@ -1492,7 +1630,7 @@ AudioRegionView::update_transient(float /*old_pos*/, float new_pos)
                if (rint(new_pos) == rint(*pos)) {
 
                    framepos_t old_frame = (*l).first;
-                   framepos_t new_frame = trackview.editor().pixel_to_frame (new_pos);
+                   framepos_t new_frame = trackview.editor().pixel_to_sample (new_pos);
 
                    _region->update_transient (old_frame, new_frame);
 
@@ -1524,248 +1662,41 @@ AudioRegionView::thaw_after_trim ()
 {
        RegionView::thaw_after_trim ();
        unhide_envelope ();
+       drag_end ();
 }
 
-void
-AudioRegionView::redraw_start_xfade ()
-{
-       boost::shared_ptr<AudioRegion> ar (audio_region());
-
-       if (!ar->fade_in() || ar->fade_in()->empty()) {
-               return;
-       }
-
-       redraw_start_xfade_to (ar, ar->fade_in()->back()->when);
-}
 
 void
-AudioRegionView::redraw_start_xfade_to (boost::shared_ptr<AudioRegion> ar, framecnt_t len)
+AudioRegionView::show_xfades ()
 {
-       int32_t const npoints = trackview.editor().frame_to_pixel (len);
-
-       if (npoints < 3) {
-               return;
-       }
-
-       if (!start_xfade_in) {
-               start_xfade_in = new ArdourCanvas::Line (*group);
-               start_xfade_in->property_width_pixels() = 1;
-               start_xfade_in->property_fill_color_rgba() = ARDOUR_UI::config()->canvasvar_GainLine.get();
-       }
-
-       if (!start_xfade_out) {
-               start_xfade_out = new ArdourCanvas::Line (*group);
-               start_xfade_out->property_width_pixels() = 1;
-               start_xfade_out->property_fill_color_rgba() = ARDOUR_UI::config()->canvasvar_GainLine.get();
-       }
-
-       if (!start_xfade_rect) {
-               start_xfade_rect = new ArdourCanvas::SimpleRect (*group);
-               start_xfade_rect->property_draw() = true;
-               start_xfade_rect->property_fill() = true;;
-               start_xfade_rect->property_fill_color_rgba() =  ARDOUR_UI::config()->canvasvar_ActiveCrossfade.get();
-               start_xfade_rect->property_outline_pixels() = 0;
-               start_xfade_rect->signal_event().connect (sigc::bind (sigc::mem_fun (PublicEditor::instance(), &PublicEditor::canvas_start_xfade_event), start_xfade_rect, this));
-               start_xfade_rect->set_data ("regionview", this);
-       }
-
-       Points* points = get_canvas_points ("xfade edit redraw", npoints);
-       boost::scoped_ptr<float> vec (new float[npoints]);
-
-       ar->fade_in()->curve().get_vector (0, ar->fade_in()->back()->when, vec.get(), npoints);
-
-       for (int i = 0, pci = 0; i < npoints; ++i) {
-               Gnome::Art::Point &p ((*points)[pci++]);
-               p.set_x (i);
-               p.set_y (_height - (_height * vec.get()[i]));
-       }
-
-       start_xfade_rect->property_x1() = ((*points)[0]).get_x();
-       start_xfade_rect->property_y1() = 0;
-       start_xfade_rect->property_x2() = ((*points)[npoints-1]).get_x();
-       start_xfade_rect->property_y2() = _height;
-       start_xfade_rect->show ();
-       start_xfade_rect->raise_to_top ();
-
-       start_xfade_in->property_points() = *points;
-       start_xfade_in->show ();
-       start_xfade_in->raise_to_top ();
-
-       /* fade out line */
-
-       boost::shared_ptr<AutomationList> inverse = ar->inverse_fade_in();
-
-       if (!inverse) {
-
-               for (int i = 0, pci = 0; i < npoints; ++i) {
-                       Gnome::Art::Point &p ((*points)[pci++]);
-                       p.set_x (i);
-                       p.set_y (_height - (_height * (1.0 - vec.get()[i])));
-               }
-
-       } else {
-
-               inverse->curve().get_vector (0, inverse->back()->when, vec.get(), npoints);
-
-               for (int i = 0, pci = 0; i < npoints; ++i) {
-                       Gnome::Art::Point &p ((*points)[pci++]);
-                       p.set_x (i);
-                       p.set_y (_height - (_height * vec.get()[i]));
-               }
-       }
-
-       start_xfade_out->property_points() = *points;
-       start_xfade_out->show ();
-       start_xfade_out->raise_to_top ();
-
-       delete points;
+       show_start_xfade ();
+       show_end_xfade ();
 }
 
 void
-AudioRegionView::redraw_end_xfade ()
+AudioRegionView::drag_start ()
 {
-       boost::shared_ptr<AudioRegion> ar (audio_region());
-
-       if (!ar->fade_out() || ar->fade_out()->empty()) {
-               return;
-       }
+       TimeAxisViewItem::drag_start ();
 
-       redraw_end_xfade_to (ar, ar->fade_out()->back()->when);
+       //we used to hide xfades here.  I don't see the point with the new model, but we can re-implement if needed
 }
 
 void
-AudioRegionView::redraw_end_xfade_to (boost::shared_ptr<AudioRegion> ar, framecnt_t len)
+AudioRegionView::drag_end ()
 {
-       int32_t const npoints = trackview.editor().frame_to_pixel (len);
-
-       if (npoints < 3) {
-               return;
-       }
-
-       if (!end_xfade_in) {
-               end_xfade_in = new ArdourCanvas::Line (*group);
-               end_xfade_in->property_width_pixels() = 1;
-               end_xfade_in->property_fill_color_rgba() = ARDOUR_UI::config()->canvasvar_GainLine.get();
-       }
-
-       if (!end_xfade_out) {
-               end_xfade_out = new ArdourCanvas::Line (*group);
-               end_xfade_out->property_width_pixels() = 1;
-               end_xfade_out->property_fill_color_rgba() = ARDOUR_UI::config()->canvasvar_GainLine.get();
-       }
-
-       if (!end_xfade_rect) {
-               end_xfade_rect = new ArdourCanvas::SimpleRect (*group);
-               end_xfade_rect->property_draw() = true;
-               end_xfade_rect->property_fill() = true;;
-               end_xfade_rect->property_fill_color_rgba() =  ARDOUR_UI::config()->canvasvar_ActiveCrossfade.get();
-               end_xfade_rect->property_outline_pixels() = 0;
-               end_xfade_rect->signal_event().connect (sigc::bind (sigc::mem_fun (PublicEditor::instance(), &PublicEditor::canvas_end_xfade_event), end_xfade_rect, this));
-               end_xfade_rect->set_data ("regionview", this);
-       }
-
-       Points* points = get_canvas_points ("xfade edit redraw", npoints);
-       boost::scoped_ptr<float> vec (new float[npoints]);
-
-       ar->fade_out()->curve().get_vector (0, ar->fade_out()->back()->when, vec.get(), npoints);
-
-       double rend = trackview.editor().frame_to_pixel (_region->length() - len);
-
-       for (int i = 0, pci = 0; i < npoints; ++i) {
-               Gnome::Art::Point &p ((*points)[pci++]);
-               p.set_x (rend + i);
-               p.set_y (_height - (_height * vec.get()[i]));
-       }
-
-       end_xfade_rect->property_x1() = ((*points)[0]).get_x();
-       end_xfade_rect->property_y1() = 0;
-       end_xfade_rect->property_x2() = ((*points)[npoints-1]).get_x();
-       end_xfade_rect->property_y2() = _height;
-       end_xfade_rect->show ();
-       end_xfade_rect->raise_to_top ();
-
-       end_xfade_in->property_points() = *points;
-       end_xfade_in->show ();
-       end_xfade_in->raise_to_top ();
-
-       /* fade in line */
-
-       boost::shared_ptr<AutomationList> inverse = ar->inverse_fade_out ();
-
-       if (!inverse) {
-
-               for (int i = 0, pci = 0; i < npoints; ++i) {
-                       Gnome::Art::Point &p ((*points)[pci++]);
-                       p.set_x (rend + i);
-                       p.set_y (_height - (_height * (1.0 - vec.get()[i])));
-               }
-
-       } else {
-
-               inverse->curve().get_vector (inverse->front()->when, inverse->back()->when, vec.get(), npoints);
-               
-               for (int i = 0, pci = 0; i < npoints; ++i) {
-                       Gnome::Art::Point &p ((*points)[pci++]);
-                       p.set_x (rend + i);
-                       p.set_y (_height - (_height * vec.get()[i]));
-               }
-       }
-
-       end_xfade_out->property_points() = *points;
-       end_xfade_out->show ();
-       end_xfade_out->raise_to_top ();
-
+       TimeAxisViewItem::drag_end ();
 
-       delete points;
+       //see comment for drag_start
 }
 
 void
-AudioRegionView::drag_start ()
+AudioRegionView::parameter_changed (string const & p)
 {
-       TimeAxisViewItem::drag_start ();
-       
-       if (start_xfade_in) {
-               start_xfade_in->hide();
-       }
-       if (start_xfade_out) {
-               start_xfade_out->hide();
-       }
-       if (start_xfade_rect) {
-               start_xfade_rect->hide ();
-       }
-       if (end_xfade_in) {
-               end_xfade_in->hide();
-       }
-       if (end_xfade_out) {
-               end_xfade_out->hide();
-       }
-       if (end_xfade_rect) {
-               end_xfade_rect->hide ();
+       if (p == "show-waveforms") {
+               setup_waveform_visibility ();
+       } else if (p == "waveform-scale") {
+               setup_waveform_scale ();
+       } else if (p == "waveform-shape") {
+               setup_waveform_shape ();
        }
 }
-
-void
-AudioRegionView::drag_end ()
-{
-       TimeAxisViewItem::drag_end ();
-
-       if (start_xfade_in) {
-               start_xfade_in->show();
-       }
-       if (start_xfade_out) {
-               start_xfade_out->show();
-       }
-       if (start_xfade_rect) {
-               start_xfade_rect->show ();
-       }
-       if (end_xfade_in) {
-               end_xfade_in->show();
-       }
-       if (end_xfade_out) {
-               end_xfade_out->show();
-       }
-       if (end_xfade_rect) {
-               end_xfade_rect->show ();
-       }
-}
-