move all (G)UI related configuration parameters into UIConfiguration, not RCConfiguration
[ardour.git] / gtk2_ardour / audio_region_view.cc
index 8f5371adad957df6bcddaaea7fb6b98855701b2f..72b06a23914571a15e8ce2bceb4c234f6ae777e0 100644 (file)
@@ -46,6 +46,7 @@
 #include "canvas/xfade_curve.h"
 #include "canvas/debug.h"
 #include "canvas/utils.h"
+#include "canvas/colors.h"
 
 #include "streamview.h"
 #include "audio_region_view.h"
@@ -71,7 +72,6 @@ using namespace PBD;
 using namespace Editing;
 using namespace ArdourCanvas;
 
-static const int32_t sync_mark_width = 9;
 static double const handle_size = 10; /* height of fade handles */
 
 AudioRegionView::AudioRegionView (ArdourCanvas::Container *parent, RouteTimeAxisView &tv, boost::shared_ptr<AudioRegion> r, double spu,
@@ -153,28 +153,28 @@ AudioRegionView::init (bool wfd)
                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_outline_color (ArdourCanvas::rgba_to_color (0, 0, 0, 1.0));
-               fade_in_handle->set_fill_color (ARDOUR_UI::config()->get_canvasvar_InactiveFadeHandle());
+               fade_in_handle->set_fill_color (ARDOUR_UI::config()->color ("inactive fade handle"));
                fade_in_handle->set_data ("regionview", this);
                fade_in_handle->hide ();
 
                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_outline_color (ArdourCanvas::rgba_to_color (0, 0, 0, 1.0));
-               fade_out_handle->set_fill_color (ARDOUR_UI::config()->get_canvasvar_InactiveFadeHandle());
+               fade_out_handle->set_fill_color (ARDOUR_UI::config()->color ("inactive fade handle"));
                fade_out_handle->set_data ("regionview", this);
                fade_out_handle->hide ();
 
                fade_in_trim_handle = new ArdourCanvas::Rectangle (group);
                CANVAS_DEBUG_NAME (fade_in_handle, string_compose ("fade in trim handle for %1", region()->name()));
                fade_in_trim_handle->set_outline_color (ArdourCanvas::rgba_to_color (0, 0, 0, 1.0));
-               fade_in_trim_handle->set_fill_color (ARDOUR_UI::config()->get_canvasvar_InactiveFadeHandle());
+               fade_in_trim_handle->set_fill_color (ARDOUR_UI::config()->color ("inactive fade handle"));
                fade_in_trim_handle->set_data ("regionview", this);
                fade_in_trim_handle->hide ();
 
                fade_out_trim_handle = new ArdourCanvas::Rectangle (group);
                CANVAS_DEBUG_NAME (fade_out_handle, string_compose ("fade out trim handle for %1", region()->name()));
                fade_out_trim_handle->set_outline_color (ArdourCanvas::rgba_to_color (0, 0, 0, 1.0));
-               fade_out_trim_handle->set_fill_color (ARDOUR_UI::config()->get_canvasvar_InactiveFadeHandle());
+               fade_out_trim_handle->set_fill_color (ARDOUR_UI::config()->color ("inactive fade handle"));
                fade_out_trim_handle->set_data ("regionview", this);
                fade_out_trim_handle->hide ();
        }
@@ -315,7 +315,7 @@ AudioRegionView::fade_in_active_changed ()
                if (audio_region()->fade_in_active()) {
                        start_xfade_rect->set_fill (false);
                } else {
-                       start_xfade_rect->set_fill_color (ARDOUR_UI::config()->get_canvasvar_InactiveCrossfade());
+                       start_xfade_rect->set_fill_color (ARDOUR_UI::config()->color_mod ("inactive crossfade", "inactive crossfade"));
                        start_xfade_rect->set_fill (true);
                }
        }
@@ -328,7 +328,7 @@ AudioRegionView::fade_out_active_changed ()
                if (audio_region()->fade_out_active()) {
                        end_xfade_rect->set_fill (false);
                } else {        
-                       end_xfade_rect->set_fill_color (ARDOUR_UI::config()->get_canvasvar_InactiveCrossfade());
+                       end_xfade_rect->set_fill_color (ARDOUR_UI::config()->color_mod ("inactive crossfade", "inactive crossfade"));
                        end_xfade_rect->set_fill (true);
                }
        }
@@ -486,7 +486,7 @@ AudioRegionView::set_height (gdouble height)
                        ht = (((height - 2 * wcnt) - NAME_HIGHLIGHT_SIZE) / (double) wcnt);
                }
 
-               gdouble yoff = 1 + n * (ht + 1);
+               gdouble yoff = n * (ht + 1);
 
                waves[n]->set_height (ht);
                waves[n]->set_y_position (yoff + 2);
@@ -568,7 +568,7 @@ AudioRegionView::reset_fade_in_shape_width (boost::shared_ptr<AudioRegion> ar, f
 
        if (fade_in_handle->visible()) {
                //see comment for drag_start
-               entered(false);
+               entered();
        }
 
        if (pwidth < 5) {
@@ -632,19 +632,20 @@ AudioRegionView::reset_fade_out_shape_width (boost::shared_ptr<AudioRegion> ar,
 
        width = std::max ((framecnt_t) 64, width);
 
-       double const pwidth = rint(trackview.editor().sample_to_pixel (width));
 
+       double const pwidth = rint(trackview.editor().sample_to_pixel (width));
+       
        /* the right edge should be right on the region frame is the pixel
         * width is zero. Hence the additional + 1.0 at the end.
         */
 
-       double const handle_right = rint(trackview.editor().sample_to_pixel (_region->length()) + TimeAxisViewItem::RIGHT_EDGE_SHIFT - pwidth);
-       double const trim_handle_right = rint(trackview.editor().sample_to_pixel (_region->length()) + TimeAxisViewItem::RIGHT_EDGE_SHIFT);
+       double const handle_right = rint(trackview.editor().sample_to_pixel (_region->length()) - pwidth);
+       double const trim_handle_right = rint(trackview.editor().sample_to_pixel (_region->length()));
 
        /* Put the fade out handle so that its right side is at the end-of-fade line;
         */
-       fade_out_handle->set_x0 (1 + handle_right - handle_size);
-       fade_out_handle->set_x1 (1 + handle_right);
+       fade_out_handle->set_x0 (handle_right - handle_size);
+       fade_out_handle->set_x1 (handle_right);
        if (fade_out_trim_handle) {
                fade_out_trim_handle->set_x0 (1 + trim_handle_right - handle_size);
                fade_out_trim_handle->set_x1 (1 + trim_handle_right);
@@ -652,7 +653,7 @@ AudioRegionView::reset_fade_out_shape_width (boost::shared_ptr<AudioRegion> ar,
 
        if (fade_out_handle->visible()) {
                //see comment for drag_start
-               entered(false);
+               entered();
        }
        /* don't show shape if its too small */
 
@@ -668,10 +669,10 @@ AudioRegionView::reset_fade_out_shape_width (boost::shared_ptr<AudioRegion> ar,
 
        double effective_height;
 
-       if (_height >= NAME_HIGHLIGHT_THRESH) {
-               effective_height = _height - NAME_HIGHLIGHT_SIZE;
-       } else {
-               effective_height = _height;
+       effective_height = _height - 1.0;
+
+       if (ARDOUR_UI::config()->get_show_name_highlight() && effective_height >= NAME_HIGHLIGHT_THRESH) {
+               effective_height -= NAME_HIGHLIGHT_SIZE;
        }
 
        /* points *MUST* be in anti-clockwise order */
@@ -683,10 +684,10 @@ AudioRegionView::reset_fade_out_shape_width (boost::shared_ptr<AudioRegion> ar,
        double length = list->length();
 
        points.assign (list->size(), Duple());
-
+       
        for (x = list->begin(), pi = 0; x != list->end(); ++x, ++pi) {
-               points[pi].x = 1.0 + _pixel_width - pwidth + (pwidth * ((*x)->when/length));
-               points[pi].y = effective_height - ((*x)->value * effective_height);
+               points[pi].x = _pixel_width - pwidth + (pwidth * ((*x)->when/length));
+               points[pi].y = 1.0 + effective_height - ((*x)->value * effective_height);
        }
 
        /* draw the line */
@@ -736,18 +737,16 @@ AudioRegionView::redraw_start_xfade_to (boost::shared_ptr<AudioRegion> ar, frame
        if (!start_xfade_curve) {
                start_xfade_curve = new ArdourCanvas::XFadeCurve (group, ArdourCanvas::XFadeCurve::Start);
                CANVAS_DEBUG_NAME (start_xfade_curve, string_compose ("xfade start out line for %1", region()->name()));
-               start_xfade_curve->set_fill_color (ARDOUR_UI::config()->get_canvasvar_ActiveCrossfade());
-               start_xfade_curve->set_outline_color (ARDOUR_UI::config()->get_canvasvar_CrossfadeLine());
+               start_xfade_curve->set_fill_color (ARDOUR_UI::config()->color_mod ("active crossfade", "crossfade alpha"));
+               start_xfade_curve->set_outline_color (ARDOUR_UI::config()->color ("crossfade line"));
                start_xfade_curve->set_ignore_events (true);
        }
        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_outline_color (ARDOUR_UI::config()->get_canvasvar_CrossfadeLine());
+               start_xfade_rect->set_outline_color (ARDOUR_UI::config()->color ("crossfade line"));
                start_xfade_rect->set_fill (false);
                start_xfade_rect->set_outline (false);
-               start_xfade_rect->set_outline_what (ArdourCanvas::Rectangle::What (ArdourCanvas::Rectangle::RIGHT));
-               start_xfade_rect->set_outline_width (0.5);
                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);
        }
@@ -773,7 +772,7 @@ AudioRegionView::redraw_start_xfade_to (boost::shared_ptr<AudioRegion> ar, frame
                for (Points::size_type i = 0, pci = 0; i < npoints; ++i, ++pci) {
                        ArdourCanvas::Duple &p (ipoints[pci]);
                        /* leave x-axis alone but invert with respect to y-axis */
-                       p.y = effective_height - points[pci].y;
+                       p.y = 1.0 + effective_height - points[pci].y;
                }
 
        } else {
@@ -792,8 +791,8 @@ AudioRegionView::redraw_start_xfade_to (boost::shared_ptr<AudioRegion> ar, frame
 
                for (x = inverse->begin(), pi = 0; x != inverse->end(); ++x, ++pi) {
                        ArdourCanvas::Duple& p (ipoints[pi]);
-                       p.x = 1.0 + (rect_width * ((*x)->when/length));
-                       p.y = effective_height - ((*x)->value * effective_height);
+                       p.x = (rect_width * ((*x)->when/length));
+                       p.y = 1.0 + effective_height - ((*x)->value * effective_height);
                }
        }
 
@@ -827,24 +826,22 @@ AudioRegionView::redraw_end_xfade_to (boost::shared_ptr<AudioRegion> ar, framecn
        if (!end_xfade_curve) {
                end_xfade_curve = new ArdourCanvas::XFadeCurve (group, ArdourCanvas::XFadeCurve::End);
                CANVAS_DEBUG_NAME (end_xfade_curve, string_compose ("xfade end out line for %1", region()->name()));
-               end_xfade_curve->set_fill_color (ARDOUR_UI::config()->get_canvasvar_ActiveCrossfade());
-               end_xfade_curve->set_outline_color (ARDOUR_UI::config()->get_canvasvar_CrossfadeLine());
+               end_xfade_curve->set_fill_color (ARDOUR_UI::config()->color_mod ("active crossfade", "crossfade alpha"));
+               end_xfade_curve->set_outline_color (ARDOUR_UI::config()->color ("crossfade line"));
                end_xfade_curve->set_ignore_events (true);
        }
 
        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_outline_color (ARDOUR_UI::config()->get_canvasvar_CrossfadeLine());
+               end_xfade_rect->set_outline_color (ARDOUR_UI::config()->color ("crossfade line"));
                end_xfade_rect->set_fill (false);
                end_xfade_rect->set_outline (false);
-               end_xfade_rect->set_outline_what (ArdourCanvas::Rectangle::What (ArdourCanvas::Rectangle::LEFT));
-               end_xfade_rect->set_outline_width (0.5);
                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);
        }
 
-       end_xfade_rect->set (ArdourCanvas::Rect (rect_edge, 0.0, rect_edge + rect_width + TimeAxisViewItem::RIGHT_EDGE_SHIFT, effective_height));
+       end_xfade_rect->set (ArdourCanvas::Rect (rect_edge, 0.0, rect_edge + rect_width, effective_height));
 
        /* fade in line */
 
@@ -888,8 +885,8 @@ AudioRegionView::redraw_end_xfade_to (boost::shared_ptr<AudioRegion> ar, framecn
 
                for (x = inverse->begin(), i = 0, pi = 0; x != inverse->end(); ++x, ++pi, ++i) {
                        ArdourCanvas::Duple& p (ipoints[pi]);
-                       p.x = 1.0 + (rect_width * ((*x)->when/length)) + rend;
-                       p.y = effective_height - ((*x)->value * effective_height);
+                       p.x = (rect_width * ((*x)->when/length)) + rend;
+                       p.y = 1.0 + effective_height - ((*x)->value * effective_height);
                }
        }
 
@@ -962,7 +959,7 @@ AudioRegionView::set_samples_per_pixel (gdouble fpp)
 {
        RegionView::set_samples_per_pixel (fpp);
 
-       if (Config->get_show_waveforms ()) {
+       if (ARDOUR_UI::config()->get_show_waveforms ()) {
                for (uint32_t n = 0; n < waves.size(); ++n) {
                        waves[n]->set_samples_per_pixel (fpp);
                }
@@ -990,33 +987,33 @@ AudioRegionView::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());
+                                          ARDOUR_UI::config()->color ("gain line") : 
+                                          ARDOUR_UI::config()->color_mod ("gain line inactive", "gain line inactive"));
        }
 
        set_waveform_colors ();
 
        if (start_xfade_curve) {
-               start_xfade_curve->set_fill_color (ARDOUR_UI::config()->get_canvasvar_ActiveCrossfade());
-               start_xfade_curve->set_outline_color (ARDOUR_UI::config()->get_canvasvar_CrossfadeLine());
+               start_xfade_curve->set_fill_color (ARDOUR_UI::config()->color_mod ("active crossfade", "crossfade alpha"));
+               start_xfade_curve->set_outline_color (ARDOUR_UI::config()->color ("crossfade line"));
        }
        if (end_xfade_curve) {
-               end_xfade_curve->set_fill_color (ARDOUR_UI::config()->get_canvasvar_ActiveCrossfade());
-               end_xfade_curve->set_outline_color (ARDOUR_UI::config()->get_canvasvar_CrossfadeLine());
+               end_xfade_curve->set_fill_color (ARDOUR_UI::config()->color_mod ("active crossfade", "crossfade alpha"));
+               end_xfade_curve->set_outline_color (ARDOUR_UI::config()->color ("crossfade line"));
        }
 
        if (start_xfade_rect) {
-               start_xfade_rect->set_outline_color (ARDOUR_UI::config()->get_canvasvar_CrossfadeLine());
+               start_xfade_rect->set_outline_color (ARDOUR_UI::config()->color ("crossfade line"));
        }
        if (end_xfade_rect) {
-               end_xfade_rect->set_outline_color (ARDOUR_UI::config()->get_canvasvar_CrossfadeLine());
+               end_xfade_rect->set_outline_color (ARDOUR_UI::config()->color ("crossfade line"));
        }
 }
 
 void
 AudioRegionView::setup_waveform_visibility ()
 {
-       if (Config->get_show_waveforms ()) {
+       if (ARDOUR_UI::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.
@@ -1053,10 +1050,10 @@ AudioRegionView::update_envelope_visibility ()
                return;
        }
 
-       if (Config->get_show_region_gain() || trackview.editor().current_mouse_mode() == Editing::MouseGain || trackview.editor().current_mouse_mode() == Editing::MouseRange ) {
-               gain_line->add_visibility (AutomationLine::Line);
+       if (ARDOUR_UI::config()->get_show_region_gain() || trackview.editor().current_mouse_mode() == Editing::MouseDraw || trackview.editor().current_mouse_mode() == Editing::MouseRange ) {
+               gain_line->set_visibility (AutomationLine::VisibleAspects(AutomationLine::ControlPoints|AutomationLine::Line));
        } else {
-               gain_line->hide ();
+               gain_line->set_visibility (AutomationLine::VisibleAspects(0));
        }
 }
 
@@ -1132,7 +1129,7 @@ AudioRegionView::create_one_wave (uint32_t which, bool /*direct*/)
                ht = ((trackview.current_height() - NAME_HIGHLIGHT_SIZE) / (double) nchans);
        }
 
-       gdouble yoff = 1 + which * ht;
+       gdouble yoff = which * ht;
 
        WaveView *wave = new WaveView (group, audio_region ());
        CANVAS_DEBUG_NAME (wave, string_compose ("wave view for chn %1 of %2", which, get_item_name()));
@@ -1142,11 +1139,11 @@ AudioRegionView::create_one_wave (uint32_t which, bool /*direct*/)
        wave->set_height (ht);
        wave->set_samples_per_pixel (samples_per_pixel);
        wave->set_show_zero_line (true);
-       wave->set_clip_level (Config->get_waveform_clip_level ());
+       wave->set_clip_level (ARDOUR_UI::config()->get_waveform_clip_level ());
 
        wave->Event.connect (sigc::bind (sigc::mem_fun (PublicEditor::instance(), &PublicEditor::canvas_wave_view_event), wave, this));
        
-       switch (Config->get_waveform_shape()) {
+       switch (ARDOUR_UI::config()->get_waveform_shape()) {
        case Rectified:
                wave->set_shape (WaveView::Rectified);
                break;
@@ -1154,13 +1151,13 @@ AudioRegionView::create_one_wave (uint32_t which, bool /*direct*/)
                wave->set_shape (WaveView::Normal);
        }
                
-       wave->set_logscaled (Config->get_waveform_scale() == Logarithmic);
+       wave->set_logscaled (ARDOUR_UI::config()->get_waveform_scale() == Logarithmic);
 
        vector<ArdourCanvas::WaveView*> v;
        v.push_back (wave);
        set_some_waveform_colors (v);
 
-       if (!Config->get_show_waveforms ()) {
+       if (!ARDOUR_UI::config()->get_show_waveforms ()) {
                wave->hide();
        }
 
@@ -1240,7 +1237,7 @@ AudioRegionView::add_gain_point_event (ArdourCanvas::Item *item, GdkEvent *ev, b
           can represent automation data with it.
        */
 
-       trackview.session()->begin_reversible_command (_("add gain control point"));
+       trackview.editor().begin_reversible_command (_("add gain control point"));
        XMLNode &before = audio_region()->envelope()->get_state();
 
        if (!audio_region()->envelope_active()) {
@@ -1254,7 +1251,7 @@ AudioRegionView::add_gain_point_event (ArdourCanvas::Item *item, GdkEvent *ev, b
 
        XMLNode &after = audio_region()->envelope()->get_state();
        trackview.session()->add_command (new MementoCommand<AutomationList>(*audio_region()->envelope().get(), &before, &after));
-       trackview.session()->commit_reversible_command ();
+       trackview.editor().commit_reversible_command ();
 }
 
 void
@@ -1301,17 +1298,14 @@ AudioRegionView::add_ghost (TimeAxisView& tv)
 }
 
 void
-AudioRegionView::entered (bool internal_editing)
+AudioRegionView::entered ()
 {
        trackview.editor().set_current_trimmable (_region);
        trackview.editor().set_current_movable (_region);
-       
-       if (gain_line) {
-               /* these may or may not be visible depending on mouse mode */
-               gain_line->add_visibility (AutomationLine::ControlPoints);
-       }
 
-       if (!internal_editing &&  ( trackview.editor().current_mouse_mode() == Editing::MouseObject ) ) {
+       update_envelope_visibility();
+
+       if ((trackview.editor().current_mouse_mode() == Editing::MouseObject)) {
                if (start_xfade_rect) {
                        start_xfade_rect->set_outline (true);
                }
@@ -1370,8 +1364,8 @@ AudioRegionView::envelope_active_changed ()
 {
        if (gain_line) {
                gain_line->set_line_color (audio_region()->envelope_active() ? 
-                                          ARDOUR_UI::config()->get_canvasvar_GainLine() : 
-                                          ARDOUR_UI::config()->get_canvasvar_GainLineInactive());
+                                          ARDOUR_UI::config()->color ("gain line") : 
+                                          ARDOUR_UI::config()->color_mod ("gain line inactive", "gain line inactive"));
                update_envelope_visibility ();
        }
 }
@@ -1402,44 +1396,30 @@ AudioRegionView::set_some_waveform_colors (vector<ArdourCanvas::WaveView*>& wave
 {
        ArdourCanvas::Color fill;
        ArdourCanvas::Color outline;
-       ArdourCanvas::Color clip = ARDOUR_UI::config()->get_canvasvar_WaveFormClip();
-       ArdourCanvas::Color zero = ARDOUR_UI::config()->get_canvasvar_ZeroLine();
+       ArdourCanvas::Color clip = ARDOUR_UI::config()->color ("clipped waveform");
+       ArdourCanvas::Color zero = ARDOUR_UI::config()->color ("zero line");
 
        if (_selected) {
                if (_region->muted()) {
                        /* hide outline with zero alpha */
-                       outline = UINT_RGBA_CHANGE_A(ARDOUR_UI::config()->get_canvasvar_SelectedWaveForm(), 0);
-                       fill = UINT_RGBA_CHANGE_A(ARDOUR_UI::config()->get_canvasvar_SelectedWaveFormFill(), MUTED_ALPHA);
+                       outline = UINT_RGBA_CHANGE_A(ARDOUR_UI::config()->color ("selected waveform outline"), 0);
+                       fill = UINT_RGBA_CHANGE_A(ARDOUR_UI::config()->color ("selected waveform fill"), MUTED_ALPHA);
                } else {
-                       outline = ARDOUR_UI::config()->get_canvasvar_SelectedWaveForm();
-                       fill = ARDOUR_UI::config()->get_canvasvar_SelectedWaveFormFill();
-
-                       if (ARDOUR_UI::config()->get_color_regions_using_track_color()) {
-                               /* just use a slightly transparent version of the selected
-                                * color so that some of the track color bleeds through
-                                */
-                               fill = UINT_RGBA_CHANGE_A (fill, 217);
-                       }
+                       outline = ARDOUR_UI::config()->color ("selected waveform outline");
+                       fill = ARDOUR_UI::config()->color ("selected waveform fill");
                }
        } else {
                if (_recregion) {
-                       outline = ARDOUR_UI::config()->get_canvasvar_RecWaveForm();
-                       fill = ARDOUR_UI::config()->get_canvasvar_RecWaveFormFill();
+                       outline = ARDOUR_UI::config()->color ("recording waveform outline");
+                       fill = ARDOUR_UI::config()->color ("recording waveform fill");
                } else {
                        if (_region->muted()) {
                                /* hide outline with zero alpha */
-                               outline = UINT_RGBA_CHANGE_A(ARDOUR_UI::config()->get_canvasvar_WaveForm(), 0); 
-                               fill = UINT_RGBA_CHANGE_A(ARDOUR_UI::config()->get_canvasvar_WaveFormFill(), MUTED_ALPHA);
+                               outline = UINT_RGBA_CHANGE_A(ARDOUR_UI::config()->color ("waveform outline"), 0);       
+                               fill = UINT_RGBA_CHANGE_A(ARDOUR_UI::config()->color ("waveform fill"), MUTED_ALPHA);
                        } else {
-                               outline = ARDOUR_UI::config()->get_canvasvar_WaveForm();
-                               fill = ARDOUR_UI::config()->get_canvasvar_WaveFormFill();
-
-                               if (ARDOUR_UI::config()->get_color_regions_using_track_color()) {
-                                       /* just use a slightly transparent version of the selected
-                                        * color so that some of the track color bleeds through
-                                        */
-                                       fill = UINT_RGBA_CHANGE_A (fill, 217);
-                               }
+                               outline = ARDOUR_UI::config()->color ("waveform outline");
+                               fill = ARDOUR_UI::config()->color ("waveform fill");
                        }
                }
        }
@@ -1631,7 +1611,7 @@ AudioRegionView::drag_end ()
                // if fade_in_trim_handle or fade_out_trim_handle should
                // be visible. -- If the fade_in_handle is visible
                // we have focus and are not in internal edit mode.
-               entered(false);
+               entered();
        }
 }