From 216ead22141e8f0a6d1c891e799560e9d66163a4 Mon Sep 17 00:00:00 2001
From: James Crook <james.k.crook@gmail.com>
Date: Sat, 27 May 2017 20:14:56 +0100
Subject: [PATCH] Provide two choices for defaults in KeyPreferences

This is a step towards reducing the default list of shortcuts.
---
 src/prefs/KeyConfigPrefs.cpp | 36 ++++++++++++++++++++++++++++++++++--
 src/prefs/KeyConfigPrefs.h   |  1 +
 2 files changed, 35 insertions(+), 2 deletions(-)

diff --git a/src/prefs/KeyConfigPrefs.cpp b/src/prefs/KeyConfigPrefs.cpp
index b6aebb918..e5db8871e 100644
--- a/src/prefs/KeyConfigPrefs.cpp
+++ b/src/prefs/KeyConfigPrefs.cpp
@@ -387,12 +387,44 @@ void KeyConfigPrefs::OnExport(wxCommandEvent & WXUNUSED(event))
    } );
 }
 
-void KeyConfigPrefs::OnDefaults(wxCommandEvent & WXUNUSED(event))
+
+
+// There currently is only one clickable AButton
+// so we just do what it needs.
+void KeyConfigPrefs::OnDefaults(wxCommandEvent & event)
 {
+   wxMenu Menu;
+   Menu.Append( 0, _("Import Standard Defaults") );
+   Menu.Append( 1, _("Import Max Defaults") );
+   Menu.Bind( wxEVT_COMMAND_MENU_SELECTED, &KeyConfigPrefs::OnImportDefaults, this );
+   // Pop it up where the mouse is.
+   PopupMenu(&Menu);//, wxPoint(0, 0));
+}
+
+void KeyConfigPrefs::OnImportDefaults(wxCommandEvent & event)
+{
+   int id = event.GetId();
    mNewKeys = mDefaultKeys;
 
    for (size_t i = 0; i < mNewKeys.GetCount(); i++) {
-      mManager->SetKeyFromIndex(i, mNewKeys[i]);
+      // Proof of concept for idea for freeing up some unwanted bindings.
+      // There will be neater code idioms we can use.
+      bool bDeleteBinding = false;
+      if( id == 0 ){
+         bDeleteBinding |= ( mNewKeys[i] == "A" );
+         bDeleteBinding |= ( mNewKeys[i] == "D" );
+         bDeleteBinding |= ( mNewKeys[i] == "Ctrl+Shift+A" );
+         bDeleteBinding |= ( mNewKeys[i] == "Alt+X" );
+         bDeleteBinding |= ( mNewKeys[i] == "Alt+K" );
+         bDeleteBinding |= ( mNewKeys[i] == "Alt+Shift+X" );
+         bDeleteBinding |= ( mNewKeys[i] == "Alt+Shift+K" );
+         bDeleteBinding |= ( mNewKeys[i] == "Alt+L" );
+         bDeleteBinding |= ( mNewKeys[i] == "Alt+Shift+C" );
+         bDeleteBinding |= ( mNewKeys[i] == "Alt+I" );
+         bDeleteBinding |= ( mNewKeys[i] == "Alt+J" );
+         bDeleteBinding |= ( mNewKeys[i] == "Alt+Shift+J" );
+      }
+      mManager->SetKeyFromIndex(i, bDeleteBinding ? "" : mNewKeys[i]);
    }
 
    RefreshBindings(true);
diff --git a/src/prefs/KeyConfigPrefs.h b/src/prefs/KeyConfigPrefs.h
index 0ed56104f..ac1d4b109 100644
--- a/src/prefs/KeyConfigPrefs.h
+++ b/src/prefs/KeyConfigPrefs.h
@@ -52,6 +52,7 @@ private:
 
    void OnViewBy(wxCommandEvent & e);
    void OnDefaults(wxCommandEvent & e);
+   void OnImportDefaults(wxCommandEvent & e);
    void OnImport(wxCommandEvent & e);
    void OnExport(wxCommandEvent & e);
    void OnSet(wxCommandEvent & e);