Fix inverted logic of SnapOff with snap modifiers pressed.
[ardour.git] / gtk2_ardour / editor_drag.cc
index 799d441b16743abc48d537264b2772e6da632f69..014b4679e8e4499cfe6b078fa44ff56d2ca5867c 100644 (file)
@@ -226,6 +226,7 @@ Drag::Drag (Editor* e, ArdourCanvas::Item* i, bool trackview_only)
        , _raw_grab_frame (0)
        , _grab_frame (0)
        , _last_pointer_frame (0)
+       , _snap_delta (0)
 {
 
 }
@@ -250,18 +251,9 @@ Drag::start_grab (GdkEvent* event, Gdk::Cursor *cursor)
 {
        // if dragging with button2, the motion is x constrained, with Alt-button2 it is y constrained
 
-       if (Keyboard::is_button2_event (&event->button)) {
-               if (Keyboard::modifier_state_equals (event->button.state, Keyboard::SecondaryModifier)) {
-                       _y_constrained = true;
-                       _x_constrained = false;
-               } else {
-                       _y_constrained = false;
-                       _x_constrained = true;
-               }
-       } else {
-               _x_constrained = false;
-               _y_constrained = false;
-       }
+       /* we set up x/y dragging constraints on first move */
+       _x_constrained = false;
+       _y_constrained = false;
 
        _raw_grab_frame = _editor->canvas_event_sample (event, &_grab_x, &_grab_y);
 
@@ -344,6 +336,16 @@ Drag::adjusted_current_frame (GdkEvent const * event, bool snap) const
        return adjusted_frame (_drags->current_pointer_frame (), event, snap);
 }
 
+frameoffset_t
+Drag::snap_delta (guint state) const
+{
+       if (ArdourKeyboard::indicates_snap_delta (state)) {
+               return 0;
+       }
+
+       return _snap_delta;
+}
+
 double
 Drag::current_pointer_x() const
 {
@@ -360,6 +362,14 @@ Drag::current_pointer_y () const
        return _drags->current_pointer_y () - _editor->get_trackview_group()->canvas_origin().y;
 }
 
+void
+Drag::setup_snap_delta (framepos_t pos)
+{
+       framepos_t temp = pos;
+       _editor->snap_to (temp, ARDOUR::RoundNearest, false, true);
+       _snap_delta = temp - pos;
+}
+
 bool
 Drag::motion_handler (GdkEvent* event, bool from_autoscroll)
 {
@@ -391,10 +401,27 @@ Drag::motion_handler (GdkEvent* event, bool from_autoscroll)
                                /* just changed */
 
                                if (fabs (current_pointer_y() - _grab_y) > fabs (current_pointer_x() - _grab_x)) {
+                                       if ((event->motion.state & Gdk::BUTTON2_MASK) && Config->get_edit_mode() != Lock) {
+                                               _x_constrained = true;
+                                               _y_constrained = false;
+                                       }
                                        _initially_vertical = true;
                                } else {
+                                       if ((event->motion.state & Gdk::BUTTON2_MASK) && Config->get_edit_mode() != Lock) {
+                                               _x_constrained = false;
+                                               _y_constrained = true;
+                                       }
                                        _initially_vertical = false;
                                }
+
+                               if (Config->get_edit_mode() == Lock) {
+                                       if (event->button.state & Gdk::BUTTON2_MASK) {
+                                               _x_constrained = false;
+                                       } else {
+                                               _x_constrained = true;
+                                       }
+                                       _y_constrained = false;
+                               }
                        }
 
                        if (!from_autoscroll) {
@@ -556,11 +583,9 @@ RegionMotionDrag::RegionMotionDrag (Editor* e, ArdourCanvas::Item* i, RegionView
        , _total_x_delta (0)
        , _last_pointer_time_axis_view (0)
        , _last_pointer_layer (0)
-       , _single_axis (false)
        , _ndropzone (0)
        , _pdropzone (0)
        , _ddropzone (0)
-       , _snap_delta (0)
 {
        DEBUG_TRACE (DEBUG::Drags, "New RegionMotionDrag\n");
 }
@@ -569,16 +594,7 @@ void
 RegionMotionDrag::start_grab (GdkEvent* event, Gdk::Cursor* cursor)
 {
        Drag::start_grab (event, cursor);
-
-       if (_editor->snap_delta () == SnapRelative) {
-               framepos_t temp = _last_frame_position;
-               _editor->snap_to_with_modifier (temp, event, RoundUpMaybe);
-               _snap_delta = temp - _last_frame_position;
-       }
-
-       if (Keyboard::modifier_state_contains (event->button.state, Keyboard::TertiaryModifier)) {
-               _single_axis = true;
-       }
+       setup_snap_delta (_last_frame_position);
 
        show_verbose_cursor_time (_last_frame_position);
 
@@ -596,7 +612,7 @@ RegionMotionDrag::compute_x_delta (GdkEvent const * event, framepos_t* pending_r
        /* compute the amount of pointer motion in frames, and where
           the region would be if we moved it by that much.
        */
-       *pending_region_position = adjusted_frame (_drags->current_pointer_frame () + _snap_delta, event, true);
+       *pending_region_position = adjusted_frame (_drags->current_pointer_frame () + snap_delta (event->button.state), event, true);
 
        framepos_t sync_frame;
        framecnt_t sync_offset;
@@ -612,7 +628,7 @@ RegionMotionDrag::compute_x_delta (GdkEvent const * event, framepos_t* pending_r
 
                _editor->snap_to_with_modifier (sync_frame, event);
 
-               *pending_region_position = _primary->region()->adjust_to_sync (sync_frame) - _snap_delta;
+               *pending_region_position = _primary->region()->adjust_to_sync (sync_frame) - snap_delta (event->button.state);
 
        } else {
                *pending_region_position = _last_frame_position;
@@ -624,8 +640,7 @@ RegionMotionDrag::compute_x_delta (GdkEvent const * event, framepos_t* pending_r
 
        double dx = 0;
 
-       /* in locked edit mode, reverse the usual meaning of _x_constrained */
-       bool const x_move_allowed = Config->get_edit_mode() == Lock ? _x_constrained : !_x_constrained;
+       bool const x_move_allowed = !_x_constrained;
 
        if ((*pending_region_position != _last_frame_position) && x_move_allowed) {
 
@@ -758,18 +773,6 @@ RegionMotionDrag::motion (GdkEvent* event, bool first_move)
 
        assert (!_views.empty ());
 
-       if (first_move) {
-               if (_single_axis) {
-                       if (initially_vertical()) {
-                               _y_constrained = false;
-                               _x_constrained = true;
-                       } else {
-                               _y_constrained = true;
-                               _x_constrained = false;
-                       }
-               }
-       }
-
        /* Note: time axis views in this method are often expressed as an index into the _time_axis_views vector */
 
        /* Find the TimeAxisView that the pointer is now over */
@@ -1312,14 +1315,6 @@ RegionMoveDrag::finished (GdkEvent* ev, bool movement_occurred)
                return;
        }
 
-       /* reverse this here so that we have the correct logic to finalize
-          the drag.
-       */
-
-       if (Config->get_edit_mode() == Lock) {
-               _x_constrained = !_x_constrained;
-       }
-
        assert (!_views.empty ());
 
        /* We might have hidden region views so that they weren't visible during the drag
@@ -2351,24 +2346,13 @@ NoteResizeDrag::start_grab (GdkEvent* event, Gdk::Cursor* /*ignored*/)
 
        region = &cnote->region_view();
 
-       if (_editor->snap_delta () == SnapRelative) {
-               double temp;
-               temp = region->snap_to_pixel(cnote->x0 ());
-               _snap_delta = temp - cnote->x0 ();
-               /*
-               if (at_front) {
-                       temp = region->snap_to_pixel(cnote->x0 ());
-                       _snap_delta = temp - cnote->x0 ();
-               } else {
-                       temp = region->snap_to_pixel(cnote->x1 ());
-                       _snap_delta = temp - cnote->x1 ();
-               }
-               */
-       }
+       double temp;
+       temp = region->snap_to_pixel (cnote->x0 (), true);
+       _snap_delta = temp - cnote->x0 ();
 
        _item->grab ();
 
-       if (event->motion.state & Keyboard::PrimaryModifier) {
+       if (event->motion.state & ArdourKeyboard::note_size_relative_modifier ()) {
                relative = false;
        } else {
                relative = true;
@@ -2401,7 +2385,7 @@ NoteResizeDrag::start_grab (GdkEvent* event, Gdk::Cursor* /*ignored*/)
 }
 
 void
-NoteResizeDrag::motion (GdkEvent* /*event*/, bool /*first_move*/)
+NoteResizeDrag::motion (GdkEvent* event, bool /*first_move*/)
 {
        MidiRegionSelection& ms (_editor->get_selection().midi_regions);
        for (MidiRegionSelection::iterator r = ms.begin(); r != ms.end(); ++r) {
@@ -2409,21 +2393,62 @@ NoteResizeDrag::motion (GdkEvent* /*event*/, bool /*first_move*/)
                assert (nb);
                MidiRegionView* mrv = dynamic_cast<MidiRegionView*>(*r);
                if (mrv) {
-                       mrv->update_resizing (nb, at_front, _drags->current_pointer_x() - grab_x(), relative, _snap_delta);
+                       double sd = 0.0;
+                       bool snap = true;
+                       bool apply_snap_delta = !ArdourKeyboard::indicates_snap_delta (event->button.state);
+
+                       if (ArdourKeyboard::indicates_snap (event->button.state)) {
+                               if (_editor->snap_mode () != SnapOff) {
+                                       snap = false;
+                               }
+                       } else {
+                               if (_editor->snap_mode () == SnapOff) {
+                                       snap = false;
+                                       /* inverted logic here - we;re in snapoff but we've pressed the snap delta modifier */
+                                       if (!apply_snap_delta) {
+                                               snap = true;
+                                       }
+                               }
+                       }
+
+                       if (apply_snap_delta) {
+                               sd = _snap_delta;
+                       }
+
+                       mrv->update_resizing (nb, at_front, _drags->current_pointer_x() - grab_x(), relative, sd, snap);
                }
        }
 }
 
 void
-NoteResizeDrag::finished (GdkEvent*, bool /*movement_occurred*/)
+NoteResizeDrag::finished (GdkEvent* event, bool /*movement_occurred*/)
 {
        MidiRegionSelection& ms (_editor->get_selection().midi_regions);
        for (MidiRegionSelection::iterator r = ms.begin(); r != ms.end(); ++r) {
                NoteBase* nb = reinterpret_cast<NoteBase*> (_item->get_data ("notebase"));
                assert (nb);
                MidiRegionView* mrv = dynamic_cast<MidiRegionView*>(*r);
+               double sd = 0.0;
+               bool snap = true;
+               bool apply_snap_delta = !ArdourKeyboard::indicates_snap_delta (event->button.state);
                if (mrv) {
-                       mrv->commit_resizing (nb, at_front, _drags->current_pointer_x() - grab_x(), relative, _snap_delta);
+                       if (ArdourKeyboard::indicates_snap (event->button.state)) {
+                               if (_editor->snap_mode () != SnapOff) {
+                                       snap = false;
+                               }
+                       } else {
+                               if (_editor->snap_mode () == SnapOff) {
+                                       snap = false;
+                                       /* inverted logic here - we;re in snapoff but we've pressed the snap delta modifier */
+                                       if (!apply_snap_delta) {
+                                               snap = true;
+                                       }
+                               }
+                       }
+                       if (apply_snap_delta) {
+                               sd = _snap_delta;
+                       }
+                       mrv->commit_resizing (nb, at_front, _drags->current_pointer_x() - grab_x(), relative, sd, snap);
                }
        }
 
@@ -2601,7 +2626,6 @@ TrimDrag::TrimDrag (Editor* e, ArdourCanvas::Item* i, RegionView* p, list<Region
        : RegionDrag (e, i, p, v)
        , _preserve_fade_anchor (preserve_fade_anchor)
        , _jump_position_when_done (false)
-       , _snap_delta (0)
 {
        DEBUG_TRACE (DEBUG::Drags, "New TrimDrag\n");
 }
@@ -2622,14 +2646,9 @@ TrimDrag::start_grab (GdkEvent* event, Gdk::Cursor*)
        framecnt_t const region_length = (framecnt_t) (_primary->region()->length() / speed);
 
        framepos_t const pf = adjusted_current_frame (event);
+       setup_snap_delta (region_start);
 
-       if (_editor->snap_delta () == SnapRelative) {
-               framepos_t temp = region_start;
-               _editor->snap_to_with_modifier (temp, event, RoundUpMaybe);
-               _snap_delta = temp - region_start;
-       }
-
-       if (Keyboard::modifier_state_equals (event->button.state, Keyboard::PrimaryModifier)) {
+       if (Keyboard::modifier_state_equals (event->button.state, ArdourKeyboard::trim_contents_modifier ())) {
                /* Move the contents of the region around without changing the region bounds */
                _operation = ContentsTrim;
                Drag::start_grab (event, _editor->cursors()->trimmer);
@@ -2638,8 +2657,7 @@ TrimDrag::start_grab (GdkEvent* event, Gdk::Cursor*)
                if (pf < (region_start + region_length/2)) {
                        /* closer to front */
                        _operation = StartTrim;
-
-                       if (Keyboard::modifier_state_equals (event->button.state, Keyboard::TertiaryModifier)) {
+                       if (Keyboard::modifier_state_equals (event->button.state, ArdourKeyboard::trim_anchored_modifier ())) {
                                Drag::start_grab (event, _editor->cursors()->anchored_left_side_trim);
                        } else {
                                Drag::start_grab (event, _editor->cursors()->left_side_trim);
@@ -2647,17 +2665,18 @@ TrimDrag::start_grab (GdkEvent* event, Gdk::Cursor*)
                } else {
                        /* closer to end */
                        _operation = EndTrim;
-                       if (Keyboard::modifier_state_equals (event->button.state, Keyboard::TertiaryModifier)) {
+                       if (Keyboard::modifier_state_equals (event->button.state, ArdourKeyboard::trim_anchored_modifier ())) {
                                Drag::start_grab (event, _editor->cursors()->anchored_right_side_trim);
                        } else {
                                Drag::start_grab (event, _editor->cursors()->right_side_trim);
                        }
                }
        }
-
-       if (Keyboard::modifier_state_equals (event->button.state, Keyboard::TertiaryModifier)) {
+       /* jump trim disabled for now
+       if (Keyboard::modifier_state_equals (event->button.state, Keyboard::trim_jump_modifier ())) {
                _jump_position_when_done = true;
        }
+       */
 
        switch (_operation) {
        case StartTrim:
@@ -2693,8 +2712,8 @@ TrimDrag::motion (GdkEvent* event, bool first_move)
        if (tv && tv->is_track()) {
                speed = tv->track()->speed();
        }
-       framecnt_t adj_frame = adjusted_frame (_drags->current_pointer_frame () + _snap_delta, event, true);
-       framecnt_t dt = adj_frame - raw_grab_frame () + _pointer_frame_offset - _snap_delta;
+       framecnt_t adj_frame = adjusted_frame (_drags->current_pointer_frame () + snap_delta (event->button.state), event, true);
+       framecnt_t dt = adj_frame - raw_grab_frame () + _pointer_frame_offset - snap_delta (event->button.state);
 
        if (first_move) {
 
@@ -2740,7 +2759,7 @@ TrimDrag::motion (GdkEvent* event, bool first_move)
 
        bool non_overlap_trim = false;
 
-       if (event && Keyboard::modifier_state_equals (event->button.state, Keyboard::TertiaryModifier)) {
+       if (event && Keyboard::modifier_state_contains (event->button.state, ArdourKeyboard::trim_overlap_modifier ())) {
                non_overlap_trim = true;
        }
 
@@ -2834,7 +2853,6 @@ TrimDrag::motion (GdkEvent* event, bool first_move)
        }
 }
 
-
 void
 TrimDrag::finished (GdkEvent* event, bool movement_occurred)
 {
@@ -3244,7 +3262,6 @@ CursorDrag::CursorDrag (Editor* e, EditorCursor& c, bool s)
        : Drag (e, &c.track_canvas_item(), false)
        , _cursor (c)
        , _stop (s)
-       , _snap_delta (0)
 {
        DEBUG_TRACE (DEBUG::Drags, "New CursorDrag\n");
 }
@@ -3279,16 +3296,11 @@ void
 CursorDrag::start_grab (GdkEvent* event, Gdk::Cursor* c)
 {
        Drag::start_grab (event, c);
-
-       if (_editor->snap_delta () == SnapRelative) {
-               framepos_t temp = _editor->playhead_cursor->current_frame ();
-               _editor->snap_to_with_modifier (temp, event);
-               _snap_delta = temp - _editor->playhead_cursor->current_frame ();
-       }
+       setup_snap_delta (_editor->playhead_cursor->current_frame ());
 
        _grab_zoom = _editor->samples_per_pixel;
 
-       framepos_t where = _editor->canvas_event_sample (event) + _snap_delta;
+       framepos_t where = _editor->canvas_event_sample (event) + snap_delta (event->button.state);
 
        _editor->snap_to_with_modifier (where, event);
 
@@ -3326,16 +3338,16 @@ CursorDrag::start_grab (GdkEvent* event, Gdk::Cursor* c)
                }
        }
 
-       fake_locate (where - _snap_delta);
+       fake_locate (where - snap_delta (event->button.state));
 }
 
 void
 CursorDrag::motion (GdkEvent* event, bool)
 {
-       framepos_t where = _editor->canvas_event_sample (event) + _snap_delta;
+       framepos_t where = _editor->canvas_event_sample (event) + snap_delta (event->button.state);
        _editor->snap_to_with_modifier (where, event);
        if (where != last_pointer_frame()) {
-               fake_locate (where - _snap_delta);
+               fake_locate (where - snap_delta (event->button.state));
        }
 }
 
@@ -3375,7 +3387,6 @@ CursorDrag::aborted (bool)
 
 FadeInDrag::FadeInDrag (Editor* e, ArdourCanvas::Item* i, RegionView* p, list<RegionView*> const & v)
        : RegionDrag (e, i, p, v)
-       , _snap_delta (0)
 {
        DEBUG_TRACE (DEBUG::Drags, "New FadeInDrag\n");
 }
@@ -3387,12 +3398,7 @@ FadeInDrag::start_grab (GdkEvent* event, Gdk::Cursor* cursor)
 
        AudioRegionView* arv = dynamic_cast<AudioRegionView*> (_primary);
        boost::shared_ptr<AudioRegion> const r = arv->audio_region ();
-
-       if (_editor->snap_delta () == SnapRelative) {
-               framepos_t temp = r->position();
-               _editor->snap_to_with_modifier (temp, event);
-               _snap_delta = temp - r->position();
-       }
+       setup_snap_delta (r->position ());
 
        show_verbose_cursor_duration (r->position(), r->position() + r->fade_in()->back()->when, 32);
 }
@@ -3410,9 +3416,9 @@ FadeInDrag::motion (GdkEvent* event, bool)
 {
        framecnt_t fade_length;
 
-       framepos_t pos = _editor->canvas_event_sample (event) + _snap_delta;
+       framepos_t pos = _editor->canvas_event_sample (event) + snap_delta (event->button.state);
        _editor->snap_to_with_modifier (pos, event);
-       pos -= _snap_delta;
+       pos -= snap_delta (event->button.state);
 
        boost::shared_ptr<AudioRegion> region = boost::dynamic_pointer_cast<AudioRegion> (_primary->region ());
 
@@ -3446,9 +3452,9 @@ FadeInDrag::finished (GdkEvent* event, bool movement_occurred)
        }
 
        framecnt_t fade_length;
-       framepos_t pos = _editor->canvas_event_sample (event) + _snap_delta;
+       framepos_t pos = _editor->canvas_event_sample (event) + snap_delta (event->button.state);
        _editor->snap_to_with_modifier (pos, event);
-       pos -= _snap_delta;
+       pos -= snap_delta (event->button.state);
 
        boost::shared_ptr<AudioRegion> region = boost::dynamic_pointer_cast<AudioRegion> (_primary->region ());
 
@@ -3510,12 +3516,7 @@ FadeOutDrag::start_grab (GdkEvent* event, Gdk::Cursor* cursor)
 
        AudioRegionView* arv = dynamic_cast<AudioRegionView*> (_primary);
        boost::shared_ptr<AudioRegion> r = arv->audio_region ();
-
-       if (_editor->snap_delta () == SnapRelative) {
-               framepos_t temp = r->last_frame ();
-               _editor->snap_to_with_modifier (temp, event);
-               _snap_delta = temp - r->last_frame();
-       }
+       setup_snap_delta (r->last_frame ());
 
        show_verbose_cursor_duration (r->last_frame() - r->fade_out()->back()->when, r->last_frame());
 }
@@ -3533,9 +3534,9 @@ FadeOutDrag::motion (GdkEvent* event, bool)
 {
        framecnt_t fade_length;
 
-       framepos_t pos = _editor->canvas_event_sample (event) + _snap_delta;
+       framepos_t pos = _editor->canvas_event_sample (event) + snap_delta (event->button.state);
        _editor->snap_to_with_modifier (pos, event);
-       pos -= _snap_delta;
+       pos -= snap_delta (event->button.state);
 
        boost::shared_ptr<AudioRegion> region = boost::dynamic_pointer_cast<AudioRegion> (_primary->region ());
 
@@ -3570,9 +3571,9 @@ FadeOutDrag::finished (GdkEvent* event, bool movement_occurred)
 
        framecnt_t fade_length;
 
-       framepos_t pos = _editor->canvas_event_sample (event) + _snap_delta;
+       framepos_t pos = _editor->canvas_event_sample (event) + snap_delta (event->button.state);
        _editor->snap_to_with_modifier (pos, event);
-       pos -= _snap_delta;
+       pos -= snap_delta (event->button.state);
 
        boost::shared_ptr<AudioRegion> region = boost::dynamic_pointer_cast<AudioRegion> (_primary->region ());
 
@@ -3768,7 +3769,7 @@ MarkerDrag::motion (GdkEvent* event, bool)
        framepos_t const newframe = adjusted_current_frame (event);
        framepos_t next = newframe;
 
-       if (Keyboard::modifier_state_equals (event->button.state, Keyboard::PrimaryModifier)) {
+       if (Keyboard::modifier_state_contains (event->button.state, ArdourKeyboard::push_points_modifier ())) {
                move_both = true;
        }
 
@@ -4027,14 +4028,7 @@ ControlPointDrag::start_grab (GdkEvent* event, Gdk::Cursor* /*cursor*/)
        _fixed_grab_y = _point->get_y();
 
        framepos_t pos = _editor->pixel_to_sample (_fixed_grab_x);
-
-       if (_editor->snap_delta () == SnapRelative) {
-               framepos_t temp = pos;
-               if (!_x_constrained) {
-                       _editor->snap_to_with_modifier (temp, event);
-               }
-               _snap_delta = temp - pos;
-       }
+       setup_snap_delta (pos);
 
        float const fraction = 1 - (_point->get_y() / _point->line().height());
 
@@ -4042,7 +4036,7 @@ ControlPointDrag::start_grab (GdkEvent* event, Gdk::Cursor* /*cursor*/)
 
        show_verbose_cursor_text (_point->line().get_verbose_cursor_string (fraction));
 
-       _pushing = Keyboard::modifier_state_contains (event->button.state, Keyboard::PrimaryModifier);
+       _pushing = Keyboard::modifier_state_equals (event->button.state, ArdourKeyboard::push_points_modifier ());
 
        if (!_point->can_slide ()) {
                _x_constrained = true;
@@ -4055,7 +4049,7 @@ ControlPointDrag::motion (GdkEvent* event, bool)
        double dx = _drags->current_pointer_x() - last_pointer_x();
        double dy = current_pointer_y() - last_pointer_y();
 
-       if (event->button.state & Keyboard::SecondaryModifier) {
+       if (event->button.state & ArdourKeyboard::fine_adjust_modifier ()) {
                dx *= 0.1;
                dy *= 0.1;
        }
@@ -4088,13 +4082,13 @@ ControlPointDrag::motion (GdkEvent* event, bool)
        cy = max (0.0, cy);
        cy = min ((double) _point->line().height(), cy);
 
-       framepos_t cx_frames = _editor->pixel_to_sample (cx) + _snap_delta;
+       framepos_t cx_frames = _editor->pixel_to_sample (cx) + snap_delta (event->button.state);
 
        if (!_x_constrained) {
                _editor->snap_to_with_modifier (cx_frames, event);
        }
 
-       cx_frames -= _snap_delta;
+       cx_frames -= snap_delta (event->button.state);
        cx_frames = min (cx_frames, _point->line().maximum_time());
 
        float const fraction = 1.0 - (cy / _point->line().height());
@@ -4110,8 +4104,7 @@ ControlPointDrag::finished (GdkEvent* event, bool movement_occurred)
        if (!movement_occurred) {
 
                /* just a click */
-
-               if (Keyboard::modifier_state_equals (event->button.state, Keyboard::TertiaryModifier)) {
+               if (Keyboard::modifier_state_equals (event->button.state, Keyboard::ModifierMask (Keyboard::TertiaryModifier))) {
                        _editor->reset_point_selection ();
                }
 
@@ -4195,7 +4188,7 @@ LineDrag::motion (GdkEvent* event, bool)
 {
        double dy = current_pointer_y() - last_pointer_y();
 
-       if (event->button.state & Keyboard::SecondaryModifier) {
+       if (event->button.state & ArdourKeyboard::fine_adjust_modifier ()) {
                dy *= 0.1;
        }
 
@@ -4495,7 +4488,6 @@ RubberbandSelectDrag::aborted (bool)
 
 TimeFXDrag::TimeFXDrag (Editor* e, ArdourCanvas::Item* i, RegionView* p, std::list<RegionView*> const & v)
        : RegionDrag (e, i, p, v)
-       , _snap_delta (0)
 {
        DEBUG_TRACE (DEBUG::Drags, "New TimeFXDrag\n");
 }
@@ -4508,11 +4500,7 @@ TimeFXDrag::start_grab (GdkEvent* event, Gdk::Cursor* cursor)
        _editor->get_selection().add (_primary);
 
        framepos_t where = _primary->region()->position();
-       if (_editor->snap_delta () == SnapRelative) {
-               framepos_t temp = where;
-               _editor->snap_to_with_modifier (temp, event);
-               _snap_delta = temp - where;
-       }
+       setup_snap_delta (where);
 
        show_verbose_cursor_duration (where, adjusted_current_frame (event), 0);
 }
@@ -4526,9 +4514,9 @@ TimeFXDrag::motion (GdkEvent* event, bool)
        pair<TimeAxisView*, double> const tv = _editor->trackview_by_y_position (grab_y());
        int layer = tv.first->layer_display() == Overlaid ? 0 : tv.second;
        int layers = tv.first->layer_display() == Overlaid ? 1 : cv->layers();
-       framepos_t pf = _editor->canvas_event_sample (event) + _snap_delta;
+       framepos_t pf = _editor->canvas_event_sample (event) + snap_delta (event->button.state);
        _editor->snap_to_with_modifier (pf, event);
-       pf -= _snap_delta;
+       pf -= snap_delta (event->button.state);
 
        if (pf > rv->region()->position()) {
                rv->get_time_axis_view().show_timestretch (rv->region()->position(), pf, layers, layer);
@@ -4992,7 +4980,7 @@ RangeMarkerBarDrag::start_grab (GdkEvent* event, Gdk::Cursor *)
        case CreateTransportMarker:
        case CreateCDMarker:
 
-               if (Keyboard::modifier_state_equals (event->button.state, Keyboard::TertiaryModifier)) {
+               if (Keyboard::modifier_state_equals (event->button.state, Keyboard::CopyModifier)) {
                        _copy = true;
                } else {
                        _copy = false;
@@ -5200,7 +5188,6 @@ NoteDrag::NoteDrag (Editor* e, ArdourCanvas::Item* i)
        : Drag (e, i)
        , _cumulative_dx (0)
        , _cumulative_dy (0)
-       , _snap_delta (0)
 {
        DEBUG_TRACE (DEBUG::Drags, "New NoteDrag\n");
 
@@ -5214,13 +5201,7 @@ void
 NoteDrag::start_grab (GdkEvent* event, Gdk::Cursor *)
 {
        Drag::start_grab (event);
-
-       if (_editor->snap_delta () == SnapRelative) {
-               framepos_t where = _region->source_beats_to_absolute_frames (_primary->note()->time ());
-               framepos_t temp = where;
-               _editor->snap_to_with_modifier (temp, event);
-               _snap_delta = temp - where;
-       }
+       setup_snap_delta (_region->source_beats_to_absolute_frames (_primary->note()->time ()));
 
        if (!(_was_selected = _primary->selected())) {
 
@@ -5248,7 +5229,7 @@ NoteDrag::start_grab (GdkEvent* event, Gdk::Cursor *)
 
 /** @return Current total drag x change in frames */
 frameoffset_t
-NoteDrag::total_dx () const
+NoteDrag::total_dx (const guint state) const
 {
        /* dx in frames */
        frameoffset_t const dx = _editor->pixel_to_sample (_drags->current_pointer_x() - grab_x());
@@ -5257,15 +5238,32 @@ NoteDrag::total_dx () const
        frameoffset_t const n = _region->source_beats_to_absolute_frames (_primary->note()->time ());
 
        /* new time of the primary note in session frames */
-       frameoffset_t st = n + dx + _snap_delta;
+       frameoffset_t st = n + dx + snap_delta (state);
 
        framepos_t const rp = _region->region()->position ();
 
        /* prevent the note being dragged earlier than the region's position */
        st = max (st, rp);
 
-       /* snap and return corresponding delta */
-       return _region->snap_frame_to_frame (st - rp) + rp - n - _snap_delta;
+       /* possibly snap and return corresponding delta */
+
+       bool snap = true;
+
+       if (ArdourKeyboard::indicates_snap (state)) {
+               if (_editor->snap_mode () != SnapOff) {
+                       snap = false;
+               }
+       } else {
+               if (_editor->snap_mode () == SnapOff) {
+                       snap = false;
+                       /* inverted logic here - we;re in snapoff but we've pressed the snap delta modifier */
+                       if (ArdourKeyboard::indicates_snap_delta (state)) {
+                               snap = true;
+                       }
+               }
+       }
+
+       return _region->snap_frame_to_frame (st - rp, snap) + rp - n - snap_delta (state);
 }
 
 /** @return Current total drag y change in note number */
@@ -5284,10 +5282,10 @@ NoteDrag::total_dy () const
 }
 
 void
-NoteDrag::motion (GdkEvent *, bool)
+NoteDrag::motion (GdkEvent * event, bool)
 {
        /* Total change in x and y since the start of the drag */
-       frameoffset_t const dx = total_dx ();
+       frameoffset_t const dx = total_dx (event->button.state);
        int8_t const dy = total_dy ();
 
        /* Now work out what we have to do to the note canvas items to set this new drag delta */
@@ -5356,7 +5354,7 @@ NoteDrag::finished (GdkEvent* ev, bool moved)
                        }
                }
        } else {
-               _region->note_dropped (_primary, total_dx(), total_dy());
+               _region->note_dropped (_primary, total_dx (ev->button.state), total_dy());
        }
 }
 
@@ -5971,7 +5969,7 @@ RegionCutDrag::RegionCutDrag (Editor* e, ArdourCanvas::Item* item, framepos_t po
        , line (new EditorCursor (*e))
 {
        line->set_position (pos);
-       line->how ();
+       line->show ();
 }
 
 RegionCutDrag::~RegionCutDrag ()