Fix MIDI recording display when zoom/etc changes.
[ardour.git] / gtk2_ardour / region_view.cc
index c566d54f2d16de84bd65f36291c1872434bf6e08..735e5dd09c36d81779c5a3803c7ad3886b309090 100644 (file)
@@ -18,7 +18,6 @@
 */
 
 #include <cmath>
-#include <cassert>
 #include <algorithm>
 
 #include <gtkmm.h>
@@ -33,6 +32,8 @@
 #include "canvas/pixbuf.h"
 #include "canvas/text.h"
 #include "canvas/line.h"
+#include "canvas/utils.h"
+#include "canvas/colors.h"
 
 #include "ardour_ui.h"
 #include "global_signals.h"
@@ -53,6 +54,7 @@
 
 using namespace std;
 using namespace ARDOUR;
+using namespace ARDOUR_UI_UTILS;
 using namespace PBD;
 using namespace Editing;
 using namespace Gtk;
@@ -62,12 +64,12 @@ static const int32_t sync_mark_width = 9;
 
 PBD::Signal1<void,RegionView*> RegionView::RegionViewGoingAway;
 
-RegionView::RegionView (ArdourCanvas::Group*              parent,
+RegionView::RegionView (ArdourCanvas::Container*          parent,
                         TimeAxisView&                     tv,
                         boost::shared_ptr<ARDOUR::Region> r,
                         double                            spu,
-                        Gdk::Color const &                basic_color,
-                       bool                              automation)
+                        uint32_t                          basic_color,
+                        bool                              automation)
        : TimeAxisViewItem (r->name(), *parent, tv, spu, basic_color, r->position(), r->length(), false, automation,
                            (automation ? TimeAxisViewItem::ShowFrame :
                             TimeAxisViewItem::Visibility (TimeAxisViewItem::ShowNameText|
@@ -82,9 +84,7 @@ RegionView::RegionView (ArdourCanvas::Group*              parent,
        , _pixel_width(1.0)
        , in_destructor(false)
        , wait_for_data(false)
-        , _silence_text (0)
-       , _region_relative_time_converter(r->session().tempo_map(), r->position())
-       , _source_relative_time_converter(r->session().tempo_map(), r->position() - r->start())
+       , _silence_text (0)
 {
        GhostRegion::CatchDeletion.connect (*this, invalidator (*this), boost::bind (&RegionView::remove_ghost, this, _1), gui_context());
 }
@@ -92,9 +92,7 @@ RegionView::RegionView (ArdourCanvas::Group*              parent,
 RegionView::RegionView (const RegionView& other)
        : sigc::trackable(other)
        , TimeAxisViewItem (other)
-        , _silence_text (0)
-       , _region_relative_time_converter(other.region_relative_time_converter())
-       , _source_relative_time_converter(other.source_relative_time_converter())
+       , _silence_text (0)
 {
        /* derived concrete type will call init () */
 
@@ -109,9 +107,7 @@ RegionView::RegionView (const RegionView& other)
 RegionView::RegionView (const RegionView& other, boost::shared_ptr<Region> other_region)
        : sigc::trackable(other)
        , TimeAxisViewItem (other)
-        , _silence_text (0)
-       , _region_relative_time_converter(other_region->session().tempo_map(), other_region->position())
-       , _source_relative_time_converter(other_region->session().tempo_map(), other_region->position() - other_region->start())
+       , _silence_text (0)
 {
        /* this is a pseudo-copy constructor used when dragging regions
           around on the canvas.
@@ -127,13 +123,13 @@ RegionView::RegionView (const RegionView& other, boost::shared_ptr<Region> other
        GhostRegion::CatchDeletion.connect (*this, invalidator (*this), boost::bind (&RegionView::remove_ghost, this, _1), gui_context());
 }
 
-RegionView::RegionView (ArdourCanvas::Group*         parent,
-                        TimeAxisView&                tv,
+RegionView::RegionView (ArdourCanvas::Container*          parent,
+                        TimeAxisView&                     tv,
                         boost::shared_ptr<ARDOUR::Region> r,
-                        double                       spu,
-                        Gdk::Color const &           basic_color,
-                       bool                         recording,
-                        TimeAxisViewItem::Visibility visibility)
+                        double                            spu,
+                        uint32_t                          basic_color,
+                        bool                              recording,
+                        TimeAxisViewItem::Visibility      visibility)
        : TimeAxisViewItem (r->name(), *parent, tv, spu, basic_color, r->position(), r->length(), recording, false, visibility)
        , _region (r)
        , sync_mark(0)
@@ -145,14 +141,12 @@ RegionView::RegionView (ArdourCanvas::Group*         parent,
        , _pixel_width(1.0)
        , in_destructor(false)
        , wait_for_data(false)
-        , _silence_text (0)
-       , _region_relative_time_converter(r->session().tempo_map(), r->position())
-       , _source_relative_time_converter(r->session().tempo_map(), r->position() - r->start())
+       , _silence_text (0)
 {
 }
 
 void
-RegionView::init (Gdk::Color const & basic_color, bool wfd)
+RegionView::init (bool wfd)
 {
        editor        = 0;
        valid         = true;
@@ -163,8 +157,6 @@ RegionView::init (Gdk::Color const & basic_color, bool wfd)
        sync_mark     = 0;
        sync_line     = 0;
 
-       compute_colors (basic_color);
-
        if (name_highlight) {
                name_highlight->set_data ("regionview", this);
                name_highlight->Event.connect (sigc::bind (sigc::mem_fun (PublicEditor::instance(), &PublicEditor::canvas_region_view_name_highlight_event), name_highlight, this));
@@ -224,7 +216,10 @@ RegionView::~RegionView ()
 bool
 RegionView::canvas_group_event (GdkEvent* event)
 {
-       return trackview.editor().canvas_region_view_event (event, group, this);
+       if (!in_destructor) {
+               return trackview.editor().canvas_region_view_event (event, group, this);
+       }
+       return false;
 }
 
 void
@@ -239,11 +234,11 @@ RegionView::set_silent_frames (const AudioIntervalResult& silences, double /*thr
                 return;
         }
 
-        uint32_t const color = ARDOUR_UI::config()->get_canvasvar_Silence();
+        uint32_t const color = ARDOUR_UI::config()->color_mod ("silence", "silence");
 
        for (AudioIntervalResult::const_iterator i = silences.begin(); i != silences.end(); ++i) {
 
-               ArdourCanvas::Rectangle* cr = new ArdourCanvas::Rectangle (group);
+               ArdourCanvas::Rectangle* cr = new ArdourCanvas::TimeRectangle (group);
                cr->set_ignore_events (true);
                _silent_frames.push_back (cr);
 
@@ -280,7 +275,7 @@ RegionView::set_silent_frames (const AudioIntervalResult& silences, double /*thr
         _silence_text = new ArdourCanvas::Text (group);
        _silence_text->set_ignore_events (true);
         _silence_text->set_font_description (get_font_for_style (N_("SilenceText")));
-        _silence_text->set_color (ARDOUR_UI::config()->get_canvasvar_SilenceText());
+        _silence_text->set_color (ARDOUR_UI::config()->color ("silence text"));
 
         /* both positions are relative to the region start offset in source */
 
@@ -413,11 +408,6 @@ RegionView::region_resized (const PropertyChange& what_changed)
 
        if (what_changed.contains (ARDOUR::Properties::position)) {
                set_position (_region->position(), 0);
-               _region_relative_time_converter.set_origin_b (_region->position());
-       }
-
-       if (what_changed.contains (ARDOUR::Properties::start) || what_changed.contains (ARDOUR::Properties::position)) {
-               _source_relative_time_converter.set_origin_b (_region->position() - _region->start());
        }
 
        PropertyChange s_and_l;
@@ -524,40 +514,33 @@ void
 RegionView::set_colors ()
 {
        TimeAxisViewItem::set_colors ();
-
-       if (sync_mark) {
-               /* XXX: make these colours themable */
-               sync_mark->set_fill_color (RGBA_TO_UINT (0, 255, 0, 255));
-               sync_line->set_outline_color (RGBA_TO_UINT (0, 255, 0, 255));
-       }
+       set_sync_mark_color ();
 }
 
 void
-RegionView::set_frame_color ()
+RegionView::set_sync_mark_color ()
 {
-       if (!frame) {
-               return;
+       if (sync_mark) {
+               ArdourCanvas::Color c = ARDOUR_UI::config()->color ("sync mark");
+               sync_mark->set_fill_color (c);
+               sync_mark->set_outline_color (c);
+               sync_line->set_outline_color (c);
        }
+}
 
+uint32_t
+RegionView::get_fill_color () const
+{
+       ArdourCanvas::Color f = TimeAxisViewItem::get_fill_color();
+       char const *modname;
+       
        if (_region->opaque()) {
-               fill_opacity = 130;
+               modname = "opaque region base";
        } else {
-               fill_opacity = 60;
+               modname = "transparent region base";
        }
 
-       TimeAxisViewItem::set_frame_color ();
-}
-
-void
-RegionView::fake_set_opaque (bool yn)
-{
-       if (yn) {
-               fill_opacity = 130;
-       } else {
-               fill_opacity = 60;
-       }
-
-       set_frame_color ();
+       return HSV(f).mod (ARDOUR_UI::config()->modifier (modname)).color ();
 }
 
 void
@@ -642,11 +625,10 @@ RegionView::region_sync_changed ()
 
                sync_mark = new ArdourCanvas::Polygon (group);
                CANVAS_DEBUG_NAME (sync_mark, string_compose ("sync mark for %1", get_item_name()));
-               sync_mark->set_fill_color (RGBA_TO_UINT(0,255,0,255));    // FIXME make a themeable colour
-
                sync_line = new ArdourCanvas::Line (group);
                CANVAS_DEBUG_NAME (sync_line, string_compose ("sync mark for %1", get_item_name()));
-               sync_line->set_outline_color (RGBA_TO_UINT(0,255,0,255)); // FIXME make a themeable colour
+
+               set_sync_mark_color ();
        }
 
        /* this has to handle both a genuine change of position, a change of samples_per_pixel
@@ -802,7 +784,7 @@ RegionView::update_coverage_frames (LayerDisplay d)
        bool me = false;
 
        /* the color that will be used to show parts of regions that will not be heard */
-       uint32_t const non_playing_color = ARDOUR_UI::config()->get_canvasvar_CoveredRegion ();
+       uint32_t const non_playing_color = ARDOUR_UI::config()->color_mod ("covered region", "covered region base");
 
        while (t < end) {
 
@@ -818,7 +800,7 @@ RegionView::update_coverage_frames (LayerDisplay d)
 
                /* start off any new rect, if required */
                if (cr == 0 || me != new_me) {
-                       cr = new ArdourCanvas::Rectangle (group);
+                       cr = new ArdourCanvas::TimeRectangle (group);
                        _coverage_frames.push_back (cr);
                        cr->set_x0 (trackview.editor().sample_to_pixel (t - position));
                        cr->set_y0 (1);
@@ -969,12 +951,12 @@ RegionView::snap_frame_to_frame (frameoffset_t x) const
 
        /* try a snap in either direction */
        framepos_t frame = session_frame;
-       editor.snap_to (frame, 0);
+       editor.snap_to (frame, RoundNearest);
 
        /* if we went off the beginning of the region, snap forwards */
        if (frame < _region->position ()) {
                frame = session_frame;
-               editor.snap_to (frame, 1);
+               editor.snap_to (frame, RoundUpAlways);
        }
 
        /* back to region relative */