enough with umpteen "i18n.h" files. Consolidate on pbd/i18n.h
[ardour.git] / gtk2_ardour / midi_region_view.cc
index 3a3fe58be92de5a76056b8853cf1543bc0fb49ba..79933b94d6cdb58afb0877c796720801733885c3 100644 (file)
@@ -82,7 +82,7 @@
 #include "sys_ex.h"
 #include "ui_config.h"
 
-#include "i18n.h"
+#include "pbd/i18n.h"
 
 using namespace ARDOUR;
 using namespace PBD;
@@ -90,8 +90,6 @@ using namespace Editing;
 using namespace std;
 using Gtkmm2ext::Keyboard;
 
-PBD::Signal1<void, MidiRegionView *> MidiRegionView::SelectionCleared;
-
 #define MIDI_BP_ZERO ((Config->get_first_midi_bank_is_zero())?0:1)
 
 MidiRegionView::MidiRegionView (ArdourCanvas::Container*      parent,
@@ -125,6 +123,7 @@ MidiRegionView::MidiRegionView (ArdourCanvas::Container*      parent,
        , _last_event_y (0)
        , _grabbed_keyboard (false)
        , _entered (false)
+       , _note_entered (false)
        , _mouse_changed_selection (false)
 {
        CANVAS_DEBUG_NAME (_note_group, string_compose ("note group for %1", get_item_name()));
@@ -133,11 +132,6 @@ MidiRegionView::MidiRegionView (ArdourCanvas::Container*      parent,
 
        Config->ParameterChanged.connect (*this, invalidator (*this), boost::bind (&MidiRegionView::parameter_changed, this, _1), gui_context());
        connect_to_diskstream ();
-
-       SelectionCleared.connect (_selection_cleared_connection, invalidator (*this), boost::bind (&MidiRegionView::selection_cleared, this, _1), gui_context ());
-
-       PublicEditor& editor (trackview.editor());
-       editor.get_selection().ClearMidiNoteSelection.connect (_clear_midi_selection_connection, invalidator (*this), boost::bind (&MidiRegionView::clear_midi_selection, this), gui_context ());
 }
 
 MidiRegionView::MidiRegionView (ArdourCanvas::Container*      parent,
@@ -173,6 +167,7 @@ MidiRegionView::MidiRegionView (ArdourCanvas::Container*      parent,
        , _last_event_y (0)
        , _grabbed_keyboard (false)
        , _entered (false)
+       , _note_entered (false)
        , _mouse_changed_selection (false)
 {
        CANVAS_DEBUG_NAME (_note_group, string_compose ("note group for %1", get_item_name()));
@@ -181,11 +176,6 @@ MidiRegionView::MidiRegionView (ArdourCanvas::Container*      parent,
        PublicEditor::DropDownKeys.connect (sigc::mem_fun (*this, &MidiRegionView::drop_down_keys));
 
        connect_to_diskstream ();
-
-       SelectionCleared.connect (_selection_cleared_connection, invalidator (*this), boost::bind (&MidiRegionView::selection_cleared, this, _1), gui_context ());
-
-       PublicEditor& editor (trackview.editor());
-       editor.get_selection().ClearMidiNoteSelection.connect (_clear_midi_selection_connection, invalidator (*this), boost::bind (&MidiRegionView::clear_midi_selection, this), gui_context ());
 }
 
 void
@@ -226,6 +216,7 @@ MidiRegionView::MidiRegionView (const MidiRegionView& other)
        , _last_event_y (0)
        , _grabbed_keyboard (false)
        , _entered (false)
+       , _note_entered (false)
        , _mouse_changed_selection (false)
 {
        init (false);
@@ -258,6 +249,7 @@ MidiRegionView::MidiRegionView (const MidiRegionView& other, boost::shared_ptr<M
        , _last_event_y (0)
        , _grabbed_keyboard (false)
        , _entered (false)
+       , _note_entered (false)
        , _mouse_changed_selection (false)
 {
        init (true);
@@ -279,12 +271,12 @@ MidiRegionView::init (bool wfd)
 
        RegionView::init (false);
 
-       set_height (trackview.current_height());
+       //set_height (trackview.current_height());
 
        region_muted ();
        region_sync_changed ();
        region_resized (ARDOUR::bounds_change);
-       region_locked ();
+       //region_locked ();
 
        set_colors ();
 
@@ -316,11 +308,6 @@ MidiRegionView::init (bool wfd)
 
        Config->ParameterChanged.connect (*this, invalidator (*this), boost::bind (&MidiRegionView::parameter_changed, this, _1), gui_context());
        connect_to_diskstream ();
-
-       SelectionCleared.connect (_selection_cleared_connection, invalidator (*this), boost::bind (&MidiRegionView::selection_cleared, this, _1), gui_context ());
-
-       PublicEditor& editor (trackview.editor());
-       editor.get_selection().ClearMidiNoteSelection.connect (_clear_midi_selection_connection, invalidator (*this), boost::bind (&MidiRegionView::clear_midi_selection, this), gui_context ());
 }
 
 InstrumentInfo&
@@ -430,13 +417,26 @@ MidiRegionView::mouse_mode_changed ()
        set_frame_color();
 
        if (_entered) {
-               if (trackview.editor().internal_editing()) {
-                       // Switched in to internal editing mode while entered
-                       enter_internal();
-               } else {
-                       // Switched out of internal editing mode while entered
+               if (!trackview.editor().internal_editing()) {
+                       /* Switched out of internal editing mode while entered.
+                       Only necessary for leave as a mouse_mode_change over a region
+                       automatically triggers an enter event. */
                        leave_internal();
                }
+               else if (trackview.editor().current_mouse_mode() == MouseContent) {
+                       // hide cursor and ghost note after changing to internal edit mode
+                       remove_ghost_note ();
+
+                       /* XXX This is problematic as the function is executed for every region
+                       and only for one region _note_entered can be true. Still it's
+                       necessary as to hide the verbose cursor when we're changing from
+                       draw mode to internal edit mode. These lines are the reason why
+                       in some situations no verbose cursor is shown when we enter internal
+                       edit mode over a note. */
+                       if (!_note_entered) {
+                               hide_verbose_cursor ();
+                       }
+               }
        }
 }
 
@@ -468,6 +468,7 @@ MidiRegionView::leave_internal()
 {
        hide_verbose_cursor ();
        remove_ghost_note ();
+       _note_entered = false;
 
        if (_grabbed_keyboard) {
                Keyboard::magic_widget_drop_focus();
@@ -535,8 +536,8 @@ MidiRegionView::button_release (GdkEventButton* ev)
 
                switch (editor.current_mouse_mode()) {
                case MouseRange:
-                       /* no motion occured - simple click */
-                       clear_selection ();
+                       /* no motion occurred - simple click */
+                       clear_editor_note_selection ();
                        _mouse_changed_selection = true;
                        break;
 
@@ -554,15 +555,9 @@ MidiRegionView::button_release (GdkEventButton* ev)
                                        group->canvas_to_item (event_x, event_y);
 
                                        Evoral::Beats beats = get_grid_beats(editor.pixel_to_sample(event_x));
-
-                                       /* Shorten the length by 1 tick so that we can add a new note at the next
-                                          grid snap without it overlapping this one.
-                                       */
-                                       beats -= Evoral::Beats::tick();
-
                                        create_note_at (editor.pixel_to_sample (event_x), event_y, beats, true);
                                } else {
-                                       clear_selection ();
+                                       clear_editor_note_selection ();
                                }
 
                                break;
@@ -570,14 +565,7 @@ MidiRegionView::button_release (GdkEventButton* ev)
                case MouseDraw:
                        {
                                Evoral::Beats beats = get_grid_beats(editor.pixel_to_sample(event_x));
-
-                               /* Shorten the length by 1 tick so that we can add a new note at the next
-                                  grid snap without it overlapping this one.
-                               */
-                               beats -= Evoral::Beats::tick();
-
                                create_note_at (editor.pixel_to_sample (event_x), event_y, beats, true);
-
                                break;
                        }
                default:
@@ -587,11 +575,12 @@ MidiRegionView::button_release (GdkEventButton* ev)
                _mouse_state = None;
                break;
 
-       case SelectRectDragging:
        case AddDragging:
+               /* Only create a ghost note when we added a note, not when we were drag-selecting. */
+               create_ghost_note (ev->x, ev->y);
+       case SelectRectDragging:
                editor.drags()->end_grab ((GdkEvent *) ev);
                _mouse_state = None;
-               create_ghost_note (ev->x, ev->y);
                break;
 
 
@@ -612,25 +601,33 @@ MidiRegionView::motion (GdkEventMotion* ev)
 {
        PublicEditor& editor = trackview.editor ();
 
-       if (!_ghost_note && editor.current_mouse_mode() == MouseContent &&
-           Keyboard::modifier_state_contains (ev->state, Keyboard::insert_note_modifier()) &&
-           _mouse_state != AddDragging) {
+       if (!_note_entered) {
 
-               create_ghost_note (ev->x, ev->y);
+               if (!_ghost_note && editor.current_mouse_mode() == MouseContent &&
+                   Keyboard::modifier_state_contains (ev->state, Keyboard::insert_note_modifier()) &&
+                   _mouse_state != AddDragging) {
 
-       } else if (_ghost_note && editor.current_mouse_mode() == MouseContent &&
-                  Keyboard::modifier_state_contains (ev->state, Keyboard::insert_note_modifier())) {
+                       create_ghost_note (ev->x, ev->y);
 
-               update_ghost_note (ev->x, ev->y);
+               } else if (_ghost_note && editor.current_mouse_mode() == MouseContent &&
+                          Keyboard::modifier_state_contains (ev->state, Keyboard::insert_note_modifier())) {
 
-       } else if (_ghost_note && editor.current_mouse_mode() == MouseContent) {
+                       update_ghost_note (ev->x, ev->y);
 
-               remove_ghost_note ();
-               hide_verbose_cursor ();
+               } else if (_ghost_note && editor.current_mouse_mode() == MouseContent) {
+
+                       remove_ghost_note ();
+                       hide_verbose_cursor ();
 
-       } else if (_ghost_note && editor.current_mouse_mode() == MouseDraw) {
+               } else if (editor.current_mouse_mode() == MouseDraw) {
 
-               update_ghost_note (ev->x, ev->y);
+                       if (_ghost_note) {
+                               update_ghost_note (ev->x, ev->y);
+                       }
+                       else {
+                               create_ghost_note (ev->x, ev->y);
+                       }
+               }
        }
 
        /* any motion immediately hides velocity text that may have been visible */
@@ -655,7 +652,7 @@ MidiRegionView::motion (GdkEventMotion* ev)
                        } else if (m == MouseContent) {
                                editor.drags()->set (new MidiRubberbandSelectDrag (dynamic_cast<Editor *> (&editor), this), (GdkEvent *) ev);
                                if (!Keyboard::modifier_state_equals (ev->state, Keyboard::TertiaryModifier)) {
-                                       clear_selection ();
+                                       clear_editor_note_selection ();
                                        _mouse_changed_selection = true;
                                }
                                _mouse_state = SelectRectDragging;
@@ -738,7 +735,7 @@ MidiRegionView::key_press (GdkEventKey* ev)
                return true;
 
        } else if (ev->keyval == GDK_Escape && unmodified) {
-               clear_selection();
+               clear_editor_note_selection ();
                _mouse_state = None;
 
        } else if (ev->keyval == GDK_comma || ev->keyval == GDK_period) {
@@ -931,16 +928,24 @@ MidiRegionView::create_note_at (framepos_t t, double y, Evoral::Beats length, bo
 
        MidiTimeAxisView* const mtv  = dynamic_cast<MidiTimeAxisView*>(&trackview);
        MidiStreamView* const   view = mtv->midi_view();
+       boost::shared_ptr<MidiRegion> mr  = boost::dynamic_pointer_cast<MidiRegion> (_region);
+
+       if (!mr) {
+               return;
+       }
 
        // Start of note in frames relative to region start
+       uint32_t divisions = 0;
+
        if (snap_t) {
                framecnt_t grid_frames;
                t = snap_frame_to_grid_underneath (t, grid_frames);
+               divisions = trackview.editor().get_grid_music_divisions (0);
        }
 
-       const MidiModel::TimeType beat_time = region_frames_to_region_beats(
-               t + _region->start());
 
+       const MidiModel::TimeType beat_time = Evoral::Beats (trackview.session()->tempo_map().exact_beat_at_frame (_region->position() + t, divisions)
+                                                            - (mr->beat() - mr->start_beats().to_double()));
        const double  note     = view->y_to_note(y);
        const uint8_t chan     = mtv->get_channel_for_add();
        const uint8_t velocity = get_velocity_for_add(beat_time);
@@ -956,7 +961,7 @@ MidiRegionView::create_note_at (framepos_t t, double y, Evoral::Beats length, bo
 
        start_note_diff_command(_("add note"));
 
-       clear_selection ();
+       clear_editor_note_selection ();
        note_diff_add_note (new_note, true, false);
 
        apply_diff();
@@ -965,9 +970,10 @@ MidiRegionView::create_note_at (framepos_t t, double y, Evoral::Beats length, bo
 }
 
 void
-MidiRegionView::clear_events (bool with_selection_signal)
+MidiRegionView::clear_events ()
 {
-       clear_selection (with_selection_signal);
+       // clear selection without signaling
+       clear_selection_internal ();
 
        MidiGhostRegion* gr;
        for (std::vector<GhostRegion*>::iterator g = ghosts.begin(); g != ghosts.end(); ++g) {
@@ -994,7 +1000,7 @@ MidiRegionView::display_model(boost::shared_ptr<MidiModel> model)
        content_connection.disconnect ();
        _model->ContentsChanged.connect (content_connection, invalidator (*this), boost::bind (&MidiRegionView::redisplay_model, this), gui_context());
        /* Don't signal as nobody else needs to know until selection has been altered. */
-       clear_events (false);
+       clear_events ();
 
        if (_enable_display) {
                redisplay_model();
@@ -1096,7 +1102,7 @@ MidiRegionView::abort_command()
 {
        delete _note_diff_command;
        _note_diff_command = 0;
-       clear_selection();
+       clear_editor_note_selection();
 }
 
 NoteBase*
@@ -1182,21 +1188,20 @@ MidiRegionView::redisplay_model()
        _optimization_iterator = _events.begin();
 
        bool empty_when_starting = _events.empty();
+       NoteBase* cne;
 
        for (MidiModel::Notes::iterator n = notes.begin(); n != notes.end(); ++n) {
 
                boost::shared_ptr<NoteType> note (*n);
-               NoteBase* cne;
                bool visible;
 
                if (note_in_region_range (note, visible)) {
 
                        if (!empty_when_starting && (cne = find_canvas_note (note)) != 0) {
 
-                               cne->validate ();
-                               update_note (cne);
-
                                if (visible) {
+                                       cne->validate ();
+                                       update_note (cne);
                                        cne->show ();
                                } else {
                                        cne->hide ();
@@ -1391,10 +1396,8 @@ MidiRegionView::~MidiRegionView ()
                end_write();
        }
 
-       _selection_cleared_connection.disconnect ();
-
        _selection.clear();
-       clear_events (false);
+       clear_events ();
 
        delete _note_group;
        delete _note_diff_command;
@@ -1405,21 +1408,24 @@ MidiRegionView::~MidiRegionView ()
 void
 MidiRegionView::region_resized (const PropertyChange& what_changed)
 {
-       RegionView::region_resized(what_changed);
+       RegionView::region_resized(what_changed); // calls RegionView::set_duration()
 
        if (what_changed.contains (ARDOUR::Properties::position)) {
                _region_relative_time_converter.set_origin_b(_region->position());
                _region_relative_time_converter_double.set_origin_b(_region->position());
-               set_duration(_region->length(), 0);
-               if (_enable_display) {
-                       redisplay_model();
-               }
+               /* reset_width dependent_items() redisplays model */
+
        }
 
        if (what_changed.contains (ARDOUR::Properties::start) ||
            what_changed.contains (ARDOUR::Properties::position)) {
                _source_relative_time_converter.set_origin_b (_region->position() - _region->start());
        }
+       /* catch an end trim so we can live update */
+       if (!what_changed.contains (ARDOUR::Properties::start) &&
+           what_changed.contains (ARDOUR::Properties::length)) {
+               enable_display (true);
+       }
 }
 
 void
@@ -1429,7 +1435,7 @@ MidiRegionView::reset_width_dependent_items (double pixel_width)
 
        if (_enable_display) {
                redisplay_model();
-       }
+               }
 
        for (PatchChanges::iterator x = _patch_changes.begin(); x != _patch_changes.end(); ++x) {
                if ((*x)->canvas_item()->width() >= _pixel_width) {
@@ -1674,12 +1680,9 @@ MidiRegionView::start_playing_midi_chord (vector<boost::shared_ptr<NoteType> > n
 bool
 MidiRegionView::note_in_region_range (const boost::shared_ptr<NoteType> note, bool& visible) const
 {
-       /* This is imprecise due to all the conversion conversion involved, so only
-          hide notes if they seem to start more than one tick before the start. */
-       const framecnt_t tick_frames       = Evoral::Beats::tick().to_ticks(trackview.session()->frame_rate());
-       const framepos_t note_start_frames = source_beats_to_region_frames (note->time());
-       const bool       outside           = ((note_start_frames <= -tick_frames) ||
-                                             (note_start_frames >= _region->length()));
+       const boost::shared_ptr<ARDOUR::MidiRegion> midi_reg = midi_region();
+       const bool outside = (note->time() < midi_reg->start_beats() ||
+                             note->time() > midi_reg->start_beats() + midi_reg->length_beats());
 
        visible = (note->note() >= midi_stream_view()->lowest_note()) &&
                (note->note() <= midi_stream_view()->highest_note());
@@ -1706,23 +1709,36 @@ MidiRegionView::update_note (NoteBase* note, bool update_ghost_regions)
 void
 MidiRegionView::update_sustained (Note* ev, bool update_ghost_regions)
 {
+       TempoMap& map (trackview.session()->tempo_map());
+       const boost::shared_ptr<ARDOUR::MidiRegion> mr = midi_region();
        boost::shared_ptr<NoteType> note = ev->note();
-       const double x = trackview.editor().sample_to_pixel (source_beats_to_region_frames (note->time()));
+       const framepos_t note_start_frames = map.frame_at_beat (_region->beat() - mr->start_beats().to_double()
+                                                               + note->time().to_double()) - _region->position();
+
+       const double x0 = trackview.editor().sample_to_pixel (note_start_frames);
+       double x1;
        const double y0 = 1 + floor(midi_stream_view()->note_to_y(note->note()));
+       double y1;/* trim note display to not overlap the end of its region */
 
-       ev->set_x0 (x);
-       ev->set_y0 (y0);
+       if (note->length() > 0) {
+               Evoral::Beats note_end_time = note->end_time();
 
-       /* trim note display to not overlap the end of its region */
+               if (note->end_time() > mr->start_beats() + mr->length_beats()) {
+                       note_end_time = mr->start_beats() + mr->length_beats();
+               }
 
-       if (note->length() > 0) {
-               const framepos_t note_end_frames = min (source_beats_to_region_frames (note->end_time()), _region->length());
-               ev->set_x1 (trackview.editor().sample_to_pixel (note_end_frames));
+               const framepos_t note_end_frames = map.frame_at_beat (_region->beat() - mr->start_beats().to_double()
+                                                                     + note_end_time.to_double()) - _region->position();
+
+               x1 = std::max(1., trackview.editor().sample_to_pixel (note_end_frames)) - 1;
        } else {
-               ev->set_x1 (trackview.editor().sample_to_pixel (_region->length()));
+               x1 = std::max(1., trackview.editor().sample_to_pixel (_region->length())) - 1;
        }
 
-       ev->set_y1 (y0 + std::max(1., floor(midi_stream_view()->note_height()) - 1));
+       y1 = y0 + std::max(1., floor(midi_stream_view()->note_height()) - 1);
+
+       ArdourCanvas::Rect rect (x0, y0, x1, y1);
+       ev->set (rect);
 
        if (!note->length()) {
                if (_active_notes && note->note() < 128) {
@@ -1730,7 +1746,7 @@ MidiRegionView::update_sustained (Note* ev, bool update_ghost_regions)
                        if (old_rect) {
                                /* There is an active note on this key, so we have a stuck
                                   note.  Finish the old rectangle here. */
-                               old_rect->set_x1 (x);
+                               old_rect->set_x1 (x1);
                                old_rect->set_outline_all ();
                        }
                        _active_notes[note->note()] = ev;
@@ -1746,8 +1762,9 @@ MidiRegionView::update_sustained (Note* ev, bool update_ghost_regions)
        }
 
        // Update color in case velocity has changed
-       ev->set_fill_color(ev->base_color());
-       ev->set_outline_color(ev->calculate_outline(ev->base_color(), ev->selected()));
+       //const uint32_t base_col = ev->base_color();
+       //ev->set_fill_color(base_col);
+       //ev->set_outline_color(ev->calculate_outline(base_col, ev->selected()));
 
        if (update_ghost_regions) {
                for (std::vector<GhostRegion*>::iterator i = ghosts.begin(); i != ghosts.end(); ++i) {
@@ -1764,7 +1781,8 @@ MidiRegionView::update_hit (Hit* ev, bool update_ghost_regions)
 {
        boost::shared_ptr<NoteType> note = ev->note();
 
-       const framepos_t note_start_frames = source_beats_to_region_frames(note->time());
+       const framepos_t note_start_frames = trackview.session()->tempo_map().frame_at_beat (_region->beat() - midi_region()->start_beats().to_double()
+                                                                                                + note->time().to_double()) - _region->position();
        const double x = trackview.editor().sample_to_pixel(note_start_frames);
        const double diamond_size = std::max(1., floor(midi_stream_view()->note_height()) - 2.);
        const double y = 1.5 + floor(midi_stream_view()->note_to_y(note->note())) + diamond_size * .5;
@@ -1872,7 +1890,8 @@ MidiRegionView::step_add_note (uint8_t channel, uint8_t number, uint8_t velocity
        framepos_t region_end = _region->last_frame();
 
        if (end_frame > region_end) {
-               _region->set_length (end_frame - _region->position());
+               /* XX sets length in beats from audio space. make musical */
+               _region->set_length (end_frame - _region->position(), 0);
        }
 
        MidiTimeAxisView* const mtv = dynamic_cast<MidiTimeAxisView*>(&trackview);
@@ -1884,7 +1903,7 @@ MidiRegionView::step_add_note (uint8_t channel, uint8_t number, uint8_t velocity
 
        start_note_diff_command (_("step add"));
 
-       clear_selection ();
+       clear_editor_note_selection ();
        note_diff_add_note (new_note, true, false);
 
        apply_diff();
@@ -2134,65 +2153,50 @@ MidiRegionView::delete_note (boost::shared_ptr<NoteType> n)
 }
 
 void
-MidiRegionView::clear_selection_except (NoteBase* ev, bool signal)
+MidiRegionView::clear_editor_note_selection ()
 {
-       for (Selection::iterator i = _selection.begin(); i != _selection.end(); ) {
-               if ((*i) != ev) {
-                       Selection::iterator tmp = i;
-                       ++tmp;
+       DEBUG_TRACE(DEBUG::Selection, "MRV::clear_editor_note_selection\n");
+       PublicEditor& editor(trackview.editor());
+       editor.get_selection().clear_midi_notes();
+}
 
-                       (*i)->set_selected (false);
-                       (*i)->hide_velocity ();
-                       _selection.erase (i);
+void
+MidiRegionView::clear_selection ()
+{
+       clear_selection_internal();
+       PublicEditor& editor(trackview.editor());
+       editor.get_selection().remove(this);
+}
 
-                       i = tmp;
-               } else {
-                       ++i;
-               }
+void
+MidiRegionView::clear_selection_internal ()
+{
+       DEBUG_TRACE(DEBUG::Selection, "MRV::clear_selection_internal\n");
+
+       for (Selection::iterator i = _selection.begin(); i != _selection.end(); ++i) {
+               (*i)->set_selected(false);
+               (*i)->hide_velocity();
        }
+       _selection.clear();
 
-       if (!ev && _entered) {
+       if (_entered) {
                // Clearing selection entirely, ungrab keyboard
                Keyboard::magic_widget_drop_focus();
                _grabbed_keyboard = false;
        }
-
-       /* this does not change the status of this regionview w.r.t the editor
-          selection.
-       */
-
-       if (signal) {
-               SelectionCleared (this); /* EMIT SIGNAL */
-       }
 }
 
 void
 MidiRegionView::unique_select(NoteBase* ev)
 {
-       const bool selection_was_empty = _selection.empty();
-
-       clear_selection_except (ev);
-
-       /* don't bother with checking to see if we should remove this
-          regionview from the editor selection, since we're about to add
-          another note, and thus put/keep this regionview in the editor
-          selection anyway.
-       */
-
-       if (!ev->selected()) {
-               add_to_selection (ev);
-               if (selection_was_empty && _entered) {
-                       // Grab keyboard for moving notes with arrow keys
-                       Keyboard::magic_widget_grab_focus();
-                       _grabbed_keyboard = true;
-               }
-       }
+       clear_editor_note_selection();
+       add_to_selection(ev);
 }
 
 void
 MidiRegionView::select_all_notes ()
 {
-       clear_selection ();
+       clear_editor_note_selection ();
 
        for (Events::iterator i = _events.begin(); i != _events.end(); ++i) {
                add_to_selection (*i);
@@ -2202,7 +2206,7 @@ MidiRegionView::select_all_notes ()
 void
 MidiRegionView::select_range (framepos_t start, framepos_t end)
 {
-       clear_selection ();
+       clear_editor_note_selection ();
 
        for (Events::iterator i = _events.begin(); i != _events.end(); ++i) {
                framepos_t t = source_beats_to_absolute_frames((*i)->note()->time());
@@ -2267,7 +2271,7 @@ MidiRegionView::select_matching_notes (uint8_t notenum, uint16_t channel_mask, b
        }
 
        if (!add) {
-               clear_selection ();
+               clear_editor_note_selection ();
 
                if (!extend && (low_note == high_note) && (high_note == notenum)) {
                        /* only note previously selected is the one we are
@@ -2342,11 +2346,8 @@ void
 MidiRegionView::note_selected (NoteBase* ev, bool add, bool extend)
 {
        if (!add) {
-               clear_selection_except (ev);
-               if (!_selection.empty()) {
-                       PublicEditor& editor (trackview.editor());
-                       editor.get_selection().add (this);
-               }
+               clear_editor_note_selection();
+               add_to_selection (ev);
        }
 
        if (!extend) {
@@ -2749,6 +2750,7 @@ MidiRegionView::begin_resizing (bool /*at_front*/)
 void
 MidiRegionView::update_resizing (NoteBase* primary, bool at_front, double delta_x, bool relative, double snap_delta, bool with_snap)
 {
+       TempoMap& tmap (trackview.session()->tempo_map());
        bool cursor_set = false;
        bool const ensure_snap = trackview.editor().snap_mode () != SnapMagnetic;
 
@@ -2810,9 +2812,19 @@ MidiRegionView::update_resizing (NoteBase* primary, bool at_front, double delta_
                                sign = -1;
                        }
 
-                       const double  snapped_x = (with_snap ? snap_pixel_to_sample (current_x, ensure_snap) : trackview.editor ().pixel_to_sample (current_x));
-                       Evoral::Beats beats     = region_frames_to_region_beats (snapped_x);
-                       Evoral::Beats len       = Evoral::Beats();
+                       double  snapped_x;
+                       uint32_t divisions = 0;
+
+                       if (with_snap) {
+                               snapped_x = snap_pixel_to_sample (current_x, ensure_snap);
+                               divisions = trackview.editor().get_grid_music_divisions (0);
+                       } else {
+                               snapped_x = trackview.editor ().pixel_to_sample (current_x);
+                       }
+                       const Evoral::Beats beats = Evoral::Beats (tmap.exact_beat_at_frame (snapped_x + midi_region()->position(), divisions)
+                                                                    - midi_region()->beat()) + midi_region()->start_beats();
+
+                       Evoral::Beats len         = Evoral::Beats();
 
                        if (at_front) {
                                if (beats < canvas_note->note()->end_time()) {
@@ -2845,6 +2857,7 @@ void
 MidiRegionView::commit_resizing (NoteBase* primary, bool at_front, double delta_x, bool relative, double snap_delta, bool with_snap)
 {
        _note_diff_command = _model->new_note_diff_command (_("resize notes"));
+       TempoMap& tmap (trackview.session()->tempo_map());
 
        /* XX why doesn't snap_pixel_to_sample() handle this properly? */
        bool const ensure_snap = trackview.editor().snap_mode () != SnapMagnetic;
@@ -2892,16 +2905,19 @@ MidiRegionView::commit_resizing (NoteBase* primary, bool at_front, double delta_
                        sign = -1;
                }
 
+               uint32_t divisions = 0;
                /* Convert the new x position to a frame within the source */
                framepos_t current_fr;
                if (with_snap) {
-                       current_fr = snap_pixel_to_sample (current_x, ensure_snap) + _region->start ();
+                       current_fr = snap_pixel_to_sample (current_x, ensure_snap);
+                       divisions = trackview.editor().get_grid_music_divisions (0);
                } else {
-                       current_fr = trackview.editor().pixel_to_sample (current_x) + _region->start ();
+                       current_fr = trackview.editor().pixel_to_sample (current_x);
                }
 
                /* and then to beats */
-               const Evoral::Beats x_beats = region_frames_to_region_beats (current_fr);
+               const Evoral::Beats x_beats = Evoral::Beats (tmap.exact_beat_at_frame (current_fr + midi_region()->position(), divisions)
+                                                            - midi_region()->beat()) + midi_region()->start_beats();
 
                if (at_front && x_beats < canvas_note->note()->end_time()) {
                        note_diff_add_change (canvas_note, MidiModel::NoteDiffCommand::StartTime, x_beats - (sign * snap_delta_beats));
@@ -3250,7 +3266,7 @@ MidiRegionView::nudge_notes (bool forward, bool fine)
        if (!fine) {
 
                /* non-fine, move by 1 bar regardless of snap */
-               delta = Evoral::Beats(trackview.session()->tempo_map().meter_at(ref_point).divisions_per_bar());
+               delta = Evoral::Beats(trackview.session()->tempo_map().meter_at_frame (ref_point).divisions_per_bar());
 
        } else if (trackview.editor().snap_mode() == Editing::SnapOff) {
 
@@ -3317,13 +3333,22 @@ MidiRegionView::change_channel(uint8_t channel)
 void
 MidiRegionView::note_entered(NoteBase* ev)
 {
+       _note_entered = true;
+
        Editor* editor = dynamic_cast<Editor*>(&trackview.editor());
 
        if (_mouse_state == SelectTouchDragging) {
+
                note_selected (ev, true);
+
        } else if (editor->current_mouse_mode() == MouseContent) {
+
+               remove_ghost_note ();
                show_verbose_cursor (ev->note ());
+
        } else if (editor->current_mouse_mode() == MouseDraw) {
+
+               remove_ghost_note ();
                show_verbose_cursor (ev->note ());
        }
 }
@@ -3331,6 +3356,8 @@ MidiRegionView::note_entered(NoteBase* ev)
 void
 MidiRegionView::note_left (NoteBase*)
 {
+       _note_entered = false;
+
        for (Selection::iterator i = _selection.begin(); i != _selection.end(); ++i) {
                (*i)->hide_velocity ();
        }
@@ -3342,9 +3369,8 @@ void
 MidiRegionView::patch_entered (PatchChange* p)
 {
        ostringstream s;
-       /* XXX should get patch name if we can */
        s << _("Bank ") << (p->patch()->bank() + MIDI_BP_ZERO) << '\n'
-         << _("Program ") << ((int) p->patch()->program()) + MIDI_BP_ZERO << '\n'
+         << instrument_info().get_patch_name_without (p->patch()->bank(), p->patch()->program(), p->patch()->channel()) << '\n'
          << _("Channel ") << ((int) p->patch()->channel() + 1);
        show_verbose_cursor (s.str(), 10, 20);
        p->item().grab_focus();
@@ -3498,7 +3524,7 @@ MidiRegionView::selection_as_cut_buffer () const
 
 /** This method handles undo */
 bool
-MidiRegionView::paste (framepos_t pos, const ::Selection& selection, PasteContext& ctx)
+MidiRegionView::paste (framepos_t pos, const ::Selection& selection, PasteContext& ctx, const int32_t sub_num)
 {
        bool commit = false;
        // Paste notes, if available
@@ -3513,7 +3539,7 @@ MidiRegionView::paste (framepos_t pos, const ::Selection& selection, PasteContex
        typedef RouteTimeAxisView::AutomationTracks ATracks;
        const ATracks& atracks = midi_view()->automation_tracks();
        for (ATracks::const_iterator a = atracks.begin(); a != atracks.end(); ++a) {
-               if (a->second->paste(pos, selection, ctx)) {
+               if (a->second->paste(pos, selection, ctx, sub_num)) {
                        commit = true;
                }
        }
@@ -3549,7 +3575,7 @@ MidiRegionView::paste_internal (framepos_t pos, unsigned paste_count, float time
                                                       duration, pos, _region->position(),
                                                       pos_beats));
 
-       clear_selection ();
+       clear_editor_note_selection ();
 
        for (int n = 0; n < (int) times; ++n) {
 
@@ -3576,7 +3602,8 @@ MidiRegionView::paste_internal (framepos_t pos, unsigned paste_count, float time
                DEBUG_TRACE (DEBUG::CutNPaste, string_compose ("Paste extended region from %1 to %2\n", region_end, end_frame));
 
                _region->clear_changes ();
-               _region->set_length (end_frame - _region->position());
+               /* we probably need to get the snap modifier somehow to make this correct for non-musical use */
+               _region->set_length (end_frame - _region->position(), trackview.editor().get_grid_music_divisions (0));
                trackview.session()->add_command (new StatefulDiffCommand (_region));
        }
 
@@ -3838,9 +3865,6 @@ void
 MidiRegionView::enable_display (bool yn)
 {
        RegionView::enable_display (yn);
-       if (yn) {
-               redisplay_model ();
-       }
 }
 
 void
@@ -3886,7 +3910,9 @@ MidiRegionView::set_step_edit_cursor_width (Evoral::Beats beats)
        _step_edit_cursor_width = beats;
 
        if (_step_edit_cursor) {
-               _step_edit_cursor->set_x1 (_step_edit_cursor->x0() + trackview.editor().sample_to_pixel (region_beats_to_region_frames (beats)));
+               _step_edit_cursor->set_x1 (_step_edit_cursor->x0() + trackview.editor().sample_to_pixel (
+                                                  region_beats_to_region_frames (_step_edit_cursor_position + beats)
+                                                  - region_beats_to_region_frames (_step_edit_cursor_position)));
        }
 }
 
@@ -3973,6 +3999,10 @@ MidiRegionView::trim_front_ending ()
                /* Trim drag made start time -ve; fix this */
                midi_region()->fix_negative_start ();
        }
+       /* until _start is modified on the fly during front trim,
+          we have to redisplay the model when a start trim has finished.
+       */
+       enable_display (true);
 }
 
 void
@@ -4008,11 +4038,10 @@ MidiRegionView::delete_sysex (SysEx* /*sysex*/)
        // display_sysexes();
 }
 
-void
-MidiRegionView::show_verbose_cursor (boost::shared_ptr<NoteType> n) const
+std::string
+MidiRegionView::get_note_name (boost::shared_ptr<NoteType> n, uint8_t note_value) const
 {
        using namespace MIDI::Name;
-
        std::string name;
 
        MidiTimeAxisView* mtv = dynamic_cast<MidiTimeAxisView*>(&trackview);
@@ -4025,19 +4054,36 @@ MidiRegionView::show_verbose_cursor (boost::shared_ptr<NoteType> n) const
                                                       n->channel(),
                                                       patch_key.bank(),
                                                       patch_key.program(),
-                                                      n->note());
+                                                      note_value);
                }
-               mtv->set_note_highlight (n->note());
        }
 
        char buf[128];
        snprintf (buf, sizeof (buf), "%d %s\nCh %d Vel %d",
-                 (int) n->note (),
-                 name.empty() ? Evoral::midi_note_name (n->note()).c_str() : name.c_str(),
+                 (int) note_value,
+                 name.empty() ? ParameterDescriptor::midi_note_name (note_value).c_str() : name.c_str(),
                  (int) n->channel() + 1,
                  (int) n->velocity());
 
-       show_verbose_cursor(buf, 10, 20);
+       return buf;
+}
+
+void
+MidiRegionView::show_verbose_cursor_for_new_note_value(boost::shared_ptr<NoteType> current_note,
+                                                       uint8_t new_value) const
+{
+       MidiTimeAxisView* mtv = dynamic_cast<MidiTimeAxisView*>(&trackview);
+       if (mtv) {
+               mtv->set_note_highlight (new_value);
+       }
+
+       show_verbose_cursor(get_note_name(current_note, new_value), 10, 20);
+}
+
+void
+MidiRegionView::show_verbose_cursor (boost::shared_ptr<NoteType> n) const
+{
+       show_verbose_cursor_for_new_note_value(n, n->note());
 }
 
 void
@@ -4083,10 +4129,10 @@ framepos_t
 MidiRegionView::snap_frame_to_grid_underneath (framepos_t p, framecnt_t& grid_frames) const
 {
        PublicEditor& editor = trackview.editor ();
-
        const Evoral::Beats grid_beats = get_grid_beats(p);
+       const Evoral::Beats p_beat = max (Evoral::Beats(), region_frames_to_region_beats (p));
 
-       grid_frames = region_beats_to_region_frames (grid_beats);
+       grid_frames = region_beats_to_region_frames (p_beat + grid_beats) - region_beats_to_region_frames (p_beat);
 
        /* Hack so that we always snap to the note that we are over, instead of snapping
           to the next one if we're more than halfway through the one we're over.
@@ -4098,20 +4144,6 @@ MidiRegionView::snap_frame_to_grid_underneath (framepos_t p, framecnt_t& grid_fr
        return snap_frame_to_frame (p);
 }
 
-/** Called when the selection has been cleared in any MidiRegionView.
- *  @param rv MidiRegionView that the selection was cleared in.
- */
-void
-MidiRegionView::selection_cleared (MidiRegionView* rv)
-{
-       if (rv == this) {
-               return;
-       }
-
-       /* Clear our selection in sympathy; but don't signal the fact */
-       clear_selection (false);
-}
-
 ChannelMode
 MidiRegionView::get_channel_mode () const
 {
@@ -4132,7 +4164,7 @@ MidiRegionView::get_grid_beats(framepos_t pos) const
 {
        PublicEditor& editor  = trackview.editor();
        bool          success = false;
-       Evoral::Beats beats   = editor.get_grid_type_as_beats(success, pos);
+       Evoral::Beats beats   = editor.get_grid_type_as_beats (success, pos);
        if (!success) {
                beats = Evoral::Beats(1);
        }