fix: regression that broke support for hardware keyboard handling in Direct Input

should fix #67

Author: Arian04

app/src/main/java/me/arianb/usb_hid_client/input_views/DirectInputKeyboardView.kt

@@ -71,19 +71,23 @@ fun DirectInput(
 
             etDirectInput.setKeySender(keySender)
 
-            // For some reason, the input connection doesn't receive media keys, so i'm listening for them here
-            // and just passing them through
+            // For some reason, the input connection doesn't receive media keys, but this listener *does*, so
+            // I'm listening for them here and just passing them through.
+            //
+            // As another note, this key listener seems to be triggered all the time when the key is sent from a
+            // hardware keyboard (during admittedly limited testing).
             etDirectInput.setOnKeyListener { _, keyCode, event ->
                 Timber.d("OnKeyListener received KeyEvent: %s", event.toString())
                 // If key is a media key and user doesn't want us to pass it through, then just
-                // ignore it and let the system handle it normally
+                // ignore it and let the system handle it normally. Otherwise, send it.
                 // TODO: rename this preference to "media key passthrough" or something similar since that's more accurate
                 if (KeyCodeTranslation.isMediaKey(keyCode)) {
-                    if (userPreferencesState.isVolumeButtonPassthroughEnabled) {
-                        return@setOnKeyListener etDirectInput.sendKeyEvent(event)
+                    if (!userPreferencesState.isVolumeButtonPassthroughEnabled) {
+                        return@setOnKeyListener false
                     }
                 }
-                false
+
+                return@setOnKeyListener etDirectInput.sendKeyEvent(event)
             }
         }
     )