enough with umpteen "i18n.h" files. Consolidate on pbd/i18n.h
[ardour.git] / gtk2_ardour / keyboard.cc
index 17390b8b4db79020fb24d82c5e855a55eaa95d4d..6341071451f94d3b5e99b72169e8405652ff53e2 100644 (file)
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2001 Paul Davis 
+    Copyright (C) 2001 Paul Davis
 
     This program is free software; you can redistribute it and/or modify
     it under the terms of the GNU General Public License as published by
     along with this program; if not, write to the Free Software
     Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
 
-    $Id$
 */
 
-#include "ardour_ui.h"
-
-#include <algorithm>
-#include <fstream>
+#include "pbd/convert.h"
+#include "pbd/error.h"
+#include "pbd/file_utils.h"
+#include "pbd/basename.h"
 
-#include <ctype.h>
-
-#include <X11/keysymdef.h>
-#include <gdk/gdkx.h>
-#include <gdk/gdkkeysyms.h>
-#include <pbd/error.h>
+#include "ardour/filesystem_paths.h"
 
+#include "ardour_ui.h"
+#include "public_editor.h"
 #include "keyboard.h"
-#include "gui_thread.h"
+#include "opts.h"
+#include "ui_config.h"
 
-#include "i18n.h"
+#include "pbd/i18n.h"
 
+using namespace std;
+using namespace Gtk;
 using namespace PBD;
+using namespace ARDOUR;
+using Gtkmm2ext::Keyboard;
 
-#define KBD_DEBUG 1
-bool debug_keyboard = false;
-
-guint Keyboard::edit_but = 3;
-guint Keyboard::edit_mod = GDK_CONTROL_MASK;
-guint Keyboard::delete_but = 3;
-guint Keyboard::delete_mod = GDK_SHIFT_MASK;
-guint Keyboard::snap_mod = GDK_MOD3_MASK;
-
-uint32_t Keyboard::Control = GDK_CONTROL_MASK;
-uint32_t Keyboard::Shift = GDK_SHIFT_MASK;
-uint32_t Keyboard::Alt = GDK_MOD1_MASK;
-uint32_t Keyboard::Meta = GDK_MOD2_MASK;
-
-Keyboard* Keyboard::_the_keyboard = 0;
+#ifdef __APPLE__
+guint ArdourKeyboard::constraint_mod = Keyboard::PrimaryModifier;
+#else
+guint ArdourKeyboard::constraint_mod = Keyboard::SecondaryModifier;
+#endif
+guint ArdourKeyboard::trim_contents_mod = Keyboard::PrimaryModifier;
+guint ArdourKeyboard::trim_overlap_mod = Keyboard::TertiaryModifier;
+guint ArdourKeyboard::trim_anchored_mod = Keyboard::TertiaryModifier;
+guint ArdourKeyboard::fine_adjust_mod = Keyboard::SecondaryModifier;
+guint ArdourKeyboard::push_points_mod = Keyboard::PrimaryModifier;
+guint ArdourKeyboard::note_size_relative_mod = Keyboard::PrimaryModifier;
 
-/* set this to initially contain the modifiers we care about, then track changes in ::set_edit_modifier() etc. */
+void
+ArdourKeyboard::find_bindings_files (map<string,string>& files)
+{
+       vector<std::string> found;
+       Searchpath spath = ardour_config_search_path();
 
-GdkModifierType Keyboard::RelevantModifierKeyMask = 
-                               GdkModifierType (GDK_SHIFT_MASK|GDK_CONTROL_MASK|GDK_MOD1_MASK|GDK_MOD3_MASK);
+       find_files_matching_pattern (found, spath, string_compose ("*%1", Keyboard::binding_filename_suffix));
 
+       if (found.empty()) {
+               return;
+       }
 
-Keyboard::Keyboard ()
-{
-       if (_the_keyboard == 0) {
-               _the_keyboard = this;
+       for (vector<std::string>::iterator x = found.begin(); x != found.end(); ++x) {
+               std::string path(*x);
+               pair<string,string> namepath;
+               namepath.second = path;
+               namepath.first = PBD::basename_nosuffix (path);
+               files.insert (namepath);
        }
+}
 
-       collecting_prefix = false;
+void
+ArdourKeyboard::setup_keybindings ()
+{
+       using namespace ARDOUR_COMMAND_LINE;
+       string default_bindings = string_compose ("%1%2", UIConfiguration::instance().get_default_bindings(), Keyboard::binding_filename_suffix);
+       vector<string> strs;
 
-       get_modifier_masks ();
+       binding_files.clear ();
 
-       snooper_id = gtk_key_snooper_install (_snooper, (gpointer) this);
+       find_bindings_files (binding_files);
 
-       XMLNode* node = ARDOUR_UI::instance()->keyboard_settings();
-       set_state (*node);
-}
+       /* set up the per-user bindings path */
 
-Keyboard::~Keyboard ()
-{
-       gtk_key_snooper_remove (snooper_id);
-       delete [] modifier_masks;
-}
+       string lowercase_program_name = downcase (string(PROGRAM_NAME));
 
-XMLNode& 
-Keyboard::get_state (void)
-{
-       XMLNode* node = new XMLNode ("Keyboard");
-       char buf[32];
+       user_keybindings_path = Glib::build_filename (user_config_directory(), lowercase_program_name + binding_filename_suffix);
 
-       snprintf (buf, sizeof (buf), "%d", edit_but);
-       node->add_property ("edit-button", buf);
-       snprintf (buf, sizeof (buf), "%d", edit_mod);
-       node->add_property ("edit-modifier", buf);
-       snprintf (buf, sizeof (buf), "%d", delete_but);
-       node->add_property ("delete-button", buf);
-       snprintf (buf, sizeof (buf), "%d", delete_mod);
-       node->add_property ("delete-modifier", buf);
-       snprintf (buf, sizeof (buf), "%d", snap_mod);
-       node->add_property ("snap-modifier", buf);
+       if (Glib::file_test (user_keybindings_path, Glib::FILE_TEST_EXISTS)) {
+               std::pair<string,string> newpair;
+               newpair.first = _("your own");
+               newpair.second = user_keybindings_path;
+               binding_files.insert (newpair);
+       }
 
-       return *node;
-}
+       /* check to see if they gave a style name ("ergonomic") or an actual filename (*.bindings)
+       */
 
-int 
-Keyboard::set_state (const XMLNode& node)
-{
-       const XMLProperty* prop;
+       if (!keybindings_path.empty() && keybindings_path.find (binding_filename_suffix) == string::npos) {
 
-       if ((prop = node.property ("edit-button")) != 0) {
-               sscanf (prop->value().c_str(), "%d", &edit_but);
-       } 
+               // just a style name - allow user to
+               // specify the layout type.
 
-       if ((prop = node.property ("edit-modifier")) != 0) {
-               sscanf (prop->value().c_str(), "%d", &edit_mod);
-       } 
+               char* layout;
 
-       if ((prop = node.property ("delete-button")) != 0) {
-               sscanf (prop->value().c_str(), "%d", &delete_but);
-       } 
+               if ((layout = getenv ("ARDOUR_KEYBOARD_LAYOUT")) != 0 && layout[0] != '\0') {
 
-       if ((prop = node.property ("delete-modifier")) != 0) {
-               sscanf (prop->value().c_str(), "%d", &delete_mod);
-       } 
+                       /* user-specified keyboard layout */
 
-       if ((prop = node.property ("snap-modifier")) != 0) {
-               sscanf (prop->value().c_str(), "%d", &snap_mod);
-       } 
+                       keybindings_path += '-';
+                       keybindings_path += layout;
 
-       return 0;
-}
+               } else {
 
-gint
-Keyboard::_snooper (GtkWidget *widget, GdkEventKey *event, gpointer data)
-{
-       return ((Keyboard *) data)->snooper (widget, event);
-}
+                       /* default to US/ANSI - we have to pick something */
 
-gint
-Keyboard::snooper (GtkWidget *widget, GdkEventKey *event)
-{
-       uint32_t keyval;
+                       keybindings_path += "-us";
+               }
 
-#if KBD_DEBUG
-       if (debug_keyboard) {
-               cerr << "snoop widget " << widget << " key " << event->keyval << " type: " << event->type 
-                    << endl;
+               keybindings_path += binding_filename_suffix;
        }
-#endif
 
-       if (event->keyval == GDK_Shift_R) {
-               keyval = GDK_Shift_L;
+       if (keybindings_path.empty()) {
 
-       } else  if (event->keyval == GDK_Control_R) {
-               keyval = GDK_Control_L;
+               /* no path or binding name given: check the user one first */
 
-       } else {
-               keyval = event->keyval;
-       }
-               
-       if (event->type == GDK_KEY_PRESS) {
-               bool was_prefix = false;
-
-               if (collecting_prefix) {
-                       switch (keyval) {
-                       case GDK_0:
-                               current_prefix += '0';
-                               was_prefix = true;
-                               break;
-                       case GDK_1:
-                               current_prefix += '1';
-                               was_prefix = true;
-                               break;
-                       case GDK_2:
-                               current_prefix += '2';
-                               was_prefix = true;
-                               break;
-                       case GDK_3:
-                               current_prefix += '3';
-                               was_prefix = true;
-                               break;
-                       case GDK_4:
-                               current_prefix += '4';
-                               was_prefix = true;
-                               break;
-                       case GDK_5:
-                               current_prefix += '5';
-                               was_prefix = true;
-                               break;
-                       case GDK_6:
-                               current_prefix += '6';
-                               was_prefix = true;
-                               break;
-                       case GDK_7:
-                               current_prefix += '7';
-                               was_prefix = true;
-                               break;
-                       case GDK_8:
-                               current_prefix += '8';
-                               was_prefix = true;
-                               break;
-                       case GDK_9:
-                               current_prefix += '9';
-                               was_prefix = true;
-                               break;
-                       case GDK_period:
-                               current_prefix += '.';
-                               was_prefix = true;
-                               break;
-                       default:
-                               was_prefix = false;
-                               collecting_prefix = false;
-                               break;
-                       }
-               }
+               if (!Glib::file_test (user_keybindings_path, Glib::FILE_TEST_EXISTS)) {
 
-               if (find (state.begin(), state.end(), keyval) == state.end()) {
-                       state.push_back (keyval);
-                       sort (state.begin(), state.end());
-               }
-
-       } else if (event->type == GDK_KEY_RELEASE) {
+                       keybindings_path = "";
 
-               State::iterator i;
-               
-               if ((i = find (state.begin(), state.end(), keyval)) != state.end()) {
-                       state.erase (i);
-                       sort (state.begin(), state.end());
-               } 
+               } else {
 
+                       keybindings_path = user_keybindings_path;
+               }
        }
 
-       return false;
-}
+       /* if we still don't have a path at this point, use the default */
 
-bool
-Keyboard::key_is_down (uint32_t keyval)
-{
-       return find (state.begin(), state.end(), keyval) != state.end();
-}
+       if (keybindings_path.empty()) {
+               keybindings_path = default_bindings;
+       }
 
-Keyboard::State
-Keyboard::translate_key_name (const string& name)
+       cerr << "KP is " << keybindings_path << endl;
 
-{
-       string::size_type i;
-       string::size_type len;
-       bool at_end;
-       string::size_type hyphen;
-       string keyname;
-       string whatevers_left;
-       State result;
-       guint keycode;
-       
-       i = 0;
-       len = name.length();
-       at_end = (len == 0);
-
-       while (!at_end) {
-
-               whatevers_left = name.substr (i);
-
-               if ((hyphen = whatevers_left.find_first_of ('-')) == string::npos) {
-                       
-                        /* no hyphen, so use the whole thing */
-                       
-                       keyname = whatevers_left;
-                       at_end = true;
+       while (true) {
 
-               } else {
+               if (!Glib::path_is_absolute (keybindings_path)) {
 
-                       /* There is a hyphen. */
-                       
-                       if (hyphen == 0 && whatevers_left.length() == 1) {
-                               /* its the first and only character */
-                       
-                               keyname = "-";
-                               at_end = true;
+                       /* not absolute - look in the usual places */
+                       std::string keybindings_file;
 
-                       } else {
+                       if (!find_file (ardour_config_search_path(), keybindings_path, keybindings_file)) {
 
-                               /* use the text before the hypen */
-                               
-                               keyname = whatevers_left.substr (0, hyphen);
-                               
-                               if (hyphen == len - 1) {
-                                       at_end = true;
+                               if (keybindings_path == default_bindings) {
+                                       error << string_compose (_("Default keybindings not found - %1 will be hard to use!"), PROGRAM_NAME) << endmsg;
+                                       return;
                                } else {
-                                       i += hyphen + 1;
-                                       at_end = (i >= len);
+                                       warning << string_compose (_("Key bindings file \"%1\" not found. Default bindings used instead"),
+                                                                  keybindings_path)
+                                               << endmsg;
+                                       keybindings_path = default_bindings;
                                }
-                       }
-               }
-               
-               if (keyname.length() == 1 && isupper (keyname[0])) {
-                       result.push_back (GDK_Shift_L);
-               }
-               
-               if ((keycode = gdk_keyval_from_name(get_real_keyname (keyname).c_str())) == GDK_VoidSymbol) {
-                       error << string_compose(_("KeyboardTarget: keyname \"%1\" is unknown."), keyname) << endmsg;
-                       result.clear();
-                       return result;
-               }
-               
-               result.push_back (keycode);
-       }
-
-       sort (result.begin(), result.end());
 
-       return result;
-}
+                       } else {
 
-string
-Keyboard::get_real_keyname (const string& name)
-{
+                               /* use it */
 
-       if (name == "Control" || name == "Ctrl") {
-               return "Control_L";
-       } 
-       if (name == "Meta" || name == "MetaL") {
-               return "Meta_L";
-       } 
-       if (name == "MetaR") {
-               return "Meta_R";
-       } 
-       if (name == "Alt" || name == "AltL") {
-               return "Alt_L";
-       } 
-       if (name == "AltR") {
-               return "Alt_R";
-       } 
-       if (name == "Shift") {
-               return "Shift_L";
-       }
-       if (name == "Shift_R") {
-               return "Shift_L";
-       }
-       if (name == " ") {
-               return "space";
-       }
-       if (name == "!") {
-               return "exclam";
-       }
-       if (name == "\"") {
-               return "quotedbl";
-       }
-       if (name == "#") {
-               return "numbersign";
-       }
-       if (name == "$") {
-               return "dollar";
-       }
-       if (name == "%") {
-               return "percent";
-       }
-       if (name == "&") {
-               return "ampersand";
-       }
-       if (name == "'") {
-               return "apostrophe";
-       }
-       if (name == "'") {
-               return "quoteright";
-       }
-       if (name == "(") {
-               return "parenleft";
-       }
-       if (name == ")") {
-               return "parenright";
-       }
-       if (name == "*") {
-               return "asterisk";
-       }
-       if (name == "+") {
-               return "plus";
-       }
-       if (name == ",") {
-               return "comma";
-       }
-       if (name == "-") {
-               return "minus";
-       }
-       if (name == ".") {
-               return "period";
-       }
-       if (name == "/") {
-               return "slash";
-       }
-       if (name == ":") {
-               return "colon";
-       }
-       if (name == ";") {
-               return "semicolon";
-       }
-       if (name == "<") {
-               return "less";
-       }
-       if (name == "=") {
-               return "equal";
-       }
-       if (name == ">") {
-               return "greater";
-       }
-       if (name == "?") {
-               return "question";
-       }
-       if (name == "@") {
-               return "at";
-       }
-       if (name == "[") {
-               return "bracketleft";
-       }
-       if (name == "\\") {
-               return "backslash";
-       }
-       if (name == "]") {
-               return "bracketright";
-       }
-       if (name == "^") {
-               return "asciicircum";
-       }
-       if (name == "_") {
-               return "underscore";
-       }
-       if (name == "`") {
-               return "grave";
-       }
-       if (name == "`") {
-               return "quoteleft";
-       }
-       if (name == "{") {
-               return "braceleft";
-       }
-       if (name == "|") {
-               return "bar";
-       }
-       if (name == "}") {
-               return "braceright";
-       }
-       if (name == "~") {
-               return "asciitilde";
-       }
+                               keybindings_path = keybindings_file;
+                               break;
 
-       return name;
-}
+                       }
 
-int
-Keyboard::get_prefix (float& val, bool& was_floating)
-{
-       if (current_prefix.length()) {
-               if (current_prefix.find ('.') != string::npos) {
-                       was_floating = true;
                } else {
-                       was_floating = false;
-               }
-               if (sscanf (current_prefix.c_str(), "%f", &val) == 1) {
-                       return 0;
-               }
-               current_prefix = "";
-       }
-       return -1;
-}
 
-void
-Keyboard::start_prefix ()
-{
-       collecting_prefix = true;
-       current_prefix = "";
-}
+                       /* path is absolute already */
 
-void
-Keyboard::clear_modifier_state ()
-{
-       modifier_mask = 0;
-}
-
-void
-Keyboard::check_modifier_state ()
-{
-       char keys[32];
-       int i, j;
-
-       clear_modifier_state ();
-       XQueryKeymap (GDK_DISPLAY(), keys);
-
-       for (i = 0; i < 32; ++i) {
-               for (j = 0; j < 8; ++j) {
+                       if (!Glib::file_test (keybindings_path, Glib::FILE_TEST_EXISTS)) {
+                               if (keybindings_path == default_bindings) {
+                                       error << string_compose (_("Default keybindings not found - %1 will be hard to use!"), PROGRAM_NAME) << endmsg;
+                                       return;
+                               } else {
+                                       keybindings_path = default_bindings;
+                               }
 
-                       if (keys[i] & (1<<j)) {
-                               modifier_mask |= modifier_masks[(i*8)+j];
+                       } else {
+                               break;
                        }
                }
        }
+
+       info << string_compose (_("Loading keybindings from %1"), keybindings_path) << endmsg;
+
+       load_keybindings (keybindings_path);
+
+       /* catch changes made via some GTK mechanism */
+
+       // GtkAccelMap* accelmap = gtk_accel_map_get();
+       // g_signal_connect (accelmap, "changed", (GCallback) accel_map_changed, this);
 }
 
-void
-Keyboard::check_meta_numlock (char keycode, guint mod, string modname)
+XMLNode&
+ArdourKeyboard::get_state (void)
 {
-       guint alternate_meta_mod;
-       string alternate_meta_modname;
-
-       if (mod == Meta) {
-               
-               guint keysym = XKeycodeToKeysym  (GDK_DISPLAY(), keycode, 0);
-               
-               if (keysym == GDK_Num_Lock) {
-
-                       switch (mod) {
-                       case GDK_MOD2_MASK:
-                               alternate_meta_mod = GDK_MOD3_MASK;
-                               alternate_meta_modname = "Mod3";
-                               break;
-                       case GDK_MOD3_MASK:
-                               alternate_meta_mod = GDK_MOD2_MASK;
-                               alternate_meta_modname = "Mod2";
-                               break;
-                       case GDK_MOD4_MASK:
-                               alternate_meta_mod = GDK_MOD2_MASK;
-                               alternate_meta_modname = "Mod2";
-                               break;
-                       case GDK_MOD5_MASK:
-                               alternate_meta_mod = GDK_MOD2_MASK;
-                               alternate_meta_modname = "Mod2";
-                               break;
-                       default:
-                               error << string_compose (_("Your system is completely broken - NumLock uses \"%1\""
-                                                   "as its modifier. This is madness - see the man page "
-                                                   "for xmodmap to find out how to fix this."),
-                                                 modname)
-                                     << endmsg;
-                               return;
-                       }
+       XMLNode* node = &Keyboard::get_state ();
+       char buf[32];
 
-                       warning << string_compose (_("Your system generates \"%1\" when the NumLock key "
-                                             "is pressed. This can cause problems when editing "
-                                             "so Ardour will use %2 to mean Meta rather than %1"),
-                                           modname, alternate_meta_modname)
-                               << endmsg;
+       snprintf (buf, sizeof (buf), "%d", constraint_mod);
+       node->add_property ("constraint-modifier", buf);
+       snprintf (buf, sizeof (buf), "%d", trim_contents_mod);
+       node->add_property ("trim-contents-modifier", buf);
+       snprintf (buf, sizeof (buf), "%d", trim_overlap_mod);
+       node->add_property ("trim-overlap-modifier", buf);
+       snprintf (buf, sizeof (buf), "%d", trim_anchored_mod);
+       node->add_property ("trim-anchored-modifier", buf);
+       snprintf (buf, sizeof (buf), "%d", fine_adjust_mod);
+       node->add_property ("fine-adjust-modifier", buf);
+       snprintf (buf, sizeof (buf), "%d", push_points_mod);
+       node->add_property ("push-points-modifier", buf);
+       snprintf (buf, sizeof (buf), "%d", note_size_relative_mod);
+       node->add_property ("note-size-relative-modifier", buf);
 
-                       set_meta_modifier (alternate_meta_mod);
-               }
-       }
+       return *node;
 }
 
-void
-Keyboard::get_modifier_masks ()
+int
+ArdourKeyboard::set_state (const XMLNode& node, int version)
 {
-       XModifierKeymap *modifiers;
-       KeyCode *keycode;
-       int i;
-       int bound;
+       XMLProperty const * prop;
 
-       XDisplayKeycodes (GDK_DISPLAY(), &min_keycode, &max_keycode);
-
-       /* This function builds a lookup table to provide rapid answers to
-          the question: what, if any, modmask, is associated with a given
-          keycode ?
-       */
-       
-       modifiers = XGetModifierMapping (GDK_DISPLAY());
-       
-       modifier_masks = new int32_t [max_keycode+1];
-       
-       keycode = modifiers->modifiermap;
-
-       for (i = 0; i < modifiers->max_keypermod; ++i) { /* shift */
-               if (*keycode) {
-                       modifier_masks[*keycode] = GDK_SHIFT_MASK;
-                       // cerr << "Shift = " << XKeysymToString (XKeycodeToKeysym  (GDK_DISPLAY(), *keycode, 0)) << endl;
-               }
-               keycode++;
-       }
-    
-       for (i = 0; i < modifiers->max_keypermod; ++i) keycode++; /* skip lock */
-    
-       for (i = 0; i < modifiers->max_keypermod; ++i) { /* control */
-               if (*keycode) {
-                       modifier_masks[*keycode] = GDK_CONTROL_MASK;
-                       // cerr << "Control = " << XKeysymToString (XKeycodeToKeysym  (GDK_DISPLAY(), *keycode, 0)) << endl;
-               }
-               keycode++;
+       if ((prop = node.property ("constraint-modifier")) != 0) {
+               sscanf (prop->value().c_str(), "%d", &constraint_mod);
        }
 
-       bound = 0;
-       for (i = 0; i < modifiers->max_keypermod; ++i) { /* mod 1 */
-               if (*keycode) {
-                       modifier_masks[*keycode] = GDK_MOD1_MASK;
-                       // cerr << "Mod1 = " << XKeysymToString (XKeycodeToKeysym  (GDK_DISPLAY(), *keycode, 0)) << endl;
-                       bound++;
-               }
-               keycode++;
+       if ((prop = node.property ("trim-contents-modifier")) != 0) {
+               sscanf (prop->value().c_str(), "%d", &trim_contents_mod);
        }
-#ifdef WARN_ABOUT_DUPLICATE_MODIFIERS
-       if (bound > 1) {
-               warning << string_compose (_("You have %1 keys bound to \"mod1\""), bound) << endmsg;
-       }
-#endif
-       bound = 0;
-       for (i = 0; i < modifiers->max_keypermod; ++i) { /* mod2 */
-               if (*keycode) {
-                       modifier_masks[*keycode] = GDK_MOD2_MASK;
-                       check_meta_numlock (*keycode, GDK_MOD2_MASK, "Mod2");
-                       //cerr << "Mod2 = " << std::hex << (int) *keycode << std::dec << " = " << XKeysymToString (XKeycodeToKeysym  (GDK_DISPLAY(), *keycode, 0)) << endl;
-                       bound++;
-               }
-               keycode++; 
-       }
-#ifdef WARN_ABOUT_DUPLICATE_MODIFIERS
-       if (bound > 1) {
-               warning << string_compose (_("You have %1 keys bound to \"mod2\""), bound) << endmsg;
-       }
-#endif
-       bound = 0;
-       for (i = 0; i < modifiers->max_keypermod; ++i) { /* mod3 */
-               if (*keycode) {
-                       modifier_masks[*keycode] = GDK_MOD3_MASK;
-                       check_meta_numlock (*keycode, GDK_MOD3_MASK, "Mod3");
-                       // cerr << "Mod3 = " << XKeysymToString (XKeycodeToKeysym  (GDK_DISPLAY(), *keycode, 0)) << endl;
-                       bound++;
-               }
-               keycode++; 
-       }
-#ifdef WARN_ABOUT_DUPLICATE_MODIFIERS
-       if (bound > 1) {
-               warning << string_compose (_("You have %1 keys bound to \"mod3\""), bound) << endmsg;
+
+       if ((prop = node.property ("trim-overlap-modifier")) != 0) {
+               sscanf (prop->value().c_str(), "%d", &trim_overlap_mod);
        }
-#endif
-       bound = 0;
-       for (i = 0; i < modifiers->max_keypermod; ++i) { /* mod 4 */
-               if (*keycode) {
-                       modifier_masks[*keycode] = GDK_MOD4_MASK;
-                       check_meta_numlock (*keycode, GDK_MOD4_MASK, "Mod4");
-                       // cerr << "Mod4 = " << XKeysymToString (XKeycodeToKeysym  (GDK_DISPLAY(), *keycode, 0)) << endl;
-                       bound++;
-               }
-               keycode++;
+
+       if ((prop = node.property ("trim-anchored-modifier")) != 0) {
+               sscanf (prop->value().c_str(), "%d", &trim_anchored_mod);
        }
-#ifdef WARN_ABOUT_DUPLICATE_MODIFIERS
-       if (bound > 1) {
-               warning << string_compose (_("You have %1 keys bound to \"mod4\""), bound) << endmsg;
+
+       if ((prop = node.property ("fine-adjust-modifier")) != 0) {
+               sscanf (prop->value().c_str(), "%d", &fine_adjust_mod);
        }
-#endif
-       bound = 0;
-       for (i = 0; i < modifiers->max_keypermod; ++i) { /* mod 5 */
-               if (*keycode) {
-                       modifier_masks[*keycode] = GDK_MOD5_MASK;
-                       check_meta_numlock (*keycode, GDK_MOD5_MASK, "Mod5");
-                       // cerr << "Mod5 = " << XKeysymToString (XKeycodeToKeysym  (GDK_DISPLAY(), *keycode, 0)) << endl;
-                       bound++;
-               }
-               keycode++;
+
+       if ((prop = node.property ("push-points-modifier")) != 0) {
+               sscanf (prop->value().c_str(), "%d", &push_points_mod);
        }
-#ifdef WARN_ABOUT_DUPLICATE_MODIFIERS
-       if (bound > 1) {
-               warning << string_compose (_("You have %1 keys bound to \"mod5\""), bound) << endmsg;
+
+       if ((prop = node.property ("note-size-relative-modifier")) != 0) {
+               sscanf (prop->value().c_str(), "%d", &note_size_relative_mod);
        }
-#endif
 
-       XFreeModifiermap (modifiers);
+       return Keyboard::set_state (node, version);
 }
 
+/* Snap and snap delta modifiers may contain each other, so we use the
+ * following two methods to sort that out:
+ */
 bool
-Keyboard::enter_window (GdkEventCrossing *ev, Gtk::Window* win)
+ArdourKeyboard::indicates_snap (guint state)
 {
-       switch (ev->detail) {
-       case GDK_NOTIFY_INFERIOR:
-               break;
-
-       case GDK_NOTIFY_VIRTUAL:
-               /* fallthru */
+       const bool contains_s = Keyboard::modifier_state_contains (state, Keyboard::snap_modifier ());
+       const bool contains_d = Keyboard::modifier_state_contains (state, Keyboard::snap_delta_modifier ());
+       const bool s_contains_d = Keyboard::modifier_state_contains (Keyboard::snap_modifier (), Keyboard::snap_delta_modifier ());
 
-       default:
-               check_modifier_state ();
-       }
-
-       return FALSE;
+       return  (contains_s && ((contains_d && s_contains_d) || !contains_d));
 }
 
 bool
-Keyboard::leave_window (GdkEventCrossing *ev, Gtk::Window* win)
+ArdourKeyboard::indicates_snap_delta (guint state)
 {
-       switch (ev->detail) {
-       case GDK_NOTIFY_INFERIOR:
-               if (debug_keyboard) {
-                       cerr << "INFERIOR crossing ... out\n";
-               }
-               break;
-
-       case GDK_NOTIFY_VIRTUAL:
-               if (debug_keyboard) {
-                       cerr << "VIRTUAL crossing ... out\n";
-               }
-               /* fallthru */
-
-       default:
-               if (debug_keyboard) {
-                       cerr << "REAL CROSSING ... out\n";
-                       cerr << "clearing current target\n";
-               }
-               state.clear ();
-               clear_modifier_state ();
-       }
-       return FALSE;
+       const bool contains_d = Keyboard::modifier_state_contains (state, Keyboard::snap_delta_modifier ());
+       const bool contains_s = Keyboard::modifier_state_contains (state, Keyboard::snap_modifier ());
+       const bool d_contains_s = Keyboard::modifier_state_contains (Keyboard::snap_delta_modifier (), Keyboard::snap_modifier ());
 
+       return (contains_d && ((contains_s && d_contains_s) || !contains_s));
 }
 
 void
-Keyboard::set_edit_button (guint but)
+ArdourKeyboard::set_constraint_modifier (guint mod)
 {
-       edit_but = but;
+       RelevantModifierKeyMask = GdkModifierType (RelevantModifierKeyMask & ~constraint_mod);
+       constraint_mod = mod;
+       RelevantModifierKeyMask = GdkModifierType (RelevantModifierKeyMask | constraint_mod);
 }
 
 void
-Keyboard::set_edit_modifier (guint mod)
+ArdourKeyboard::set_trim_contents_modifier (guint mod)
 {
-       RelevantModifierKeyMask = GdkModifierType (RelevantModifierKeyMask & ~edit_mod);
-       edit_mod = mod;
-       RelevantModifierKeyMask = GdkModifierType (RelevantModifierKeyMask | edit_mod);
+       RelevantModifierKeyMask = GdkModifierType (RelevantModifierKeyMask & ~trim_contents_mod);
+       trim_contents_mod = mod;
+       RelevantModifierKeyMask = GdkModifierType (RelevantModifierKeyMask | trim_contents_mod);
 }
 
 void
-Keyboard::set_delete_button (guint but)
+ArdourKeyboard::set_trim_overlap_modifier (guint mod)
 {
-       delete_but = but;
+       RelevantModifierKeyMask = GdkModifierType (RelevantModifierKeyMask & ~trim_overlap_mod);
+       trim_overlap_mod = mod;
+       RelevantModifierKeyMask = GdkModifierType (RelevantModifierKeyMask | trim_overlap_mod);
 }
 
 void
-Keyboard::set_delete_modifier (guint mod)
+ArdourKeyboard::set_trim_anchored_modifier (guint mod)
 {
-       RelevantModifierKeyMask = GdkModifierType (RelevantModifierKeyMask & ~delete_mod);
-       delete_mod = mod;
-       RelevantModifierKeyMask = GdkModifierType (RelevantModifierKeyMask | delete_mod);
+       RelevantModifierKeyMask = GdkModifierType (RelevantModifierKeyMask & ~trim_anchored_mod);
+       trim_anchored_mod = mod;
+       RelevantModifierKeyMask = GdkModifierType (RelevantModifierKeyMask | trim_anchored_mod);
 }
 
 void
-Keyboard::set_meta_modifier (guint mod)
+ArdourKeyboard::set_fine_adjust_modifier (guint mod)
 {
-       /* we don't include Meta in the RelevantModifierKeyMask because its not used
-          in the same way as snap_mod, delete_mod etc. the only reason we allow it to be
-          set at all is that X Window has no convention for the keyboard modifier
-          that Meta should use. Some Linux distributions bind NumLock to Mod2, which
-          is our default Meta modifier, and this causes severe problems.
-       */
-       Meta = mod;
+       RelevantModifierKeyMask = GdkModifierType (RelevantModifierKeyMask & ~fine_adjust_mod);
+       fine_adjust_mod = mod;
+       RelevantModifierKeyMask = GdkModifierType (RelevantModifierKeyMask | fine_adjust_mod);
 }
 
 void
-Keyboard::set_snap_modifier (guint mod)
-{
-       RelevantModifierKeyMask = GdkModifierType (RelevantModifierKeyMask & ~snap_mod);
-       snap_mod = mod;
-       RelevantModifierKeyMask = GdkModifierType (RelevantModifierKeyMask | snap_mod);
-}
-
-bool
-Keyboard::is_edit_event (GdkEventButton *ev)
+ArdourKeyboard::set_push_points_modifier (guint mod)
 {
-       return (ev->type == GDK_BUTTON_PRESS || ev->type == GDK_BUTTON_RELEASE) && 
-               (ev->button == Keyboard::edit_button()) && 
-               ((ev->state & RelevantModifierKeyMask) == Keyboard::edit_modifier());
+       RelevantModifierKeyMask = GdkModifierType (RelevantModifierKeyMask & ~push_points_mod);
+       push_points_mod = mod;
+       RelevantModifierKeyMask = GdkModifierType (RelevantModifierKeyMask | push_points_mod);
 }
 
-bool
-Keyboard::is_delete_event (GdkEventButton *ev)
-{
-       return (ev->type == GDK_BUTTON_PRESS || ev->type == GDK_BUTTON_RELEASE) && 
-               (ev->button == Keyboard::delete_button()) && 
-               ((ev->state & RelevantModifierKeyMask) == Keyboard::delete_modifier());
-}
-
-bool
-Keyboard::is_context_menu_event (GdkEventButton *ev)
-{
-       return (ev->type == GDK_BUTTON_PRESS || ev->type == GDK_BUTTON_RELEASE) && 
-               (ev->button == 3) && 
-               ((ev->state & RelevantModifierKeyMask) == 0);
-}
-
-bool 
-Keyboard::no_modifiers_active (guint state)
-{
-       return (state & RelevantModifierKeyMask) == 0;
-}
-
-bool
-Keyboard::modifier_state_contains (guint state, ModifierMask mask)
-{
-       return (state & mask) == (guint) mask;
-}
-
-bool
-Keyboard::modifier_state_equals (guint state, ModifierMask mask)
+void
+ArdourKeyboard::set_note_size_relative_modifier (guint mod)
 {
-       return (state & RelevantModifierKeyMask) == (guint) mask;
+       RelevantModifierKeyMask = GdkModifierType (RelevantModifierKeyMask & ~note_size_relative_mod);
+       note_size_relative_mod = mod;
+       RelevantModifierKeyMask = GdkModifierType (RelevantModifierKeyMask | note_size_relative_mod);
 }
 
 Selection::Operation
-Keyboard::selection_type (guint state)
+ArdourKeyboard::selection_type (guint state)
 {
-       if (modifier_state_equals (state, Shift)) {
+       /* note that there is no modifier for "Add" */
+
+       if (modifier_state_equals (state, RangeSelectModifier)) {
                return Selection::Extend;
-       } else if (modifier_state_equals (state, Control)) {
+       } else if (modifier_state_equals (state, PrimaryModifier)) {
                return Selection::Toggle;
        } else {
                return Selection::Set;