Refactor so that all the locate-related stuff in CursorDrag is in one method. Also...
[ardour.git] / gtk2_ardour / editor_markers.cc
index 0eab51260fd4f3c880479fd2504c1db19eab9366..78c1edb8b0b4428dcd7c9d0c2ab06aceebe2eac5 100644 (file)
@@ -55,6 +55,7 @@ Editor::clear_marker_display ()
        }
 
        location_markers.clear ();
+       _sorted_marker_lists.clear ();
 }
 
 void
@@ -65,6 +66,9 @@ Editor::add_new_location (Location *location)
        LocationMarkers *lam = new LocationMarkers;
        uint32_t color;
 
+       /* make a note here of which group this marker ends up in */
+       ArdourCanvas::Group* group = 0;
+
        if (location->is_cd_marker()) {
                color = location_cd_marker_color;
        } else if (location->is_mark()) {
@@ -80,45 +84,54 @@ Editor::add_new_location (Location *location)
        if (location->is_mark()) {
 
                if (location->is_cd_marker() && ruler_cd_marker_action->get_active()) {
-                       lam->start = new Marker (*this, *cd_marker_group, color, location->name(), Marker::Mark, location->start());
-               }
-               else {
-                       lam->start = new Marker (*this, *marker_group, color, location->name(), Marker::Mark, location->start());
+                       lam->start = new Marker (*this, *cd_marker_group, *cursor_group, color, location->name(), Marker::Mark, location->start());
+                       group = cd_marker_group;
+               } else {
+                       lam->start = new Marker (*this, *marker_group, *cursor_group, color, location->name(), Marker::Mark, location->start());
+                       group = marker_group;
                }
-               lam->end   = 0;
+               
+               lam->end = 0;
 
        } else if (location->is_auto_loop()) {
+               
                // transport marker
-               lam->start = new Marker (*this, *transport_marker_group, color,
+               lam->start = new Marker (*this, *transport_marker_group, *cursor_group, color,
                                         location->name(), Marker::LoopStart, location->start());
-               lam->end   = new Marker (*this, *transport_marker_group, color,
+               lam->end   = new Marker (*this, *transport_marker_group, *cursor_group, color,
                                         location->name(), Marker::LoopEnd, location->end());
+               group = transport_marker_group;
 
        } else if (location->is_auto_punch()) {
+               
                // transport marker
-               lam->start = new Marker (*this, *transport_marker_group, color,
+               lam->start = new Marker (*this, *transport_marker_group, *cursor_group, color,
                                         location->name(), Marker::PunchIn, location->start());
-               lam->end   = new Marker (*this, *transport_marker_group, color,
+               lam->end   = new Marker (*this, *transport_marker_group, *cursor_group, color,
                                         location->name(), Marker::PunchOut, location->end());
-
+               group = transport_marker_group;
+               
        } else if (location->is_session_range()) {
+
                // session range
-               lam->start = new Marker (*this, *marker_group, color, _("start"), Marker::Start, location->start());
-               lam->end = new Marker (*this, *marker_group, color, _("end"), Marker::End, location->end());
+               lam->start = new Marker (*this, *marker_group, *cursor_group, color, _("start"), Marker::SessionStart, location->start());
+               lam->end = new Marker (*this, *marker_group, *cursor_group, color, _("end"), Marker::SessionEnd, location->end());
+               group = marker_group;
                
        } else {
                // range marker
                if (location->is_cd_marker() && ruler_cd_marker_action->get_active()) {
-                       lam->start = new Marker (*this, *cd_marker_group, color,
-                                                location->name(), Marker::Start, location->start());
-                       lam->end   = new Marker (*this, *cd_marker_group, color,
-                                                location->name(), Marker::End, location->end());
-               }
-               else {
-                       lam->start = new Marker (*this, *range_marker_group, color,
-                                                location->name(), Marker::Start, location->start());
-                       lam->end   = new Marker (*this, *range_marker_group, color,
-                                                location->name(), Marker::End, location->end());
+                       lam->start = new Marker (*this, *cd_marker_group, *cursor_group, color,
+                                                location->name(), Marker::RangeStart, location->start());
+                       lam->end   = new Marker (*this, *cd_marker_group, *cursor_group, color,
+                                                location->name(), Marker::RangeEnd, location->end());
+                       group = cd_marker_group;
+               } else {
+                       lam->start = new Marker (*this, *range_marker_group, *cursor_group, color,
+                                                location->name(), Marker::RangeStart, location->start());
+                       lam->end   = new Marker (*this, *range_marker_group, *cursor_group, color,
+                                                location->name(), Marker::RangeEnd, location->end());
+                       group = range_marker_group;
                }
        }
 
@@ -146,11 +159,19 @@ Editor::add_new_location (Location *location)
                select_new_marker = false;
        }
 
-       if (_show_marker_lines) {
-               lam->show_lines (cursor_group, _canvas_height);
-       } else {
-               lam->hide_lines ();
+       lam->canvas_height_set (_canvas_height);
+       lam->set_show_lines (_show_marker_lines);
+
+       /* Add these markers to the appropriate sorted marker lists, which will render
+          them unsorted until the update_marker_labels() below sorts them out.
+       */
+       _sorted_marker_lists[group].push_back (lam->start);
+       if (lam->end) {
+               _sorted_marker_lists[group].push_back (lam->end);
        }
+
+       /* Do a full update of the markers in this group */
+       update_marker_labels (group);
 }
 
 void
@@ -173,6 +194,154 @@ Editor::location_changed (Location *location)
        } else if (location->is_auto_punch()) {
                update_punch_range_view ();
        }
+
+       check_marker_label (lam->start);
+       if (lam->end) {
+               check_marker_label (lam->end);
+       }
+}
+
+/** Look at a marker and check whether its label, and those of the previous and next markers,
+ *  need to have their labels updated (in case those labels need to be shortened or can be
+ *  lengthened)
+ */
+void
+Editor::check_marker_label (Marker* m)
+{
+       /* Get a time-ordered list of markers from the last time anything changed */
+       std::list<Marker*>& sorted = _sorted_marker_lists[m->get_parent()];
+       
+       list<Marker*>::iterator i = find (sorted.begin(), sorted.end(), m);
+
+       list<Marker*>::iterator prev = sorted.end ();
+       list<Marker*>::iterator next = i;
+       ++next;
+
+       /* Look to see if the previous marker is still behind `m' in time */
+       if (i != sorted.begin()) {
+
+               prev = i;
+               --prev;
+
+               if ((*prev)->position() > m->position()) {
+                       /* This marker is no longer in the correct order with the previous one, so
+                        * update all the markers in this group.
+                        */
+                       update_marker_labels (m->get_parent ());
+                       return;
+               }
+       }
+
+       /* Look to see if the next marker is still ahead of `m' in time */
+       if (next != sorted.end() && (*next)->position() < m->position()) {
+               /* This marker is no longer in the correct order with the next one, so
+                * update all the markers in this group.
+                */
+               update_marker_labels (m->get_parent ());
+               return;
+       }
+
+       if (prev != sorted.end()) {
+
+               /* Update just the available space between the previous marker and this one */
+               
+               double const p = frame_to_pixel (m->position() - (*prev)->position());
+
+               if (m->label_on_left()) {
+                       (*prev)->set_right_label_limit (p / 2);
+               } else {
+                       (*prev)->set_right_label_limit (p);
+               }
+               
+               if ((*prev)->label_on_left ()) {
+                       m->set_left_label_limit (p);
+               } else {
+                       m->set_left_label_limit (p / 2);
+               }
+       }
+
+       if (next != sorted.end()) {
+
+               /* Update just the available space between this marker and the next */
+               
+               double const p = frame_to_pixel ((*next)->position() - m->position());
+
+               if ((*next)->label_on_left()) {
+                       m->set_right_label_limit (p / 2);
+               } else {
+                       m->set_right_label_limit (p);
+               }
+
+               if (m->label_on_left()) {
+                       (*next)->set_left_label_limit (p);
+               } else {
+                       (*next)->set_left_label_limit (p / 2);
+               }
+       }
+}
+
+struct MarkerComparator {
+       bool operator() (Marker const * a, Marker const * b) {
+               return a->position() < b->position();
+       }
+};
+
+/** Update all marker labels in all groups */
+void
+Editor::update_marker_labels ()
+{
+       for (std::map<ArdourCanvas::Group *, std::list<Marker *> >::iterator i = _sorted_marker_lists.begin(); i != _sorted_marker_lists.end(); ++i) {
+               update_marker_labels (i->first);
+       }
+}
+
+/** Look at all markers in a group and update label widths */
+void
+Editor::update_marker_labels (ArdourCanvas::Group* group)
+{
+       list<Marker*>& sorted = _sorted_marker_lists[group];
+
+       if (sorted.empty()) {
+               return;
+       }
+
+       /* We sort the list of markers and then set up the space available between each one */
+       
+       sorted.sort (MarkerComparator ());
+
+       list<Marker*>::iterator i = sorted.begin ();
+
+       list<Marker*>::iterator prev = sorted.end ();
+       list<Marker*>::iterator next = i;
+       ++next;
+       
+       while (i != sorted.end()) {
+
+               if (prev != sorted.end()) {
+                       double const p = frame_to_pixel ((*i)->position() - (*prev)->position());
+                       
+                       if ((*prev)->label_on_left()) {
+                               (*i)->set_left_label_limit (p);
+                       } else {
+                               (*i)->set_left_label_limit (p / 2);
+                       }
+                               
+               }
+
+               if (next != sorted.end()) {
+                       double const p = frame_to_pixel ((*next)->position() - (*i)->position());
+
+                       if ((*next)->label_on_left()) {
+                               (*i)->set_right_label_limit (p / 2);
+                       } else {
+                               (*i)->set_right_label_limit (p);
+                       }
+               }
+
+               prev = i;
+               ++i;
+               ++next;
+       }
 }
 
 void
@@ -327,6 +496,12 @@ Editor::refresh_location_display_internal (Locations::LocationList& locations)
                ++tmp;
 
                if (!i->second->valid) {
+
+                       remove_sorted_marker (i->second->start);
+                       if (i->second->end) {
+                               remove_sorted_marker (i->second->end);
+                       }
+                       
                        delete i->second;
                        location_markers.erase (i);
                }
@@ -377,62 +552,72 @@ Editor::LocationMarkers::show()
 }
 
 void
-Editor::LocationMarkers::show_lines (ArdourCanvas::Group* g, double h)
+Editor::LocationMarkers::canvas_height_set (double h)
 {
-       /* add_line may be required, and it calls show_line even if it isn't */
-       
-       start->add_line (g, 0, h);
-       
+       start->canvas_height_set (h);
        if (end) {
-               end->add_line (g, 0, h);
+               end->canvas_height_set (h);
        }
 }
 
 void
-Editor::LocationMarkers::hide_lines ()
+Editor::LocationMarkers::set_name (const string& str)
 {
-       start->hide_line ();
-       if (end) {
-               end->hide_line ();
+       /* XXX: hack: don't change names of session start/end markers */
+
+       if (start->type() != Marker::SessionStart) {
+               start->set_name (str);
+       }
+       
+       if (end && end->type() != Marker::SessionEnd) {
+               end->set_name (str);
        }
 }
 
 void
-Editor::LocationMarkers::set_lines_vpos (double y, double h)
+Editor::LocationMarkers::set_position (framepos_t startf,
+                                      framepos_t endf)
 {
-       start->set_line_vpos (y, h);
+       start->set_position (startf);
        if (end) {
-               end->set_line_vpos (y, h);
+               end->set_position (endf);
        }
 }
 
 void
-Editor::LocationMarkers::set_name (const string& str)
+Editor::LocationMarkers::set_color_rgba (uint32_t rgba)
 {
-       /* XXX: hack: don't change names of session start/end markers */
-       
-       if (start->type() != Marker::Start) {
-               start->set_name (str);
+       start->set_color_rgba (rgba);
+       if (end) {
+               end->set_color_rgba (rgba);
        }
-       
-       if (end && end->type() != Marker::End) {
-               end->set_name (str);
+}
+
+void
+Editor::LocationMarkers::set_show_lines (bool s)
+{
+       start->set_show_line (s);
+       if (end) {
+               end->set_show_line (s);
        }
 }
 
 void
-Editor::LocationMarkers::set_position (framepos_t startf,
-                                      framepos_t endf)
+Editor::LocationMarkers::set_selected (bool s)
 {
-       start->set_position (startf);
-       if (end) { end->set_position (endf); }
+       start->set_selected (s);
+       if (end) {
+               end->set_selected (s);
+       }
 }
 
 void
-Editor::LocationMarkers::set_color_rgba (uint32_t rgba)
+Editor::LocationMarkers::setup_lines ()
 {
-       start->set_color_rgba (rgba);
-       if (end) { end->set_color_rgba (rgba); }
+       start->setup_line ();
+       if (end) {
+               end->setup_line ();
+       }
 }
 
 void
@@ -521,8 +706,15 @@ Editor::location_gone (Location *location)
        }
 
        for (i = location_markers.begin(); i != location_markers.end(); ++i) {
-               if ((*i).first == location) {
-                       delete (*i).second;
+               if (i->first == location) {
+
+                       remove_sorted_marker (i->second->start);
+                       if (i->second->end) {
+                               remove_sorted_marker (i->second->end);
+                       }
+                       
+                       
+                       delete i->second;
                        location_markers.erase (i);
                        break;
                }
@@ -1243,15 +1435,12 @@ Editor::marker_selection_changed ()
                return;
        }
 
-       if (!_show_marker_lines) {
-               for (LocationMarkerMap::iterator i = location_markers.begin(); i != location_markers.end(); ++i) {
-                       i->second->hide_lines ();
-               }
+       for (LocationMarkerMap::iterator i = location_markers.begin(); i != location_markers.end(); ++i) {
+               i->second->set_selected (false);
        }
 
        for (MarkerSelection::iterator x = selection->markers.begin(); x != selection->markers.end(); ++x) {
-               (*x)->add_line (cursor_group, 0, _canvas_height);
-               (*x)->show_line ();
+               (*x)->set_selected (true);
        }
 }
 
@@ -1318,10 +1507,14 @@ Editor::toggle_marker_lines ()
        _show_marker_lines = !_show_marker_lines;
        
        for (LocationMarkerMap::iterator i = location_markers.begin(); i != location_markers.end(); ++i) {
-               if (_show_marker_lines) {
-                       i->second->show_lines (cursor_group, _canvas_height);
-               } else {
-                       i->second->hide_lines ();
-               }
+               i->second->set_show_lines (_show_marker_lines);
+       }
+}
+
+void
+Editor::remove_sorted_marker (Marker* m)
+{
+       for (std::map<ArdourCanvas::Group *, std::list<Marker *> >::iterator i = _sorted_marker_lists.begin(); i != _sorted_marker_lists.end(); ++i) {
+               i->second.remove (m);
        }
 }