From 17849b12f232c89bdae16f7e179d3a92a32280ad Mon Sep 17 00:00:00 2001
From: Stanislaw Halik <sthalik@misaki.pl>
Date: Wed, 11 Nov 2015 13:44:45 +0100
Subject: shortcuts: map joystick buttons on depress only

Some buttons like the X65 mode switch are held all the time. Prevent
them from hogging all the keybindings.

Issue: #118
---
 gui/options-dialog.cpp | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

(limited to 'gui/options-dialog.cpp')

diff --git a/gui/options-dialog.cpp b/gui/options-dialog.cpp
index c8bf668d..4cff6e77 100644
--- a/gui/options-dialog.cpp
+++ b/gui/options-dialog.cpp
@@ -90,11 +90,14 @@ void OptionsDialog::bind_key(Shortcuts::key_opts& kopts, QLabel* label)
         kopts.button = -1;
         d.close();
     });
-    connect(k, &KeyboardListener::joystick_button_pressed, [&](QString guid, int idx) -> void {
-        kopts.guid = guid;
-        kopts.keycode = "";
-        kopts.button = idx;
-        d.close();
+    connect(k, &KeyboardListener::joystick_button_pressed, [&](QString guid, int idx, bool held) -> void {
+        if (!held)
+        {
+            kopts.guid = guid;
+            kopts.keycode = "";
+            kopts.button = idx;
+            d.close();
+        }
     });
     d.exec();
     label->setText(kopts_to_string(kopts));
-- 
cgit v1.2.3