prepare GUI notifications for plugin scan

[ardour.git] / gtk2_ardour / editor_markers.cc

diff --git a/gtk2_ardour/editor_markers.cc b/gtk2_ardour/editor_markers.cc
index 6dee565287967b0912cbe4ac7bc4237e34bb0993..1302804cec2d31bce22e2203ec61b5318abcb8de 100644 (file)
--- a/gtk2_ardour/editor_markers.cc
+++ b/gtk2_ardour/editor_markers.cc
@@ -20,7 +20,6 @@
 #include <cstdlib>
 #include <cmath>
 
-#include <libgnomecanvas/libgnomecanvas.h>
 #include <gtkmm2ext/gtk_ui.h>
 
 #include "ardour/session.h"
@@ -28,12 +27,15 @@
 #include "ardour/profile.h"
 #include "pbd/memento_command.h"
 
+#include "canvas/canvas.h"
+#include "canvas/item.h"
+#include "canvas/rectangle.h"
+
 #include "editor.h"
 #include "marker.h"
 #include "selection.h"
 #include "editing.h"
 #include "gui_thread.h"
-#include "simplerect.h"
 #include "actions.h"
 #include "prompter.h"
 #include "editor_drag.h"
@@ -66,6 +68,14 @@ Editor::add_new_location (Location *location)
 
        /* Do a full update of the markers in this group */
        update_marker_labels (group);
+       
+       if (location->is_auto_punch()) {
+               update_punch_range_view ();
+       }
+
+       if (location->is_auto_loop()) {
+               update_loop_range_view ();
+       }
 }
 
 /** Add a new location, without a time-consuming update of all marker labels;
@@ -171,7 +181,7 @@ Editor::add_new_location_internal (Location* location)
                select_new_marker = false;
        }
 
-       lam->canvas_height_set (_canvas_height);
+       lam->canvas_height_set (_visible_canvas_height);
        lam->set_show_lines (_show_marker_lines);
 
        /* Add these markers to the appropriate sorted marker lists, which will render
@@ -256,7 +266,7 @@ Editor::check_marker_label (Marker* m)
 
                /* Update just the available space between the previous marker and this one */
 
-               double const p = frame_to_pixel (m->position() - (*prev)->position());
+               double const p = sample_to_pixel (m->position() - (*prev)->position());
 
                if (m->label_on_left()) {
                        (*prev)->set_right_label_limit (p / 2);
@@ -275,7 +285,7 @@ Editor::check_marker_label (Marker* m)
 
                /* Update just the available space between this marker and the next */
 
-               double const p = frame_to_pixel ((*next)->position() - m->position());
+               double const p = sample_to_pixel ((*next)->position() - m->position());
 
                if ((*next)->label_on_left()) {
                        m->set_right_label_limit (p / 2);
@@ -329,7 +339,7 @@ Editor::update_marker_labels (ArdourCanvas::Group* group)
        while (i != sorted.end()) {
 
                if (prev != sorted.end()) {
-                       double const p = frame_to_pixel ((*i)->position() - (*prev)->position());
+                       double const p = sample_to_pixel ((*i)->position() - (*prev)->position());
 
                        if ((*prev)->label_on_left()) {
                                (*i)->set_left_label_limit (p);
@@ -340,7 +350,7 @@ Editor::update_marker_labels (ArdourCanvas::Group* group)
                }
 
                if (next != sorted.end()) {
-                       double const p = frame_to_pixel ((*next)->position() - (*i)->position());
+                       double const p = sample_to_pixel ((*next)->position() - (*i)->position());
 
                        if ((*next)->label_on_left()) {
                                (*i)->set_right_label_limit (p / 2);
@@ -521,8 +531,8 @@ Editor::refresh_location_display_internal (Locations::LocationList& locations)
                i = tmp;
        }
 
-       update_punch_range_view (false);
-       update_loop_range_view (false);
+       update_punch_range_view ();
+       update_loop_range_view ();
 }
 
 void
@@ -671,7 +681,7 @@ Editor::mouse_add_new_range (framepos_t where)
           it's reasonably easy to manipulate after creation.
        */
 
-       framepos_t const end = where + current_page_frames() / 8;
+       framepos_t const end = where + current_page_samples() / 8;
 
        string name;
        _session->locations()->next_available_name (name, _("range"));
@@ -727,11 +737,11 @@ Editor::location_gone (Location *location)
        LocationMarkerMap::iterator i;
 
        if (location == transport_loop_location()) {
-               update_loop_range_view (true);
+               update_loop_range_view ();
        }
 
        if (location == transport_punch_location()) {
-               update_punch_range_view (true);
+               update_punch_range_view ();
        }
 
        for (i = location_markers.begin(); i != location_markers.end(); ++i) {
@@ -1338,12 +1348,23 @@ Editor::marker_menu_rename ()
                /*NOTREACHED*/
        }
 
+
+       rename_marker (marker);
+}
+
+void
+Editor::rename_marker(Marker *marker)
+{
        Location* loc;
        bool is_start;
 
        loc = find_location_from_marker (marker, is_start);
 
-       if (!loc) return;
+       if (!loc)
+              return;
+
+       if (loc == transport_loop_location() || loc == transport_punch_location() || loc->is_session_range())
+               return;
 
        ArdourPrompter dialog (true);
        string txt;
@@ -1378,6 +1399,7 @@ Editor::marker_menu_rename ()
 
        dialog.get_result(txt);
        loc->set_name (txt);
+       _session->set_dirty ();
 
        XMLNode &after = _session->locations()->get_state();
        _session->add_command (new MementoCommand<Locations>(*(_session->locations()), &before, &after));
@@ -1406,7 +1428,7 @@ Editor::new_transport_marker_menu_set_punch ()
 }
 
 void
-Editor::update_loop_range_view (bool visibility)
+Editor::update_loop_range_view ()
 {
        if (_session == 0) {
                return;
@@ -1416,23 +1438,21 @@ Editor::update_loop_range_view (bool visibility)
 
        if (_session->get_play_loop() && ((tll = transport_loop_location()) != 0)) {
 
-               double x1 = frame_to_pixel (tll->start());
-               double x2 = frame_to_pixel (tll->end());
-
-               transport_loop_range_rect->property_x1() = x1;
-               transport_loop_range_rect->property_x2() = x2;
+               double x1 = sample_to_pixel (tll->start());
+               double x2 = sample_to_pixel (tll->end());
 
-               if (visibility) {
-                       transport_loop_range_rect->show();
-               }
+               transport_loop_range_rect->set_x0 (x1);
+               transport_loop_range_rect->set_x1 (x2);
 
-       } else if (visibility) {
+               transport_loop_range_rect->show();
+               
+       } else {
                transport_loop_range_rect->hide();
        }
 }
 
 void
-Editor::update_punch_range_view (bool visibility)
+Editor::update_punch_range_view ()
 {
        if (_session == 0) {
                return;
@@ -1441,20 +1461,20 @@ Editor::update_punch_range_view (bool visibility)
        Location* tpl;
 
        if ((_session->config.get_punch_in() || _session->config.get_punch_out()) && ((tpl = transport_punch_location()) != 0)) {
-               guint track_canvas_width,track_canvas_height;
-               track_canvas->get_size(track_canvas_width,track_canvas_height);
+               ArdourCanvas::Rect const v = _track_canvas->visible_area ();
+
                if (_session->config.get_punch_in()) {
-                       transport_punch_range_rect->property_x1() = frame_to_pixel (tpl->start());
-                       transport_punch_range_rect->property_x2() = (_session->config.get_punch_out() ? frame_to_pixel (tpl->end()) : frame_to_pixel (JACK_MAX_FRAMES));
+                       transport_punch_range_rect->set_x0 (sample_to_pixel (tpl->start()));
+                       transport_punch_range_rect->set_x1 (_session->config.get_punch_out() ? sample_to_pixel (tpl->end()) : sample_to_pixel (JACK_MAX_FRAMES));
                } else {
-                       transport_punch_range_rect->property_x1() = 0;
-                       transport_punch_range_rect->property_x2() = (_session->config.get_punch_out() ? frame_to_pixel (tpl->end()) : track_canvas_width);
+                       transport_punch_range_rect->set_x0 (0);
+                       transport_punch_range_rect->set_x1 (_session->config.get_punch_out() ? sample_to_pixel (tpl->end()) : v.width ());
                }
+               
+               transport_punch_range_rect->show();
+
+       } else {
 
-               if (visibility) {
-                       transport_punch_range_rect->show();
-               }
-       } else if (visibility) {
                transport_punch_range_rect->hide();
        }
 }