[LyX/master] Shortcut box now stays open if bad input (#8703)

[Scott Kostyshak]

commit 27131e655af4a1b4b917c1d8b24c58cd9251d1b8
Author: Scott Kostyshak <skost...@lyx.org>
Date:   Tue Jan 14 08:04:05 2014 -0500

    Shortcut box now stays open if bad input (#8703)
    
    This allows the user to correct the bad input without
    having to enter the other fields again in a new dialog.
    
    Patch from Martin Hoffmann.

diff --git a/lib/generate_contributions.py b/lib/generate_contributions.py
index e2dcf7d..50348db 100755
--- a/lib/generate_contributions.py
+++ b/lib/generate_contributions.py
@@ -851,6 +851,14 @@ contributors = [
                  "22 February 2005",
                  u"Danish translation"),
 
+     contributor(u"Martin Hoffmann",
+                 "hoffi...@gmail.com",
+                 "GPL",
+                 "Re: #8703: 'new shortcut' box closes if no shortcut",
+                 "m=138105799411067",
+                 "6 October 2013",
+                 u"Dialog usability fix"),
+
      contributor(u"Bernard Hurley",
                  "bernard () fong-hurley ! org ! uk",
                  "GPL",
diff --git a/src/frontends/qt4/GuiPrefs.cpp b/src/frontends/qt4/GuiPrefs.cpp
index 68f6aae..8686690 100644
--- a/src/frontends/qt4/GuiPrefs.cpp
+++ b/src/frontends/qt4/GuiPrefs.cpp
@@ -2805,8 +2805,6 @@ PrefShortcuts::PrefShortcuts(GuiPreferences * form)
        shortcut_bc_.setCancel(shortcut_->cancelPB);
 
        connect(shortcut_->okPB, SIGNAL(clicked()),
-               shortcut_, SLOT(accept()));
-       connect(shortcut_->okPB, SIGNAL(clicked()),
                this, SIGNAL(changed()));
        connect(shortcut_->cancelPB, SIGNAL(clicked()),
                shortcut_, SLOT(reject()));
@@ -3180,6 +3178,8 @@ void PrefShortcuts::shortcutOkPressed()
                return;
        }
 
+       shortcut_->accept();
+
        // check to see if there's been any change
        FuncRequest oldBinding = system_bind_.getBinding(k);
        if (oldBinding.action() == LFUN_UNKNOWN_ACTION)