move marker text up by 2 pixels
[ardour.git] / gtk2_ardour / marker.cc
index ca4cfbf4c5405d1354388e10d7c8662351e986d3..652a43657e1ebe273fe3765329d9d19c04f92541 100644 (file)
 #include <sigc++/bind.h>
 #include "ardour/tempo.h"
 
+#include "canvas/rectangle.h"
+#include "canvas/group.h"
+#include "canvas/line.h"
+#include "canvas/polygon.h"
+#include "canvas/text.h"
+#include "canvas/canvas.h"
+#include "canvas/debug.h"
+
 #include "ardour_ui.h"
 /*
  * ardour_ui.h include was moved to the top of the list
@@ -30,9 +38,6 @@
 #include "marker.h"
 #include "public_editor.h"
 #include "utils.h"
-#include "canvas_impl.h"
-#include "simpleline.h"
-#include "simplerect.h"
 #include "rgb_macros.h"
 
 #include <gtkmm2ext/utils.h>
@@ -45,18 +50,17 @@ using namespace Gtkmm2ext;
 
 PBD::Signal1<void,Marker*> Marker::CatchDeletion;
 
-Marker::Marker (PublicEditor& ed, ArdourCanvas::Group& parent, ArdourCanvas::Group& line_parent, guint32 rgba, const string& annotation,
-               Type type, nframes_t frame, bool handle_events)
+Marker::Marker (PublicEditor& ed, ArdourCanvas::Group& parent, guint32 rgba, const string& annotation,
+               Type type, framepos_t frame, bool handle_events)
 
        : editor (ed)
        , _parent (&parent)
-       , _line_parent (&line_parent)
-       , _line (0)
+       , _time_bars_line (0)
+       , _track_canvas_line (0)
        , _type (type)
        , _selected (false)
        , _shown (false)
        , _line_shown (false)
-       , _canvas_height (0)
        , _color (rgba)
        , _left_label_limit (DBL_MAX)
        , _right_label_limit (DBL_MAX)
@@ -135,12 +139,12 @@ Marker::Marker (PublicEditor& ed, ArdourCanvas::Group& parent, ArdourCanvas::Gro
        case Mark:
                points = new ArdourCanvas::Points ();
 
-               points->push_back (Gnome::Art::Point (0.0, 0.0));
-               points->push_back (Gnome::Art::Point (6.0, 0.0));
-               points->push_back (Gnome::Art::Point (6.0, 5.0));
-               points->push_back (Gnome::Art::Point (3.0, 13.0));
-               points->push_back (Gnome::Art::Point (0.0, 5.0));
-               points->push_back (Gnome::Art::Point (0.0, 0.0));
+               points->push_back (ArdourCanvas::Duple (0.0, 0.0));
+               points->push_back (ArdourCanvas::Duple (6.0, 0.0));
+               points->push_back (ArdourCanvas::Duple (6.0, 5.0));
+               points->push_back (ArdourCanvas::Duple (3.0, 13.0));
+               points->push_back (ArdourCanvas::Duple (0.0, 5.0));
+               points->push_back (ArdourCanvas::Duple (0.0, 0.0));
 
                _shift = 3;
                _label_offset = 8.0;
@@ -150,12 +154,12 @@ Marker::Marker (PublicEditor& ed, ArdourCanvas::Group& parent, ArdourCanvas::Gro
        case Meter:
 
                points = new ArdourCanvas::Points ();
-               points->push_back (Gnome::Art::Point (3.0, 0.0));
-               points->push_back (Gnome::Art::Point (6.0, 5.0));
-               points->push_back (Gnome::Art::Point (6.0, 10.0));
-               points->push_back (Gnome::Art::Point (0.0, 10.0));
-               points->push_back (Gnome::Art::Point (0.0, 5.0));
-               points->push_back (Gnome::Art::Point (3.0, 0.0));
+               points->push_back (ArdourCanvas::Duple (3.0, 0.0));
+               points->push_back (ArdourCanvas::Duple (6.0, 5.0));
+               points->push_back (ArdourCanvas::Duple (6.0, 10.0));
+               points->push_back (ArdourCanvas::Duple (0.0, 10.0));
+               points->push_back (ArdourCanvas::Duple (0.0, 5.0));
+               points->push_back (ArdourCanvas::Duple (3.0, 0.0));
 
                _shift = 3;
                _label_offset = 8.0;
@@ -163,12 +167,12 @@ Marker::Marker (PublicEditor& ed, ArdourCanvas::Group& parent, ArdourCanvas::Gro
 
        case SessionStart:
        case RangeStart:
-               
+
                points = new ArdourCanvas::Points ();
-               points->push_back (Gnome::Art::Point (0.0, 0.0));
-               points->push_back (Gnome::Art::Point (6.5, 6.5));
-               points->push_back (Gnome::Art::Point (0.0, 13.0));
-               points->push_back (Gnome::Art::Point (0.0, 0.0));
+               points->push_back (ArdourCanvas::Duple (0.0, 0.0));
+               points->push_back (ArdourCanvas::Duple (6.5, 6.5));
+               points->push_back (ArdourCanvas::Duple (0.0, 13.0));
+               points->push_back (ArdourCanvas::Duple (0.0, 0.0));
 
                _shift = 0;
                _label_offset = 13.0;
@@ -177,10 +181,10 @@ Marker::Marker (PublicEditor& ed, ArdourCanvas::Group& parent, ArdourCanvas::Gro
        case SessionEnd:
        case RangeEnd:
                points = new ArdourCanvas::Points ();
-               points->push_back (Gnome::Art::Point (6.5, 6.5));
-               points->push_back (Gnome::Art::Point (13.0, 0.0));
-               points->push_back (Gnome::Art::Point (13.0, 13.0));
-               points->push_back (Gnome::Art::Point (6.5, 6.5));
+               points->push_back (ArdourCanvas::Duple (6.5, 6.5));
+               points->push_back (ArdourCanvas::Duple (13.0, 0.0));
+               points->push_back (ArdourCanvas::Duple (13.0, 13.0));
+               points->push_back (ArdourCanvas::Duple (6.5, 6.5));
 
                _shift = 13;
                _label_offset = 6.0;
@@ -188,10 +192,10 @@ Marker::Marker (PublicEditor& ed, ArdourCanvas::Group& parent, ArdourCanvas::Gro
 
        case LoopStart:
                points = new ArdourCanvas::Points ();
-               points->push_back (Gnome::Art::Point (0.0, 0.0));
-               points->push_back (Gnome::Art::Point (13.0, 13.0));
-               points->push_back (Gnome::Art::Point (0.0, 13.0));
-               points->push_back (Gnome::Art::Point (0.0, 0.0));
+               points->push_back (ArdourCanvas::Duple (0.0, 0.0));
+               points->push_back (ArdourCanvas::Duple (13.0, 13.0));
+               points->push_back (ArdourCanvas::Duple (0.0, 13.0));
+               points->push_back (ArdourCanvas::Duple (0.0, 0.0));
 
                _shift = 0;
                _label_offset = 12.0;
@@ -199,10 +203,10 @@ Marker::Marker (PublicEditor& ed, ArdourCanvas::Group& parent, ArdourCanvas::Gro
 
        case LoopEnd:
                points = new ArdourCanvas::Points ();
-               points->push_back (Gnome::Art::Point (13.0,  0.0));
-               points->push_back (Gnome::Art::Point (13.0, 13.0));
-               points->push_back (Gnome::Art::Point (0.0, 13.0));
-               points->push_back (Gnome::Art::Point (13.0, 0.0));
+               points->push_back (ArdourCanvas::Duple (13.0,  0.0));
+               points->push_back (ArdourCanvas::Duple (13.0, 13.0));
+               points->push_back (ArdourCanvas::Duple (0.0, 13.0));
+               points->push_back (ArdourCanvas::Duple (13.0, 0.0));
 
                _shift = 13;
                _label_offset = 0.0;
@@ -210,10 +214,10 @@ Marker::Marker (PublicEditor& ed, ArdourCanvas::Group& parent, ArdourCanvas::Gro
 
        case  PunchIn:
                points = new ArdourCanvas::Points ();
-               points->push_back (Gnome::Art::Point (0.0, 0.0));
-               points->push_back (Gnome::Art::Point (13.0, 0.0));
-               points->push_back (Gnome::Art::Point (0.0, 13.0));
-               points->push_back (Gnome::Art::Point (0.0, 0.0));
+               points->push_back (ArdourCanvas::Duple (0.0, 0.0));
+               points->push_back (ArdourCanvas::Duple (13.0, 0.0));
+               points->push_back (ArdourCanvas::Duple (0.0, 13.0));
+               points->push_back (ArdourCanvas::Duple (0.0, 0.0));
 
                _shift = 0;
                _label_offset = 13.0;
@@ -221,10 +225,10 @@ Marker::Marker (PublicEditor& ed, ArdourCanvas::Group& parent, ArdourCanvas::Gro
 
        case  PunchOut:
                points = new ArdourCanvas::Points ();
-               points->push_back (Gnome::Art::Point (0.0, 0.0));
-               points->push_back (Gnome::Art::Point (12.0, 0.0));
-               points->push_back (Gnome::Art::Point (12.0, 12.0));
-               points->push_back (Gnome::Art::Point (0.0, 0.0));
+               points->push_back (ArdourCanvas::Duple (0.0, 0.0));
+               points->push_back (ArdourCanvas::Duple (12.0, 0.0));
+               points->push_back (ArdourCanvas::Duple (12.0, 12.0));
+               points->push_back (ArdourCanvas::Duple (0.0, 0.0));
 
                _shift = 13;
                _label_offset = 0.0;
@@ -233,20 +237,26 @@ Marker::Marker (PublicEditor& ed, ArdourCanvas::Group& parent, ArdourCanvas::Gro
        }
 
        frame_position = frame;
-       unit_position = editor.frame_to_unit (frame);
+       unit_position = editor.sample_to_pixel (frame);
        unit_position -= _shift;
-       
-       group = new Group (parent, unit_position, 1.0);
 
-       _name_background = new ArdourCanvas::SimpleRect (*group);
-       _name_background->property_outline_pixels() = 1;
+       group = new ArdourCanvas::Group (&parent, ArdourCanvas::Duple (unit_position, 0));
+#ifdef CANVAS_DEBUG
+       group->name = string_compose ("Marker::group for %1", annotation);
+#endif 
+
+       _name_background = new ArdourCanvas::Rectangle (group);
+#ifdef CANVAS_DEBUG
+       _name_background->name = string_compose ("Marker::_name_background for %1", annotation);
+#endif 
 
        /* adjust to properly locate the tip */
 
-       mark = new Polygon (*group);
-       mark->property_points() = *points;
+       mark = new ArdourCanvas::Polygon (group);
+       CANVAS_DEBUG_NAME (mark, string_compose ("Marker::mark for %1", annotation));
+
+       mark->set (*points);
        set_color_rgba (rgba);
-       mark->property_width_pixels() = 1;
 
        /* setup name pixbuf sizes */
        name_font = get_font_for_style (N_("MarkerText"));
@@ -256,39 +266,44 @@ Marker::Marker (PublicEditor& ed, ArdourCanvas::Group& parent, ArdourCanvas::Gro
        Glib::RefPtr<Pango::Layout> layout = foo.create_pango_layout (X_("Hg")); /* ascender + descender */
        int width;
 
-       layout->set_font_description (*name_font);
+       layout->set_font_description (name_font);
        Gtkmm2ext::get_ink_pixel_size (layout, width, name_height);
-
-       name_pixbuf = new ArdourCanvas::Pixbuf(*group);
-       name_pixbuf->property_x() = _label_offset;
-       name_pixbuf->property_y() = (13/2) - (name_height/2);
+       
+       _name_item = new ArdourCanvas::Text (group);
+       CANVAS_DEBUG_NAME (_name_item, string_compose ("Marker::_name_item for %1", annotation));
+       _name_item->set_font_description (name_font);
+       _name_item->set_color (RGBA_TO_UINT (0,0,0,255));
+       _name_item->set_position (ArdourCanvas::Duple (_label_offset, (13.0 / 2.0) - (name_height / 2.0) - 2.0));
 
        set_name (annotation.c_str());
 
        editor.ZoomChanged.connect (sigc::mem_fun (*this, &Marker::reposition));
 
-       mark->set_data ("marker", this);
-       _name_background->set_data ("marker", this);
+       /* events will be handled by both the group and the mark itself, so
+        * make sure they can both be used to lookup this object.
+        */
 
+       group->set_data ("marker", this);
+       mark->set_data ("marker", this);
+       
        if (handle_events) {
-               group->signal_event().connect (sigc::bind (sigc::mem_fun (editor, &PublicEditor::canvas_marker_event), mark, this));
+               group->Event.connect (sigc::bind (sigc::mem_fun (editor, &PublicEditor::canvas_marker_event), group, this));
        }
-
 }
 
-
 Marker::~Marker ()
 {
        CatchDeletion (this); /* EMIT SIGNAL */
 
        /* destroying the parent group destroys its contents, namely any polygons etc. that we added */
        delete group;
-       delete _line;
+       delete _time_bars_line;
+       delete _track_canvas_line;
 }
 
 void Marker::reparent(ArdourCanvas::Group & parent)
 {
-       group->reparent(parent);
+       group->reparent (&parent);
        _parent = &parent;
 }
 
@@ -311,34 +326,41 @@ Marker::setup_line ()
 {
        if (_shown && (_selected || _line_shown)) {
 
-               if (_line == 0) {
-
-                       _line = new ArdourCanvas::SimpleLine (*_line_parent);
-                       _line->property_color_rgba() = ARDOUR_UI::config()->canvasvar_EditPoint.get();
-
-                       setup_line_x ();
+               if (_time_bars_line == 0) {
 
-                       _line->signal_event().connect (sigc::bind (sigc::mem_fun (editor, &PublicEditor::canvas_marker_event), mark, this));
+                       _time_bars_line = new ArdourCanvas::Line (editor.get_time_bars_group());
+                       _time_bars_line->set_outline_color (ARDOUR_UI::config()->get_canvasvar_EditPoint());
+                       _time_bars_line->Event.connect (sigc::bind (sigc::mem_fun (editor, &PublicEditor::canvas_marker_event), group, this));
+                       
+                       _track_canvas_line = new ArdourCanvas::Line (editor.get_track_canvas_group());
+                       _track_canvas_line->set_outline_color (ARDOUR_UI::config()->get_canvasvar_EditPoint());
+                       _track_canvas_line->Event.connect (sigc::bind (sigc::mem_fun (editor, &PublicEditor::canvas_marker_event), group, this));
                }
-               
-               double yo = 0;
-               if (!_selected) {
-                       /* work out where to start the line from so that it extends only as far as the mark */
-                       double x = 0;
-                       _parent->i2w (x, yo);
-                       _line_parent->w2i (x, yo);
-               }
-
-               _line->property_y1() = yo + 10;
-               _line->property_y2() = yo + 10 + _canvas_height;
 
-               _line->property_color_rgba() = _selected ? ARDOUR_UI::config()->canvasvar_EditPoint.get() : _color;
-               _line->raise_to_top ();
-               _line->show ();
+               ArdourCanvas::Duple g = group->item_to_canvas (ArdourCanvas::Duple (0, 0));
+               ArdourCanvas::Duple d = _time_bars_line->canvas_to_item (ArdourCanvas::Duple (g.x + _shift, 0));
+
+                _time_bars_line->set_x0 (d.x);
+                _time_bars_line->set_x1 (d.x);
+               _time_bars_line->set_y0 (d.y);
+               _time_bars_line->set_y1 (ArdourCanvas::COORD_MAX);
+               _time_bars_line->set_outline_color (_selected ? ARDOUR_UI::config()->get_canvasvar_EditPoint() : _color);
+               _time_bars_line->raise_to_top ();
+               _time_bars_line->show ();
+
+                d = _track_canvas_line->canvas_to_item (ArdourCanvas::Duple (g.x + _shift, 0));
+               _track_canvas_line->set_x0 (d.x);
+               _track_canvas_line->set_x1 (d.x);
+               _track_canvas_line->set_y0 (d.y);
+               _track_canvas_line->set_y1 (ArdourCanvas::COORD_MAX);
+               _track_canvas_line->set_outline_color (_selected ? ARDOUR_UI::config()->get_canvasvar_EditPoint() : _color);
+               _track_canvas_line->raise_to_top ();
+               _track_canvas_line->show ();
 
        } else {
-               if (_line) {
-                       _line->hide ();
+               if (_time_bars_line) {
+                       _time_bars_line->hide ();
+                       _track_canvas_line->hide ();
                }
        }
 }
@@ -353,7 +375,7 @@ Marker::canvas_height_set (double h)
 ArdourCanvas::Item&
 Marker::the_item() const
 {
-       return *mark;
+       return *group;
 }
 
 void
@@ -375,7 +397,7 @@ void
 Marker::setup_name_display ()
 {
        double limit = DBL_MAX;
-       
+
        if (label_on_left ()) {
                limit = _left_label_limit;
        } else {
@@ -383,48 +405,40 @@ Marker::setup_name_display ()
        }
 
        /* Work out how wide the name can be */
-       int name_width = min ((double) pixel_width (_name, *name_font) + 2, limit);
+       int name_width = min ((double) pixel_width (_name, name_font) + 2, limit);
+
        if (name_width == 0) {
                name_width = 1;
        }
 
        if (label_on_left ()) {
-               name_pixbuf->property_x() = -name_width;
+               _name_item->set_x_position (-name_width);
        }
 
-       name_pixbuf->property_pixbuf() = pixbuf_from_string (_name, name_font, name_width, name_height, Gdk::Color ("#000000"));
+       _name_item->set (_name);
+
+       // CAIROCANVAS
+       // need to "clip" name to name_width and name_height
 
        if (label_on_left ()) {
-               _name_background->property_x1() = name_pixbuf->property_x() - 2;
-               _name_background->property_x2() = name_pixbuf->property_x() + name_width + _shift;
+               _name_background->set_x0 (_name_item->position().x - 2);
+               _name_background->set_x1 (_name_item->position().x + name_width + _shift);
        } else {
-               _name_background->property_x1() = name_pixbuf->property_x() - _label_offset + 2;
-               _name_background->property_x2() = name_pixbuf->property_x() + name_width;
+               _name_background->set_x0 (_name_item->position().x - _label_offset + 2);
+               _name_background->set_x1 (_name_item->position().x + name_width);
        }
-       
-       _name_background->property_y1() = 0;
-       _name_background->property_y2() = 13;
-}
 
-void
-Marker::setup_line_x ()
-{
-       if (_line) {
-               _line->property_x1() = unit_position + _shift - 0.5;
-               _line->property_x2() = unit_position + _shift - 0.5;
-       }
+       _name_background->set_y0 (0);
+       _name_background->set_y1 (13);
 }
 
 void
 Marker::set_position (framepos_t frame)
 {
-       double new_unit_position = editor.frame_to_unit (frame);
-       new_unit_position -= _shift;
-       group->move (new_unit_position - unit_position, 0.0);
+       unit_position = editor.sample_to_pixel (frame) - _shift;
+       group->set_x_position (unit_position);
+       setup_line ();
        frame_position = frame;
-       unit_position = new_unit_position;
-
-       setup_line_x ();
 }
 
 void
@@ -437,7 +451,7 @@ void
 Marker::show ()
 {
        _shown = true;
-       
+
         group->show ();
        setup_line ();
 }
@@ -446,7 +460,7 @@ void
 Marker::hide ()
 {
        _shown = false;
-       
+
        group->hide ();
        setup_line ();
 }
@@ -455,22 +469,17 @@ void
 Marker::set_color_rgba (uint32_t c)
 {
        _color = c;
-       mark->property_fill_color_rgba() = _color;
-       mark->property_outline_color_rgba() = _color;
-
-       if (_line && !_selected) {
-               _line->property_color_rgba() = _color;
+       mark->set_fill_color (_color);
+       mark->set_outline_color (_color);
 
-               /* For reasons unknown this is necessary to ensure that the line colour
-                  gets updated.
-               */
-               _line->hide ();
-               _line->show ();
+       if (_time_bars_line && !_selected) {
+               _time_bars_line->set_outline_color (_color);
+               _track_canvas_line->set_outline_color (_color);
        }
 
-       _name_background->property_fill() = true;
-       _name_background->property_fill_color_rgba() = UINT_RGBA_CHANGE_A (_color, 0x70);
-       _name_background->property_outline_color_rgba() = _color;
+       _name_background->set_fill (true);
+       _name_background->set_fill_color (UINT_RGBA_CHANGE_A (_color, 0x70));
+       _name_background->set_outline_color (_color);
 }
 
 /** Set the number of pixels that are available for a label to the left of the centre of this marker */
@@ -482,7 +491,7 @@ Marker::set_left_label_limit (double p)
        if (_left_label_limit < 0) {
                _left_label_limit = 0;
        }
-       
+
        if (label_on_left ()) {
                setup_name_display ();
        }
@@ -497,7 +506,7 @@ Marker::set_right_label_limit (double p)
        if (_right_label_limit < 0) {
                _right_label_limit = 0;
        }
-       
+
        if (!label_on_left ()) {
                setup_name_display ();
        }
@@ -505,13 +514,13 @@ Marker::set_right_label_limit (double p)
 
 /***********************************************************************/
 
-TempoMarker::TempoMarker (PublicEditor& editor, ArdourCanvas::Group& parent, ArdourCanvas::Group& line_parent, guint32 rgba, const string& text,
+TempoMarker::TempoMarker (PublicEditor& editor, ArdourCanvas::Group& parent, guint32 rgba, const string& text,
                          ARDOUR::TempoSection& temp)
-       : Marker (editor, parent, line_parent, rgba, text, Tempo, 0, false),
+       : Marker (editor, parent, rgba, text, Tempo, 0, false),
          _tempo (temp)
 {
        set_position (_tempo.frame());
-       group->signal_event().connect (sigc::bind (sigc::mem_fun (editor, &PublicEditor::canvas_tempo_marker_event), mark, this));
+       group->Event.connect (sigc::bind (sigc::mem_fun (editor, &PublicEditor::canvas_tempo_marker_event), group, this));
 }
 
 TempoMarker::~TempoMarker ()
@@ -520,13 +529,13 @@ TempoMarker::~TempoMarker ()
 
 /***********************************************************************/
 
-MeterMarker::MeterMarker (PublicEditor& editor, ArdourCanvas::Group& parent, ArdourCanvas::Group& line_parent, guint32 rgba, const string& text,
+MeterMarker::MeterMarker (PublicEditor& editor, ArdourCanvas::Group& parent, guint32 rgba, const string& text,
                          ARDOUR::MeterSection& m)
-       : Marker (editor, parent, line_parent, rgba, text, Meter, 0, false),
+       : Marker (editor, parent, rgba, text, Meter, 0, false),
          _meter (m)
 {
        set_position (_meter.frame());
-       group->signal_event().connect (sigc::bind (sigc::mem_fun (editor, &PublicEditor::canvas_meter_marker_event), mark, this));
+       group->Event.connect (sigc::bind (sigc::mem_fun (editor, &PublicEditor::canvas_meter_marker_event), group, this));
 }
 
 MeterMarker::~MeterMarker ()