comparison layer/RegionLayer.cpp @ 414:fc19435ac0f5

* Improve snap policy for selecting regions

     return text;
 }
 
 bool
 RegionLayer::snapToFeatureFrame(View *v, int &frame,
-			      size_t &resolution,
-			      SnapType snap) const
+                                size_t &resolution,
+                                SnapType snap) const
 {
     if (!m_model) {
 	return Layer::snapToFeatureFrame(v, frame, resolution, snap);
     }
 

     for (RegionModel::PointList::const_iterator i = points.begin();
 	 i != points.end(); ++i) {
 
 	if (snap == SnapRight) {
 
-	    if (i->frame > frame) {
-		snapped = i->frame;
-		found = true;
-		break;
-	    }
+            // The best frame to snap to is the end frame of whichever
+            // feature we would have snapped to the start frame of if
+            // we had been snapping left.
+
+	    if (i->frame <= frame) {
+                if (i->frame + i->duration > frame) {
+                    snapped = i->frame + i->duration;
+                    found = true; // don't break, as the next may be better
+                }
+            } else {
+                if (!found) {
+                    snapped = i->frame;
+                    found = true;
+                }
+                break;
+            }
 
 	} else if (snap == SnapLeft) {
 
 	    if (i->frame <= frame) {
 		snapped = i->frame;