remove all calls to Outline::set_outline_width (1) because this is "wrong" when using...
authorPaul Davis <paul@linuxaudiosystems.com>
Wed, 17 Apr 2013 14:56:04 +0000 (10:56 -0400)
committerPaul Davis <paul@linuxaudiosystems.com>
Wed, 17 Apr 2013 14:56:04 +0000 (10:56 -0400)
gtk2_ardour/audio_region_view.cc
gtk2_ardour/automation_line.cc
gtk2_ardour/control_point.cc
gtk2_ardour/crossfade_edit.cc
gtk2_ardour/editor_canvas.cc
gtk2_ardour/editor_cursors.cc
gtk2_ardour/marker.cc
gtk2_ardour/region_view.cc

index eae9eb5f78abc15bf30f425b23fffd4b1da1cc02..b4cdcfb8dc4ca1eb691bb7a09d9db08990e04518 100644 (file)
@@ -304,11 +304,9 @@ AudioRegionView::fade_in_active_changed ()
        if (audio_region()->fade_in_active()) {
                /* XXX: make a themable colour */
                fade_in_shape->set_fill_color (RGBA_TO_UINT (45, 45, 45, 90));
-               fade_in_shape->set_outline_width (1);
        } else {
                /* XXX: make a themable colour */
                fade_in_shape->set_fill_color (RGBA_TO_UINT (45, 45, 45, 20));
-               fade_in_shape->set_outline_width (1);
        }
 }
 
@@ -318,11 +316,9 @@ AudioRegionView::fade_out_active_changed ()
        if (audio_region()->fade_out_active()) {
                /* XXX: make a themable colour */
                fade_out_shape->set_fill_color (RGBA_TO_UINT (45, 45, 45, 90));
-               fade_out_shape->set_outline_width (1);
        } else {
                /* XXX: make a themable colour */
                fade_out_shape->set_fill_color (RGBA_TO_UINT (45, 45, 45, 20));
-               fade_out_shape->set_outline_width (1);
        }
 }
 
@@ -1270,7 +1266,6 @@ AudioRegionView::transients_changed ()
                canvas_item->set (ArdourCanvas::Duple (-1.0, 2.0),
                                  ArdourCanvas::Duple (1.0, _height - TimeAxisViewItem::NAME_HIGHLIGHT_SIZE - 1));
 
-               canvas_item->set_outline_width (1);
                canvas_item->property_first_arrowhead() = TRUE;
                canvas_item->property_last_arrowhead() = TRUE;
                canvas_item->property_arrow_shape_a() = 11.0;
index fa26177093d3af14f95c58d13844c22240a861a6..0b894d16474ad84eb6019878be23b80c10d205fe 100644 (file)
@@ -92,7 +92,6 @@ AutomationLine::AutomationLine (const string& name, TimeAxisView& tv, ArdourCanv
        group = new ArdourCanvas::Group (&parent);
 
        line = new ArdourCanvas::Curve (group);
-       line->set_outline_width (1);
        line->set_data ("line", this);
 
        line->Event.connect (sigc::mem_fun (*this, &AutomationLine::event_handler));
index 7687beecaf8f680421921df44dcaadee9cfb745e..8491534ec81bc274a23e348d4e2a019eec070ebf 100644 (file)
@@ -48,7 +48,6 @@ ControlPoint::ControlPoint (AutomationLine& al)
        _item->set_fill (false);
        _item->set_fill_color (ARDOUR_UI::config()->get_canvasvar_ControlPointFill());
        _item->set_outline_color (ARDOUR_UI::config()->get_canvasvar_ControlPointOutline());
-       _item->set_outline_width (1);
        _item->set_data ("control_point", this);
        _item->Event.connect (sigc::mem_fun (this, &ControlPoint::event_handler));
 
@@ -74,7 +73,6 @@ ControlPoint::ControlPoint (const ControlPoint& other, bool /*dummy_arg_to_force
        _item = new ArdourCanvas::Rectangle (&_line.canvas_group());
        _item->set_fill (false);
        _item->set_outline_color (ARDOUR_UI::config()->get_canvasvar_ControlPointOutline());
-       _item->set_outline_width (1);
 
        /* NOTE: no event handling in copied ControlPoints */
 
index 728a92bc0917eb308fd53698eb742494ac8eee54..5d6e0d465d29e05888c704417e0d4554c98d3bc8 100644 (file)
@@ -141,14 +141,12 @@ CrossfadeEditor::CrossfadeEditor (Session* s, boost::shared_ptr<Crossfade> xf, d
        toplevel->Event.connect (sigc::mem_fun (*this, &CrossfadeEditor::canvas_event));
 
        fade[Out].line = new ArdourCanvas::PolyLine (canvas->root());
-       fade[Out].line->set_outline_width (1);
        fade[Out].line->set_outline_color (ARDOUR_UI::config()->get_canvasvar_CrossfadeEditorLine());
 
        fade[Out].shading = new ArdourCanvas::Polygon (canvas->root());
        fade[Out].shading->set_fill_color (ARDOUR_UI::config()->get_canvasvar_CrossfadeEditorLineShading());
 
        fade[In].line = new ArdourCanvas::PolyLine (canvas->root());
-       fade[In].line->set_outline_width (1);
        fade[In].line->set_outline_color (ARDOUR_UI::config()->get_canvasvar_CrossfadeEditorLine());
 
        fade[In].shading = new ArdourCanvas::Polygon (canvas->root());
@@ -468,7 +466,6 @@ CrossfadeEditor::make_point ()
        p->box->set_fill (true);
        p->box->set_fill_color (ARDOUR_UI::config()->get_canvasvar_CrossfadeEditorPointFill());
        p->box->set_outline_color (ARDOUR_UI::config()->get_canvasvar_CrossfadeEditorPointOutline());
-       p->box->set_outline_width (1);
 
        p->curve = fade[current].line;
 
index bf195525b94991fd141570faebe01c846a59dbb6..96236c3138e9e32d2fd7b04bf967c0611a0d0c46 100644 (file)
@@ -93,11 +93,9 @@ Editor::initialize_canvas ()
        time_line_group = new ArdourCanvas::Group (_track_canvas->root());
 
         transport_loop_range_rect = new ArdourCanvas::Rectangle (time_line_group, ArdourCanvas::Rect (0.0, 0.0, 0.0, ArdourCanvas::COORD_MAX));
-       transport_loop_range_rect->set_outline_width (1);
        transport_loop_range_rect->hide();
 
        transport_punch_range_rect = new ArdourCanvas::Rectangle (time_line_group, ArdourCanvas::Rect (0.0, 0.0, 0.0, ArdourCanvas::COORD_MAX));
-       transport_punch_range_rect->set_outline_width (0);
        transport_punch_range_rect->hide();
 
        _trackview_group = new ArdourCanvas::Group (_track_canvas->root());
@@ -108,37 +106,31 @@ Editor::initialize_canvas ()
        meter_bar_group = new ArdourCanvas::Group (_time_bars_canvas->root ());
        meter_bar = new ArdourCanvas::Rectangle (meter_bar_group, ArdourCanvas::Rect (0.0, 0.0, ArdourCanvas::COORD_MAX, timebar_height - 1));
        CANVAS_DEBUG_NAME (meter_bar, "meter Bar");
-       meter_bar->set_outline_width (1);
        meter_bar->set_outline_what (0x8);
 
        tempo_bar_group = new ArdourCanvas::Group (_time_bars_canvas->root ());
        tempo_bar = new ArdourCanvas::Rectangle (tempo_bar_group, ArdourCanvas::Rect (0.0, 0.0, ArdourCanvas::COORD_MAX, timebar_height - 1));
        CANVAS_DEBUG_NAME (tempo_bar, "Tempo  Bar");
-       tempo_bar->set_outline_width (1);
        tempo_bar->set_outline_what (0x8);
 
        range_marker_bar_group = new ArdourCanvas::Group (_time_bars_canvas->root ());
        range_marker_bar = new ArdourCanvas::Rectangle (range_marker_bar_group, ArdourCanvas::Rect (0.0, 0.0, ArdourCanvas::COORD_MAX, timebar_height - 1));
        CANVAS_DEBUG_NAME (range_marker_bar, "Range Marker Bar");
-       range_marker_bar->set_outline_width (1);
        range_marker_bar->set_outline_what (0x8);
 
        transport_marker_bar_group = new ArdourCanvas::Group (_time_bars_canvas->root ());
        transport_marker_bar = new ArdourCanvas::Rectangle (transport_marker_bar_group, ArdourCanvas::Rect (0.0, 0.0, ArdourCanvas::COORD_MAX, timebar_height - 1));
        CANVAS_DEBUG_NAME (transport_marker_bar, "transport Marker Bar");
-       transport_marker_bar->set_outline_width (1);
        transport_marker_bar->set_outline_what (0x8);
 
        marker_bar_group = new ArdourCanvas::Group (_time_bars_canvas->root ());
        marker_bar = new ArdourCanvas::Rectangle (marker_bar_group, ArdourCanvas::Rect (0.0, 0.0, ArdourCanvas::COORD_MAX, timebar_height - 1));
        CANVAS_DEBUG_NAME (marker_bar, "Marker Bar");
-       marker_bar->set_outline_width (1);
        marker_bar->set_outline_what (0x8);
 
        cd_marker_bar_group = new ArdourCanvas::Group (_time_bars_canvas->root ());
        cd_marker_bar = new ArdourCanvas::Rectangle (cd_marker_bar_group, ArdourCanvas::Rect (0.0, 0.0, ArdourCanvas::COORD_MAX, timebar_height - 1));
        CANVAS_DEBUG_NAME (cd_marker_bar, "CD Marker Bar");
-       cd_marker_bar->set_outline_width (1);
        cd_marker_bar->set_outline_what (0x8);
 
        _time_markers_group = new ArdourCanvas::Group (_time_bars_canvas->root());
@@ -181,7 +173,6 @@ Editor::initialize_canvas ()
 
        // used to show zoom mode active zooming
        zoom_rect = new ArdourCanvas::Rectangle (_track_canvas->root(), ArdourCanvas::Rect (0.0, 0.0, 0.0, 0.0));
-       zoom_rect->set_outline_width (1);
        zoom_rect->hide();
 
        zoom_rect->Event.connect (sigc::bind (sigc::mem_fun (*this, &Editor::canvas_zoom_rect_event), (ArdourCanvas::Item*) 0));
@@ -189,7 +180,6 @@ Editor::initialize_canvas ()
        // used as rubberband rect
        rubberband_rect = new ArdourCanvas::Rectangle (_trackview_group, ArdourCanvas::Rect (0.0, 0.0, 0.0, 0.0));
 
-       rubberband_rect->set_outline_width (1);
        rubberband_rect->hide();
 
        tempo_bar->Event.connect (sigc::bind (sigc::mem_fun (*this, &Editor::canvas_tempo_bar_event), tempo_bar));
index b012b97fe983a8976ba5aadd91f40ae30fd070dd..a1e172674762b1c62b08c0d8c382a01066c3b83d 100644 (file)
@@ -40,9 +40,6 @@ EditorCursor::EditorCursor (Editor& ed, bool (Editor::*callbck)(GdkEvent*,Ardour
        CANVAS_DEBUG_NAME ((&_time_bars_canvas_item), "timebars editor cursor");
        CANVAS_DEBUG_NAME ((&_track_canvas_item), "track canvas editor cursor");
 
-       _time_bars_canvas_item.set_outline_width (1);
-       _track_canvas_item.set_outline_width (1);
-
        _time_bars_canvas_item.set_show_head (0, true);
        _time_bars_canvas_item.set_head_height (0, 9);
        _time_bars_canvas_item.set_head_width (0, 16);
index e45cf0fb2a0466c1f68f764ff17a1a36c07013f5..5afc59a8b4b6491f0ad370077a7f3c1ca34fd935 100644 (file)
@@ -248,7 +248,6 @@ Marker::Marker (PublicEditor& ed, ArdourCanvas::Group& parent, guint32 rgba, con
 #ifdef CANVAS_DEBUG
        _name_background->name = string_compose ("Marker::_name_background for %1", annotation);
 #endif 
-       _name_background->set_outline_width (1);
 
        /* adjust to properly locate the tip */
 
@@ -258,7 +257,6 @@ Marker::Marker (PublicEditor& ed, ArdourCanvas::Group& parent, guint32 rgba, con
 #endif 
        mark->set (*points);
        set_color_rgba (rgba);
-       mark->set_outline_width (1);
 
        /* setup name pixbuf sizes */
        name_font = get_font_for_style (N_("MarkerText"));
index 2d3221da265b96ed54c598ad9e38e0f8e6248e4f..37ddb45917bd30e198aa3f9bd075f36da4d083b6 100644 (file)
@@ -636,7 +636,6 @@ RegionView::region_sync_changed ()
 
                sync_line = new ArdourCanvas::Line (group);
                sync_line->set_outline_color (RGBA_TO_UINT(0,255,0,255)); // FIXME make a themeable colour
-               sync_line->set_outline_width  (1);
        }
 
        /* this has to handle both a genuine change of position, a change of samples_per_pixel