Fix mismatched note resize drag cursor and behaviour.
[ardour.git] / gtk2_ardour / step_entry.cc
index 1fd6b37cfb0edce68d7f53f73d935fd10569e26b..ac13770a2955a2f05713d6d5201c10235bf220b8 100644 (file)
@@ -19,7 +19,6 @@
 
 #include <iostream>
 
-#include "pbd/filesystem.h"
 #include "pbd/file_utils.h"
 
 #include "gtkmm2ext/keyboard.h"
@@ -43,21 +42,22 @@ using namespace Glib;
 using namespace Gtkmm2ext;
 using namespace PBD;
 using namespace ARDOUR;
+using namespace ARDOUR_UI_UTILS;
 
 static void
-_note_off_event_handler (GtkWidget* widget, int note, gpointer arg)
+_note_off_event_handler (GtkWidget* /*widget*/, int note, gpointer arg)
 {
        ((StepEntry*)arg)->note_off_event_handler (note);
 }
 
 static void
-_rest_event_handler (GtkWidget* widget, gpointer arg)
+_rest_event_handler (GtkWidget* /*widget*/, gpointer arg)
 {
        ((StepEntry*)arg)->rest_event_handler ();
 }
 
 StepEntry::StepEntry (StepEditor& seditor)
-       : ArdourDialog (string_compose (_("Step Entry: %1"), seditor.name()))
+       : ArdourWindow (string_compose (_("Step Entry: %1"), seditor.name()))
         , _current_note_length (1.0)
         , _current_note_velocity (64)
        , triplet_button ("3")
@@ -69,9 +69,9 @@ StepEntry::StepEntry (StepEditor& seditor)
        , rest_button (_("rest"))
        , grid_rest_button (_("g-rest"))
        , back_button (_("back"))
-       , channel_adjustment (1, 1, 16, 1, 4) 
+       , channel_adjustment (1, 1, 16, 1, 4)
        , channel_spinner (channel_adjustment)
-        , octave_adjustment (4, 1, 11, 1, 4) // start in octave 4
+        , octave_adjustment (4, 0, 10, 1, 4) // start in octave 4
         , octave_spinner (octave_adjustment)
         , length_divisor_adjustment (1.0, 1.0, 128, 1.0, 4.0)
         , length_divisor_spinner (length_divisor_adjustment)
@@ -97,7 +97,7 @@ StepEntry::StepEntry (StepEditor& seditor)
        */
 
        uint16_t chn_mask = se->channel_selector().get_selected_channels();
-        
+
        for (uint32_t i = 0; i < 16; ++i) {
                if (chn_mask & (1<<i)) {
                        channel_adjustment.set_value (i+1);
@@ -279,8 +279,8 @@ StepEntry::StepEntry (StepEditor& seditor)
        ARDOUR_UI::instance()->set_tip (&velocity_mp_button, _("Set volume (velocity) to mezzo-piano"), "");
        ARDOUR_UI::instance()->set_tip (&velocity_mf_button, _("Set volume (velocity) to mezzo-forte"), "");
        ARDOUR_UI::instance()->set_tip (&velocity_f_button, _("Set volume (velocity) to forte"), "");
-       ARDOUR_UI::instance()->set_tip (&velocity_ff_button, _("Set volume (velocity) to forteissimo"), "");
-       ARDOUR_UI::instance()->set_tip (&velocity_fff_button, _("Set volume (velocity) to forteississimo"), "");
+       ARDOUR_UI::instance()->set_tip (&velocity_ff_button, _("Set volume (velocity) to fortissimo"), "");
+       ARDOUR_UI::instance()->set_tip (&velocity_fff_button, _("Set volume (velocity) to fortississimo"), "");
 
        note_velocity_box.pack_start (velocity_ppp_button, false, false);
        note_velocity_box.pack_start (velocity_pp_button, false, false);
@@ -376,7 +376,7 @@ StepEntry::StepEntry (StepEditor& seditor)
         act = myactions.find_action ("StepEditing/sustain");
         gtk_activatable_set_use_action_appearance (GTK_ACTIVATABLE (sustain_button.gobj()), false);
         gtk_activatable_set_related_action (GTK_ACTIVATABLE (sustain_button.gobj()), act->gobj());
-                
+
        upper_box.set_spacing (6);
        upper_box.pack_start (chord_button, false, false);
        upper_box.pack_start (note_length_box, false, false, 12);
@@ -445,7 +445,7 @@ StepEntry::StepEntry (StepEditor& seditor)
 
        g_signal_connect(G_OBJECT(_piano), "note-off", G_CALLBACK(_note_off_event_handler), this);
        g_signal_connect(G_OBJECT(_piano), "rest", G_CALLBACK(_rest_event_handler), this);
-        
+
        program_button.signal_clicked().connect (sigc::mem_fun (*this, &StepEntry::program_click));
        bank_button.signal_clicked().connect (sigc::mem_fun (*this, &StepEntry::bank_click));
        beat_resync_button.signal_clicked().connect (sigc::mem_fun (*this, &StepEntry::beat_resync_click));
@@ -458,7 +458,19 @@ StepEntry::StepEntry (StepEditor& seditor)
        packer.pack_start (*piano, false, false);
        packer.show_all ();
 
-       get_vbox()->add (packer);
+       add (packer);
+
+       /* initial settings: quarter note and mezzo forte */
+
+        act = myactions.find_action ("StepEditing/note-length-quarter");
+       RefPtr<RadioAction> r = RefPtr<RadioAction>::cast_dynamic (act);
+       assert (r);
+       r->set_active (true);
+
+        act = myactions.find_action ("StepEditing/note-velocity-mf");
+       r = RefPtr<RadioAction>::cast_dynamic (act);
+       assert (r);
+       r->set_active (true);
 }
 
 StepEntry::~StepEntry()
@@ -483,11 +495,11 @@ StepEntry::on_key_press_event (GdkEventKey* ev)
         /* focus widget gets first shot, then bindings, otherwise
            forward to main window
         */
-        
+
        if (!gtk_window_propagate_key_event (GTK_WINDOW(gobj()), ev)) {
                 KeyboardKey k (ev->state, ev->keyval);
 
-                if (bindings.activate (k, KeyboardKey::Press)) {
+                if (bindings.activate (k, Bindings::Press)) {
                         return true;
                 }
        }
@@ -501,11 +513,11 @@ StepEntry::on_key_release_event (GdkEventKey* ev)
        if (!gtk_window_propagate_key_event (GTK_WINDOW(gobj()), ev)) {
                 KeyboardKey k (ev->state, ev->keyval);
 
-                if (bindings.activate (k, KeyboardKey::Release)) {
+                if (bindings.activate (k, Bindings::Release)) {
                         return true;
                 }
        }
-        
+
         /* don't forward releases */
 
         return true;
@@ -520,8 +532,8 @@ StepEntry::rest_event_handler ()
 Evoral::MusicalTime
 StepEntry::note_length ()
 {
-        Evoral::MusicalTime base_time = 1.0 / (Evoral::MusicalTime) length_divisor_adjustment.get_value();
-        
+        Evoral::MusicalTime base_time = 4.0 / (Evoral::MusicalTime) length_divisor_adjustment.get_value();
+
         RefPtr<Action> act = myactions.find_action ("StepEditing/toggle-triplet");
         RefPtr<ToggleAction> tact = RefPtr<ToggleAction>::cast_dynamic (act);
         bool triplets = tact->get_active ();
@@ -546,7 +558,7 @@ StepEntry::note_velocity () const
         return (Evoral::MusicalTime) velocity_adjustment.get_value();
 }
 
-uint8_t 
+uint8_t
 StepEntry::note_channel() const
 {
        return channel_adjustment.get_value() - 1;
@@ -562,7 +574,7 @@ StepEntry::note_off_event_handler (int note)
 void
 StepEntry::on_show ()
 {
-       ArdourDialog::on_show ();
+       ArdourWindow::on_show ();
        //piano->grab_focus ();
 }
 
@@ -626,13 +638,14 @@ StepEntry::register_actions ()
        myactions.register_action ("StepEditing", "octave-9", _("Switch to the 10th octave"), sigc::mem_fun (*this, &StepEntry::octave_9));
        myactions.register_action ("StepEditing", "octave-10", _("Switch to the 11th octave"), sigc::mem_fun (*this, &StepEntry::octave_10));
 
-
         RadioAction::Group note_length_group;
 
-        myactions.register_radio_action ("StepEditing", note_length_group, "note-length-whole", 
+        myactions.register_radio_action ("StepEditing", note_length_group, "note-length-whole",
                                          _("Set Note Length to Whole"), sigc::mem_fun (*this, &StepEntry::note_length_change), 1);
-        myactions.register_radio_action ("StepEditing", note_length_group, "note-length-half", 
+        myactions.register_radio_action ("StepEditing", note_length_group, "note-length-half",
                                          _("Set Note Length to 1/2"), sigc::mem_fun (*this, &StepEntry::note_length_change), 2);
+        myactions.register_radio_action ("StepEditing", note_length_group, "note-length-third",
+                                         _("Set Note Length to 1/3"), sigc::mem_fun (*this, &StepEntry::note_length_change), 3);
         myactions.register_radio_action ("StepEditing", note_length_group, "note-length-quarter",
                                          _("Set Note Length to 1/4"), sigc::mem_fun (*this, &StepEntry::note_length_change), 4);
         myactions.register_radio_action ("StepEditing", note_length_group, "note-length-eighth",
@@ -668,13 +681,13 @@ StepEntry::register_actions ()
 
         RadioAction::Group dot_group;
 
-        myactions.register_radio_action ("StepEditing", dot_group, "no-dotted", _("No Dotted Notes"), 
+        myactions.register_radio_action ("StepEditing", dot_group, "no-dotted", _("No Dotted Notes"),
                                          sigc::mem_fun (*this, &StepEntry::dot_change), 0);
-        myactions.register_radio_action ("StepEditing", dot_group, "toggle-dotted", _("Toggled Dotted Notes"), 
+        myactions.register_radio_action ("StepEditing", dot_group, "toggle-dotted", _("Toggled Dotted Notes"),
                                          sigc::mem_fun (*this, &StepEntry::dot_change), 1);
-        myactions.register_radio_action ("StepEditing", dot_group, "toggle-double-dotted", _("Toggled Double-Dotted Notes"), 
+        myactions.register_radio_action ("StepEditing", dot_group, "toggle-double-dotted", _("Toggled Double-Dotted Notes"),
                                          sigc::mem_fun (*this, &StepEntry::dot_change), 2);
-        myactions.register_radio_action ("StepEditing", dot_group, "toggle-triple-dotted", _("Toggled Triple-Dotted Notes"), 
+        myactions.register_radio_action ("StepEditing", dot_group, "toggle-triple-dotted", _("Toggled Triple-Dotted Notes"),
                                          sigc::mem_fun (*this, &StepEntry::dot_change), 3);
 
         myactions.register_toggle_action ("StepEditing", "toggle-chord", _("Toggle Chord Entry"),
@@ -692,15 +705,13 @@ void
 StepEntry::load_bindings ()
 {
         /* XXX move this to a better place */
-        KeyboardKey::set_ignored_state (GDK_LOCK_MASK|GDK_MOD2_MASK|GDK_MOD3_MASK);
 
         bindings.set_action_map (myactions);
 
-       sys::path binding_file;
-       SearchPath spath = ardour_search_path() + user_config_directory() + system_config_search_path();
+       std::string binding_file;
 
-       if (find_file_in_search_path (spath, "step_editing.bindings", binding_file)) {
-                bindings.load (binding_file.to_string());
+       if (find_file (ardour_config_search_path(), "step_editing.bindings", binding_file)) {
+                bindings.load (binding_file);
         }
 }
 
@@ -742,11 +753,11 @@ StepEntry::dot_value_change ()
         for (vector<const char*>::iterator i = dot_actions.begin(); i != dot_actions.end(); ++i) {
 
                 act = myactions.find_action (*i);
-                
+
                 if (act) {
                         ract = RefPtr<RadioAction>::cast_dynamic (act);
 
-                        if (ract) { 
+                        if (ract) {
                                 if (ract->property_value() == val) {
                                         ract->set_active (true);
                                         inconsistent = false;
@@ -755,7 +766,7 @@ StepEntry::dot_value_change ()
                         }
                 }
         }
-        
+
         dot1_button.set_inconsistent (inconsistent);
         dot2_button.set_inconsistent (inconsistent);
         dot3_button.set_inconsistent (inconsistent);
@@ -790,6 +801,10 @@ StepEntry::insert_grid_rest ()
 void
 StepEntry::insert_note (uint8_t note)
 {
+        if (note > 127) {
+                return;
+        }
+
        se->step_add_note (note_channel(), note, note_velocity(), note_length());
 }
 void
@@ -865,7 +880,7 @@ StepEntry::note_length_change (GtkAction* act)
            becaome "active". so ... only bother to actually change the value when this
            is called for the "active" action.
         */
-        
+
         if (gtk_toggle_action_get_active (GTK_TOGGLE_ACTION(act))) {
                 gint v = gtk_radio_action_get_current_value (GTK_RADIO_ACTION (act));
                 length_divisor_adjustment.set_value (v);
@@ -882,7 +897,7 @@ StepEntry::note_velocity_change (GtkAction* act)
            becaome "active". so ... only bother to actually change the value when this
            is called for the "active" action.
         */
-        
+
         if (gtk_toggle_action_get_active (GTK_TOGGLE_ACTION(act))) {
                 gint v = gtk_radio_action_get_current_value (GTK_RADIO_ACTION (act));
                 velocity_adjustment.set_value (v);
@@ -910,11 +925,11 @@ StepEntry::velocity_value_change ()
         for (vector<const char*>::iterator i = velocity_actions.begin(); i != velocity_actions.end(); ++i) {
 
                 act = myactions.find_action (*i);
-                
+
                 if (act) {
                         ract = RefPtr<RadioAction>::cast_dynamic (act);
 
-                        if (ract) { 
+                        if (ract) {
                                 if (ract->property_value() == val) {
                                         ract->set_active (true);
                                         inconsistent = false;
@@ -954,11 +969,11 @@ StepEntry::length_value_change ()
         for (vector<const char*>::iterator i = length_actions.begin(); i != length_actions.end(); ++i) {
 
                 act = myactions.find_action (*i);
-                
+
                 if (act) {
                         ract = RefPtr<RadioAction>::cast_dynamic (act);
 
-                        if (ract) { 
+                        if (ract) {
                                 if (ract->property_value() == val) {
                                         ract->set_active (true);
                                         inconsistent = false;
@@ -984,7 +999,7 @@ StepEntry::radio_button_press (GdkEventButton* ev)
 {
         if (ev->button == 1) {
                 return true;
-        } 
+        }
 
         return false;
 }
@@ -994,13 +1009,13 @@ StepEntry::radio_button_release (GdkEventButton* ev, RadioButton* btn, int v)
 {
         if (ev->button == 1) {
                 GtkAction* act = gtk_activatable_get_related_action (GTK_ACTIVATABLE (btn->gobj()));
-                
+
                 if (act) {
                         gtk_radio_action_set_current_value (GTK_RADIO_ACTION(act), v);
                 }
-                
+
                 return true;
-        } 
+        }
 
         return false;
 }
@@ -1097,7 +1112,7 @@ StepEntry::next_note_velocity ()
         } else if (l < 127) {
                 l = 127;
         }
-        
+
         velocity_adjustment.set_value (l);
 }
 
@@ -1123,7 +1138,7 @@ StepEntry::prev_note_velocity ()
         } else {
                 l = 1;
         }
-        
+
         velocity_adjustment.set_value (l);
 }