Use MTC instead of MIDI Timecode as the string for the external sync button. Fixes...
[ardour.git] / gtk2_ardour / audio_region_view.cc
index 06621a20e2caae0e097890fa0ac01d459893e69f..51a016d2c3b8e7bab058573264a7389ec9640e58 100644 (file)
@@ -73,6 +73,7 @@ AudioRegionView::AudioRegionView (ArdourCanvas::Group *parent, RouteTimeAxisView
        , fade_out_shape(0)
        , fade_in_handle(0)
        , fade_out_handle(0)
+       , fade_position_line(0)
        , gain_line(0)
        , _amplitude_above_axis(1.0)
        , _flags(0)
@@ -90,6 +91,7 @@ AudioRegionView::AudioRegionView (ArdourCanvas::Group *parent, RouteTimeAxisView
        , fade_out_shape(0)
        , fade_in_handle(0)
        , fade_out_handle(0)
+       , fade_position_line(0)
        , gain_line(0)
        , _amplitude_above_axis(1.0)
        , _flags(0)
@@ -106,6 +108,7 @@ AudioRegionView::AudioRegionView (const AudioRegionView& other)
        , fade_out_shape(0)
        , fade_in_handle(0)
        , fade_out_handle(0)
+       , fade_position_line(0)
        , gain_line(0)
        , _amplitude_above_axis(1.0)
        , _flags(0)
@@ -127,6 +130,7 @@ AudioRegionView::AudioRegionView (const AudioRegionView& other, boost::shared_pt
        , fade_out_shape(0)
        , fade_in_handle(0)
        , fade_out_handle(0)
+       , fade_position_line(0)
        , gain_line(0)
        , _amplitude_above_axis(1.0)
        , _flags(0)
@@ -163,7 +167,8 @@ AudioRegionView::init (Gdk::Color const & basic_color, bool wfd)
        }
 
        /* make envelope visible if it has anything interesting in it */
-       if (audio_region()->envelope()->size() > 2) {
+       boost::shared_ptr<AutomationList> env = audio_region()->envelope ();
+       if (env->size() > 2 || (env->size() == 2 && env->front()->value != env->back()->value)) {
                _flags |= EnvelopeVisible;
        }
 
@@ -179,21 +184,25 @@ AudioRegionView::init (Gdk::Color const & basic_color, bool wfd)
        fade_out_shape->property_fill_color_rgba() = fade_color;
        fade_out_shape->set_data ("regionview", this);
 
-       {
-               uint32_t r,g,b,a;
-               UINT_TO_RGBA(fill_color,&r,&g,&b,&a);
-
+       if (!_recregion) {
                fade_in_handle = new ArdourCanvas::SimpleRect (*group);
-               fade_in_handle->property_fill_color_rgba() = RGBA_TO_UINT(r,g,b,0);
+               fade_in_handle->property_fill_color_rgba() = UINT_RGBA_CHANGE_A (fill_color, 0);
                fade_in_handle->property_outline_pixels() = 0;
 
                fade_in_handle->set_data ("regionview", this);
 
                fade_out_handle = new ArdourCanvas::SimpleRect (*group);
-               fade_out_handle->property_fill_color_rgba() = RGBA_TO_UINT(r,g,b,0);
+               fade_out_handle->property_fill_color_rgba() = UINT_RGBA_CHANGE_A (fill_color, 0);
                fade_out_handle->property_outline_pixels() = 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 ();
@@ -231,9 +240,15 @@ 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_handle->signal_event().connect (sigc::bind (sigc::mem_fun (PublicEditor::instance(), &PublicEditor::canvas_fade_in_handle_event), fade_in_handle, 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_out_shape->signal_event().connect (sigc::bind (sigc::mem_fun (PublicEditor::instance(), &PublicEditor::canvas_fade_out_event), fade_out_shape, this));
-       fade_out_handle->signal_event().connect (sigc::bind (sigc::mem_fun (PublicEditor::instance(), &PublicEditor::canvas_fade_out_handle_event), fade_out_handle, 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));
+       }
 
        set_colors ();
 
@@ -254,7 +269,7 @@ AudioRegionView::~AudioRegionView ()
                delete *i;
        }
 
-       for (list<std::pair<nframes64_t, ArdourCanvas::SimpleLine*> >::iterator i = feature_lines.begin(); i != feature_lines.end(); ++i) {
+       for (list<std::pair<framepos_t, ArdourCanvas::SimpleLine*> >::iterator i = feature_lines.begin(); i != feature_lines.end(); ++i) {
                delete ((*i).second);
        }
        
@@ -274,7 +289,7 @@ void
 AudioRegionView::region_changed (const PropertyChange& what_changed)
 {
        ENSURE_GUI_THREAD (*this, &AudioRegionView::region_changed, what_changed);
-        // cerr << "AudioRegionView::region_changed() called" << endl;
+       // cerr << "AudioRegionView::region_changed() called" << endl;
 
        RegionView::region_changed (what_changed);
 
@@ -282,7 +297,7 @@ AudioRegionView::region_changed (const PropertyChange& what_changed)
                region_scale_amplitude_changed ();
        }
        if (what_changed.contains (ARDOUR::Properties::fade_in)) {
-                fade_in_changed ();
+               fade_in_changed ();
        }
        if (what_changed.contains (ARDOUR::Properties::fade_out)) {
                fade_out_changed ();
@@ -315,36 +330,24 @@ AudioRegionView::fade_out_changed ()
 void
 AudioRegionView::fade_in_active_changed ()
 {
-//     uint32_t r,g,b,a;
-//     uint32_t col;
-//     UINT_TO_RGBA(fade_color,&r,&g,&b,&a);
-
        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;
-               fade_in_shape->property_outline_color_rgba() = RGBA_TO_UINT(180,180,180,190);                   // FIXME make a themeable colour
        } 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;
-               fade_in_shape->property_outline_color_rgba() = RGBA_TO_UINT(45,45,45,150);                      // FIXME make a themeable colour
        }
 }
 
 void
 AudioRegionView::fade_out_active_changed ()
 {
-//     uint32_t r,g,b,a;
-//     uint32_t col;
-//     UINT_TO_RGBA(fade_color,&r,&g,&b,&a);
-
        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;
-               fade_out_shape->property_outline_color_rgba() = RGBA_TO_UINT(180,180,180,200);                  // FIXME make a themeable colour
        } 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;
-               fade_out_shape->property_outline_color_rgba() = RGBA_TO_UINT(45,45,45,200);                     // FIXME make a themeable colour
        }
 }
 
@@ -363,7 +366,7 @@ AudioRegionView::region_scale_amplitude_changed ()
 void
 AudioRegionView::region_renamed ()
 {
-       Glib::ustring str = RegionView::make_name ();
+       std::string str = RegionView::make_name ();
 
        if (audio_region()->speed_mismatch (trackview.session()->frame_rate())) {
                str = string ("*") + str;
@@ -405,7 +408,7 @@ AudioRegionView::region_resized (const PropertyChange& what_changed)
                
                /* hide transient lines that extend beyond the region end */
                
-               list<std::pair<nframes64_t, ArdourCanvas::SimpleLine*> >::iterator l;
+               list<std::pair<framepos_t, ArdourCanvas::SimpleLine*> >::iterator l;
                
                for (l = feature_lines.begin(); l != feature_lines.end(); ++l) {
                        if ((*l).first > _region->length()- 1){
@@ -429,25 +432,19 @@ AudioRegionView::reset_width_dependent_items (double pixel_width)
        }
 
        if (fade_in_handle) {
-               if (pixel_width <= 6.0) {
+               if (pixel_width <= 6.0 || _height < 5.0 || !trackview.session()->config.get_show_region_fades()) {
                        fade_in_handle->hide();
                        fade_out_handle->hide();
-               } else {
-                       if (_height < 5.0) {
-                               fade_in_handle->hide();
-                               fade_out_handle->hide();
-                       } else {
-                               if (trackview.session()->config.get_show_region_fades()) {
-                                       fade_in_handle->show();
-                                       fade_out_handle->show();
-                               }
-                       }
+               }
+               else {
+                       fade_in_handle->show();
+                       fade_out_handle->show();                  
                }
        }
 
        AnalysisFeatureList analysis_features = _region->transients();
        AnalysisFeatureList::const_iterator i;
-       list<std::pair<nframes64_t, ArdourCanvas::SimpleLine*> >::iterator l;
+       list<std::pair<framepos_t, ArdourCanvas::SimpleLine*> >::iterator l;
 
        for (i = analysis_features.begin(), l = feature_lines.begin(); i != analysis_features.end() && l != feature_lines.end(); ++i, ++l) {
                (*l).second->property_x1() = trackview.editor().frame_to_pixel (*i);
@@ -497,13 +494,10 @@ AudioRegionView::set_height (gdouble height)
 
        uint32_t wcnt = waves.size();
 
-       // FIXME: ick
-       height -= 2;
-
        for (uint32_t n = 0; n < wcnt; ++n) {
                gdouble ht;
 
-               if ((height) < NAME_HIGHLIGHT_THRESH) {
+               if (height < NAME_HIGHLIGHT_THRESH) {
                        ht = ((height - 2 * wcnt) / (double) wcnt);
                } else {
                        ht = (((height - 2 * wcnt) - NAME_HIGHLIGHT_SIZE) / (double) wcnt);
@@ -532,12 +526,21 @@ AudioRegionView::set_height (gdouble height)
        reset_fade_shapes ();
        
        /* Update hights for any active feature lines */
-       list<std::pair<nframes64_t, ArdourCanvas::SimpleLine*> >::iterator l;
+       list<std::pair<framepos_t, ArdourCanvas::SimpleLine*> >::iterator l;
 
        for (l = feature_lines.begin(); l != feature_lines.end(); ++l) {
                (*l).second->property_y2() = _height - TimeAxisViewItem::NAME_HIGHLIGHT_SIZE - 1;
        }       
 
+       if (fade_position_line) {
+               if (height < NAME_HIGHLIGHT_THRESH) {
+                       fade_position_line->property_y2() = _height - 1;
+               }
+               else {
+                       fade_position_line->property_y2() = _height - TimeAxisViewItem::NAME_HIGHLIGHT_SIZE - 1;
+               }
+       }
+
        if (name_pixbuf) {
                name_pixbuf->raise_to_top();
        }
@@ -570,11 +573,11 @@ AudioRegionView::reset_fade_shapes ()
 void
 AudioRegionView::reset_fade_in_shape ()
 {
-       reset_fade_in_shape_width ((nframes_t) audio_region()->fade_in()->back()->when);
+       reset_fade_in_shape_width ((framecnt_t) audio_region()->fade_in()->back()->when);
 }
 
 void
-AudioRegionView::reset_fade_in_shape_width (nframes_t width)
+AudioRegionView::reset_fade_in_shape_width (framecnt_t width)
 {
        if (fade_in_handle == 0) {
                return;
@@ -582,10 +585,12 @@ AudioRegionView::reset_fade_in_shape_width (nframes_t width)
 
        /* smallest size for a fade is 64 frames */
 
-       width = std::max ((nframes_t) 64, width);
+       width = std::max ((framecnt_t) 64, width);
 
        Points* points;
-       double pwidth = width / samples_per_unit;
+
+       /* 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;
 
@@ -595,17 +600,11 @@ AudioRegionView::reset_fade_in_shape_width (nframes_t width)
                return;
        }
 
-       double handle_center;
-       handle_center = pwidth;
+       double const handle_center = pwidth;
 
-       if (handle_center > 7.0) {
-               handle_center -= 3.0;
-       } else {
-               handle_center = 3.0;
-       }
-
-       fade_in_handle->property_x1() =  handle_center - 3.0;
-       fade_in_handle->property_x2() =  handle_center + 3.0;
+       /* 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;
 
        if (pwidth < 5) {
                fade_in_shape->hide();
@@ -622,7 +621,7 @@ AudioRegionView::reset_fade_in_shape_width (nframes_t width)
        points = get_canvas_points ("fade in shape", npoints + 3);
 
        if (_height >= NAME_HIGHLIGHT_THRESH) {
-               h = _height - NAME_HIGHLIGHT_SIZE;
+               h = _height - NAME_HIGHLIGHT_SIZE - 2;
        } else {
                h = _height;
        }
@@ -653,17 +652,19 @@ AudioRegionView::reset_fade_in_shape_width (nframes_t width)
        delete points;
 
        /* ensure trim handle stays on top */
-       frame_handle_start->raise_to_top();
+       if (frame_handle_start) {
+               frame_handle_start->raise_to_top();
+       }
 }
 
 void
 AudioRegionView::reset_fade_out_shape ()
 {
-       reset_fade_out_shape_width ((nframes_t) audio_region()->fade_out()->back()->when);
+       reset_fade_out_shape_width ((framecnt_t) audio_region()->fade_out()->back()->when);
 }
 
 void
-AudioRegionView::reset_fade_out_shape_width (nframes_t width)
+AudioRegionView::reset_fade_out_shape_width (framecnt_t width)
 {
        if (fade_out_handle == 0) {
                return;
@@ -671,10 +672,12 @@ AudioRegionView::reset_fade_out_shape_width (nframes_t width)
 
        /* smallest size for a fade is 64 frames */
 
-       width = std::max ((nframes_t) 64, width);
+       width = std::max ((framecnt_t) 64, width);
 
        Points* points;
-       double pwidth = width / samples_per_unit;
+
+       /* 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;
 
@@ -684,17 +687,13 @@ AudioRegionView::reset_fade_out_shape_width (nframes_t width)
                return;
        }
 
-       double handle_center;
-       handle_center = (_region->length() - width) / samples_per_unit;
+       double const handle_center = (_region->length() - width) / samples_per_unit;
 
-       if (handle_center > 7.0) {
-               handle_center -= 3.0;
-       } else {
-               handle_center = 3.0;
-       }
-
-       fade_out_handle->property_x1() =  handle_center - 3.0;
-       fade_out_handle->property_x2() =  handle_center + 3.0;
+       /* 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;
 
        /* don't show shape if its too small */
 
@@ -711,7 +710,7 @@ AudioRegionView::reset_fade_out_shape_width (nframes_t width)
        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;
+               h = _height - NAME_HIGHLIGHT_SIZE - 2;
        } else {
                h = _height;
        }
@@ -724,7 +723,7 @@ AudioRegionView::reset_fade_out_shape_width (nframes_t width)
        double xdelta = pwidth/npoints;
 
        for (pi = 0, pc = 0; pc < npoints; ++pc) {
-               (*points)[pi].set_x(_pixel_width - 1 - pwidth + (pc*xdelta));
+               (*points)[pi].set_x(_pixel_width - pwidth + (pc * xdelta));
                (*points)[pi++].set_y(2 + (h - (curve[pc] * h)));
        }
 
@@ -744,7 +743,9 @@ AudioRegionView::reset_fade_out_shape_width (nframes_t width)
        delete points;
 
        /* ensure trim handle stays on top */
-       frame_handle_end->raise_to_top();
+       if (frame_handle_end) {
+               frame_handle_end->raise_to_top();
+       }
 }
 
 void
@@ -778,12 +779,9 @@ AudioRegionView::compute_colors (Gdk::Color const & basic_color)
 {
        RegionView::compute_colors (basic_color);
 
-       uint32_t r, g, b, a;
-
        /* gain color computed in envelope_active_changed() */
 
-       UINT_TO_RGBA (fill_color, &r, &g, &b, &a);
-       fade_color = RGBA_TO_UINT(r,g,b,120);
+       fade_color = UINT_RGBA_CHANGE_A (fill_color, 120);
 }
 
 void
@@ -1038,7 +1036,7 @@ AudioRegionView::add_gain_point_event (ArdourCanvas::Item *item, GdkEvent *ev)
 
        item->w2i (x, y);
 
-       nframes_t fx = trackview.editor().pixel_to_frame (x);
+       framepos_t fx = trackview.editor().pixel_to_frame (x);
 
        if (fx > _region->length()) {
                return;
@@ -1052,9 +1050,9 @@ AudioRegionView::add_gain_point_event (ArdourCanvas::Item *item, GdkEvent *ev)
 
        gain_line->view_to_model_coord (x, y);
 
-        /* XXX STATEFUL: can't convert to stateful diff until we 
-           can represent automation data with it.
-        */
+       /* XXX STATEFUL: can't convert to stateful diff until we 
+          can represent automation data with it.
+       */
 
        trackview.session()->begin_reversible_command (_("add gain control point"));
        XMLNode &before = audio_region()->envelope()->get_state();
@@ -1229,36 +1227,34 @@ AudioRegionView::add_ghost (TimeAxisView& tv)
 }
 
 void
-AudioRegionView::entered ()
+AudioRegionView::entered (bool internal_editing)
 {
+       trackview.editor().set_current_trimmable (_region);
+       trackview.editor().set_current_movable (_region);
+
        if (gain_line && _flags & EnvelopeVisible) {
                gain_line->show_all_control_points ();
        }
 
-       uint32_t r,g,b,a;
-       UINT_TO_RGBA(fade_color,&r,&g,&b,&a);
-       a=255;
-
-       if (fade_in_handle) {
-               fade_in_handle->property_fill_color_rgba() = RGBA_TO_UINT(r,g,b,a);
-               fade_out_handle->property_fill_color_rgba() = RGBA_TO_UINT(r,g,b,a);
+       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);
        }
 }
 
 void
 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 ();
        }
 
-       uint32_t r,g,b,a;
-       UINT_TO_RGBA(fade_color,&r,&g,&b,&a);
-       a=0;
-
        if (fade_in_handle) {
-               fade_in_handle->property_fill_color_rgba() = RGBA_TO_UINT(r,g,b,a);
-               fade_out_handle->property_fill_color_rgba() = RGBA_TO_UINT(r,g,b,a);
+               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);
        }
 }
 
@@ -1322,11 +1318,12 @@ AudioRegionView::set_frame_color ()
                fill_opacity = 0;
        }
 
-       uint32_t r,g,b,a;
-
        if (_selected && should_show_selection) {
-               UINT_TO_RGBA(ARDOUR_UI::config()->canvasvar_SelectedFrameBase.get(), &r, &g, &b, &a);
-               frame->property_fill_color_rgba() = RGBA_TO_UINT(r, g, b, fill_opacity ? fill_opacity : a);
+               if (fill_opacity) {
+                       frame->property_fill_color_rgba() = UINT_RGBA_CHANGE_A (ARDOUR_UI::config()->canvasvar_SelectedFrameBase.get(), fill_opacity);
+               } else {
+                       frame->property_fill_color_rgba() = ARDOUR_UI::config()->canvasvar_SelectedFrameBase.get();
+               }
 
                for (vector<ArdourCanvas::WaveView*>::iterator w = waves.begin(); w != waves.end(); ++w) {
                        if (_region->muted()) {
@@ -1338,8 +1335,7 @@ AudioRegionView::set_frame_color ()
                }
        } else {
                if (_recregion) {
-                       UINT_TO_RGBA(ARDOUR_UI::config()->canvasvar_RecordingRect.get(), &r, &g, &b, &a);
-                       frame->property_fill_color_rgba() = RGBA_TO_UINT(r, g, b, a);
+                       frame->property_fill_color_rgba() = ARDOUR_UI::config()->canvasvar_RecordingRect.get();
 
                        for (vector<ArdourCanvas::WaveView*>::iterator w = waves.begin(); w != waves.end(); ++w) {
                                if (_region->muted()) {
@@ -1350,8 +1346,11 @@ AudioRegionView::set_frame_color ()
                                }
                        }
                } else {
-                       UINT_TO_RGBA(ARDOUR_UI::config()->canvasvar_FrameBase.get(), &r, &g, &b, &a);
-                       frame->property_fill_color_rgba() = RGBA_TO_UINT(r, g, b, fill_opacity ? fill_opacity : a);
+                       if (fill_opacity) {
+                               frame->property_fill_color_rgba() = UINT_RGBA_CHANGE_A (ARDOUR_UI::config()->canvasvar_FrameBase.get(), fill_opacity);
+                       } else {
+                               frame->property_fill_color_rgba() = ARDOUR_UI::config()->canvasvar_FrameBase.get();
+                       }
 
                        for (vector<ArdourCanvas::WaveView*>::iterator w = waves.begin(); w != waves.end(); ++w) {
                                if (_region->muted()) {
@@ -1402,8 +1401,10 @@ AudioRegionView::update_coverage_frames (LayerDisplay d)
 {
        RegionView::update_coverage_frames (d);
 
-       fade_in_handle->raise_to_top ();
-       fade_out_handle->raise_to_top ();
+       if (fade_in_handle) {
+               fade_in_handle->raise_to_top ();
+               fade_out_handle->raise_to_top ();
+       }
 }
 
 void
@@ -1414,9 +1415,27 @@ 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 ()
 {
@@ -1435,7 +1454,7 @@ AudioRegionView::transients_changed ()
        }
 
        AnalysisFeatureList::const_iterator i;
-       list<std::pair<nframes64_t, ArdourCanvas::SimpleLine*> >::iterator l;
+       list<std::pair<framepos_t, ArdourCanvas::SimpleLine*> >::iterator l;
 
        for (i = analysis_features.begin(), l = feature_lines.begin(); i != analysis_features.end() && l != feature_lines.end(); ++i, ++l) {
                (*l).first = *i;
@@ -1452,17 +1471,17 @@ AudioRegionView::transients_changed ()
 }
 
 void
-AudioRegionView::update_transient(float old_pos, float new_pos)
+AudioRegionView::update_transient(float /*old_pos*/, float new_pos)
 {
        /* Find frame at old pos, calulate new frame then update region transients*/
-       list<std::pair<nframes64_t, ArdourCanvas::SimpleLine*> >::iterator l;
+       list<std::pair<framepos_t, ArdourCanvas::SimpleLine*> >::iterator l;
 
        for (l = feature_lines.begin(); l != feature_lines.end(); ++l) {
                /* Simple line x1 has been updated in drag so we compare to new_pos */
                if (rint(new_pos) == rint((*l).second->property_x1())) {
                    
-                   nframes64_t old_frame = (*l).first;
-                   nframes64_t new_frame = trackview.editor().pixel_to_frame (new_pos);
+                   framepos_t old_frame = (*l).first;
+                   framepos_t new_frame = trackview.editor().pixel_to_frame (new_pos);
 
                    _region->update_transient (old_frame, new_frame);
                    
@@ -1475,7 +1494,7 @@ void
 AudioRegionView::remove_transient(float pos)
 {
        /* Find frame at old pos, calulate new frame then update region transients*/
-       list<std::pair<nframes64_t, ArdourCanvas::SimpleLine*> >::iterator l;
+       list<std::pair<framepos_t, ArdourCanvas::SimpleLine*> >::iterator l;
 
        for (l = feature_lines.begin(); l != feature_lines.end(); ++l) {
                /* Simple line x1 has been updated in drag so we compare to new_pos */
@@ -1484,4 +1503,12 @@ AudioRegionView::remove_transient(float pos)
                    break;
                }
        }
-}
\ No newline at end of file
+}
+
+void
+AudioRegionView::thaw_after_trim ()
+{
+       RegionView::thaw_after_trim ();
+
+       unhide_envelope ();
+}