Some HIG and wording improvements to the right-click region context menu. Closes...
[ardour.git] / gtk2_ardour / marker.cc
index 392ba851526355ef604c3705f093a516787b003a..f4848228f8286ac13e89263ffea60d2e92069fbd 100644 (file)
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2001 Paul Davis 
+    Copyright (C) 2001 Paul Davis
 
     This program is free software; you can redistribute it and/or modify
     it under the terms of the GNU General Public License as published by
 #include <sigc++/bind.h>
 #include "ardour/tempo.h"
 
+#include "ardour_ui.h"
+/*
+ * ardour_ui.h include was moved to the top of the list
+ * due to a conflicting definition of 'Rect' between
+ * Apple's MacTypes.h and GTK.
+ */
+
 #include "marker.h"
 #include "public_editor.h"
 #include "utils.h"
 #include "canvas_impl.h"
-#include "ardour_ui.h"
 #include "simpleline.h"
 
 #include <gtkmm2ext/utils.h>
@@ -34,7 +40,9 @@
 using namespace std;
 using namespace ARDOUR;
 
-Marker::Marker (PublicEditor& ed, ArdourCanvas::Group& parent, guint32 rgba, const string& annotation, 
+PBD::Signal1<void,Marker*> Marker::CatchDeletion;
+
+Marker::Marker (PublicEditor& ed, ArdourCanvas::Group& parent, guint32 rgba, const string& annotation,
                Type type, nframes_t frame, bool handle_events)
 
        : editor (ed), _parent(&parent), _type(type)
@@ -49,7 +57,7 @@ Marker::Marker (PublicEditor& ed, ArdourCanvas::Group& parent, guint32 rgba, con
             ^        |
             |        V
            (0,5)    (6,5)
-              \    / 
+              \    /
                (3,10)
 
 
@@ -66,17 +74,17 @@ Marker::Marker (PublicEditor& ed, ArdourCanvas::Group& parent, guint32 rgba, con
 
            Start:
 
-          0,0\ 
-                   |  \        
+          0,0\
+                   |  \
             |   \ 6,6
            |   /
             |  /
-           0,12 
+           0,12
 
           End:
 
               /12,0
-             /     | 
+             /     |
              /      |
           6,6      |
              \      |
@@ -84,15 +92,15 @@ Marker::Marker (PublicEditor& ed, ArdourCanvas::Group& parent, guint32 rgba, con
                \    |
               12,12
 
-            
+
           TransportStart:
 
             0,0
-             | \ 
-             |  \ 
-             |   \ 
+             | \
+             |  \
+             |   \
              |    \
-             |     \  
+             |     \
             0,13 --- 13,13
 
           TransportEnd:
@@ -104,7 +112,7 @@ Marker::Marker (PublicEditor& ed, ArdourCanvas::Group& parent, guint32 rgba, con
                /      |
               /       |
             0,13 ------ 13,13
-            
+
 
             PunchIn:
 
@@ -112,23 +120,23 @@ Marker::Marker (PublicEditor& ed, ArdourCanvas::Group& parent, guint32 rgba, con
              |       /
              |      /
              |     /
-             |    / 
-             |   / 
-             |  / 
+             |    /
+             |   /
+             |  /
             0,13
 
             PunchOut
 
           0,0 -->-13,0
-           \       | 
+           \       |
             \      |
              \     |
               \    |
                \   |
                 \  |
                 13,13
-            
-          
+
+
        */
 
        switch (type) {
@@ -138,10 +146,10 @@ Marker::Marker (PublicEditor& ed, ArdourCanvas::Group& parent, guint32 rgba, con
                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, 10.0));              
-               points->push_back (Gnome::Art::Point (0.0, 5.0));               
-               points->push_back (Gnome::Art::Point (0.0, 0.0));               
-               
+               points->push_back (Gnome::Art::Point (3.0, 10.0));
+               points->push_back (Gnome::Art::Point (0.0, 5.0));
+               points->push_back (Gnome::Art::Point (0.0, 0.0));
+
                shift = 3;
                label_offset = 8.0;
                break;
@@ -151,11 +159,11 @@ Marker::Marker (PublicEditor& ed, ArdourCanvas::Group& parent, guint32 rgba, con
 
                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 (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));
 
                shift = 3;
                label_offset = 8.0;
@@ -163,10 +171,10 @@ Marker::Marker (PublicEditor& ed, ArdourCanvas::Group& parent, guint32 rgba, con
 
        case Start:
                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 (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));
 
                shift = 0;
                label_offset = 13.0;
@@ -175,21 +183,21 @@ Marker::Marker (PublicEditor& ed, ArdourCanvas::Group& parent, guint32 rgba, con
        case End:
                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 (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));
+
                shift = 13;
                label_offset = 6.0;
                break;
 
        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 (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));
+
                shift = 0;
                label_offset = 12.0;
                break;
@@ -197,10 +205,10 @@ Marker::Marker (PublicEditor& ed, ArdourCanvas::Group& parent, guint32 rgba, con
        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, 13.0));
+               points->push_back (Gnome::Art::Point (0.0, 13.0));
                points->push_back (Gnome::Art::Point (13.0, 0.0));
-               
+
                shift = 13;
                label_offset = 0.0;
                break;
@@ -208,25 +216,25 @@ Marker::Marker (PublicEditor& ed, ArdourCanvas::Group& parent, guint32 rgba, con
        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 (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));
 
                shift = 0;
                label_offset = 13.0;
                break;
-               
+
        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 (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));
 
                shift = 13;
                label_offset = 0.0;
                break;
-               
+
        }
 
        frame_position = frame;
@@ -246,30 +254,30 @@ Marker::Marker (PublicEditor& ed, ArdourCanvas::Group& parent, guint32 rgba, con
 
        /* setup name pixbuf sizes */
        name_font = get_font_for_style (N_("MarkerText"));
-       
+
        Gtk::Window win;
        Gtk::Label foo;
        win.add (foo);
-       
+
        Glib::RefPtr<Pango::Layout> layout = foo.create_pango_layout (X_("Hg")); /* ascender + descender */
        int width;
        int height;
-       
+
        layout->set_font_description (*name_font);
        Gtkmm2ext::get_ink_pixel_size (layout, width, height);
        name_height = height + 6;
-       
+
        name_pixbuf = new ArdourCanvas::Pixbuf(*group);
        name_pixbuf->property_x() = label_offset;
-       
+
        set_name (annotation.c_str());
-       
-       editor.ZoomChanged.connect (mem_fun (*this, &Marker::reposition));
+
+       editor.ZoomChanged.connect (sigc::mem_fun (*this, &Marker::reposition));
 
        mark->set_data ("marker", this);
 
        if (handle_events) {
-               group->signal_event().connect (bind (mem_fun (editor, &PublicEditor::canvas_marker_event), mark, this));
+               group->signal_event().connect (sigc::bind (sigc::mem_fun (editor, &PublicEditor::canvas_marker_event), mark, this));
        }
 
        line = 0;
@@ -279,7 +287,7 @@ Marker::Marker (PublicEditor& ed, ArdourCanvas::Group& parent, guint32 rgba, con
 
 Marker::~Marker ()
 {
-       drop_references ();
+       CatchDeletion (this); /* EMIT SIGNAL */
 
        /* destroying the parent group destroys its contents, namely any polygons etc. that we added */
        delete name_pixbuf;
@@ -318,7 +326,7 @@ Marker::add_line (ArdourCanvas::Group* group, double y_origin, double initial_he
                line->property_x2() = unit_position + shift;
                line->property_y2() = y_origin + initial_height;
 
-               line->signal_event().connect (bind (mem_fun (editor, &PublicEditor::canvas_marker_event), mark, this));
+               line->signal_event().connect (sigc::bind (sigc::mem_fun (editor, &PublicEditor::canvas_marker_event), mark, this));
        }
 
        show_line ();
@@ -333,7 +341,7 @@ Marker::show_line ()
        }
 }
 
-void 
+void
 Marker::hide_line ()
 {
        if (line) {
@@ -350,41 +358,17 @@ Marker::the_item() const
 void
 Marker::set_name (const string& new_name)
 {
-       uint32_t pb_width;
-       double font_size;
-       
-       font_size = name_font->get_size() / Pango::SCALE;
-       pb_width = new_name.length() * font_size;
-       
-       Glib::RefPtr<Gdk::Pixbuf> buf = Gdk::Pixbuf::create(Gdk::COLORSPACE_RGB, true, 8, pb_width, name_height);
-       
-       cairo_surface_t* surface = cairo_image_surface_create(CAIRO_FORMAT_ARGB32, pb_width, name_height);
-       cairo_t *cr = cairo_create (surface);
-       cairo_text_extents_t te;
-       cairo_set_source_rgba (cr, 0.0, 0.0, 0.0, 1.0);
-       cairo_select_font_face (cr, name_font->get_family().c_str(),
-                               CAIRO_FONT_SLANT_NORMAL, CAIRO_FONT_WEIGHT_NORMAL);
-       cairo_set_font_size (cr, font_size);
-       cairo_text_extents (cr, new_name.c_str(), &te);
-       
-       cairo_move_to (cr, 0.5,
-                      0.5 - te.height / 2 - te.y_bearing + name_height / 2);
-       cairo_show_text (cr, new_name.c_str());
-       
-       unsigned char* src = cairo_image_surface_get_data (surface);
-       convert_bgra_to_rgba(src, buf->get_pixels(), pb_width, name_height);
-       
-       cairo_destroy(cr);
-       name_pixbuf->property_pixbuf() = buf;
-       
+       int name_width = pixel_width (new_name, *name_font) + 2;
+
+       name_pixbuf->property_pixbuf() = pixbuf_from_ustring(new_name, name_font, name_width, name_height, Gdk::Color ("#000000"));
+
        if (_type == End || _type == LoopEnd || _type == PunchOut) {
-               name_pixbuf->property_x() = -(te.width);
-        }
-       
+               name_pixbuf->property_x() = - (name_width);
+       }
 }
 
 void
-Marker::set_position (nframes_t frame)
+Marker::set_position (nframes64_t frame)
 {
        double new_unit_position = editor.frame_to_unit (frame);
        new_unit_position -= shift;
@@ -402,7 +386,7 @@ void
 Marker::reposition ()
 {
        set_position (frame_position);
-}      
+}
 
 void
 Marker::show ()
@@ -425,13 +409,13 @@ Marker::set_color_rgba (uint32_t color)
 
 /***********************************************************************/
 
-TempoMarker::TempoMarker (PublicEditor& editor, ArdourCanvas::Group& parent, guint32 rgba, const string& text, 
+TempoMarker::TempoMarker (PublicEditor& editor, ArdourCanvas::Group& parent, guint32 rgba, const string& text,
                          ARDOUR::TempoSection& temp)
        : Marker (editor, parent, rgba, text, Tempo, 0, false),
          _tempo (temp)
 {
        set_position (_tempo.frame());
-       group->signal_event().connect (bind (mem_fun (editor, &PublicEditor::canvas_tempo_marker_event), mark, this));
+       group->signal_event().connect (sigc::bind (sigc::mem_fun (editor, &PublicEditor::canvas_tempo_marker_event), mark, this));
 }
 
 TempoMarker::~TempoMarker ()
@@ -440,13 +424,13 @@ TempoMarker::~TempoMarker ()
 
 /***********************************************************************/
 
-MeterMarker::MeterMarker (PublicEditor& editor, ArdourCanvas::Group& parent, guint32 rgba, const string& text, 
-                         ARDOUR::MeterSection& m) 
+MeterMarker::MeterMarker (PublicEditor& editor, ArdourCanvas::Group& parent, guint32 rgba, const string& text,
+                         ARDOUR::MeterSection& m)
        : Marker (editor, parent, rgba, text, Meter, 0, false),
          _meter (m)
 {
        set_position (_meter.frame());
-       group->signal_event().connect (bind (mem_fun (editor, &PublicEditor::canvas_meter_marker_event), mark, this));
+       group->signal_event().connect (sigc::bind (sigc::mem_fun (editor, &PublicEditor::canvas_meter_marker_event), mark, this));
 }
 
 MeterMarker::~MeterMarker ()