ListLayout: Another Cue is selected when pressing a Keybord Shortcut

[Stav88]

Describe the bug After having configured a Keyord Shortcut, a search in the list is executed when the shortcut is used.

To Reproduce Steps to reproduce the behavior:

1. At start, select "List Layout" as Layout
2. Add three cues
3. Rename the second one to begin with the letter Q (for exemple)
   • Right-click > Edit Cue
   • Open "Appearance" tab
   • Edit Cue Name
   • Click "OK"
4. Add a Keybord Shortcut on last one using the letter choosed at step 3 (Q)
   • Right-click > Edit Cue
   • Open "Cue Control" tab
   • Open "Keyboard Shortcut" subtab
   • Click Add
   • In key column, write "q" (in small case)
   • Click "OK"
5. Select the first cue
6. Press the letter choosed at step 3 (Q)

• The second cue is selected
• The last cue is played

Expected behavior

• The first cue is still selected
• The last cue is played

I don't know what is the expected behavior if no "Keyboard Shortcut" is configured.

Linux Show Player version:

• 0.5.1
• 0.5.2
• 0.5.3
• master

OS Version

• Ubuntu 20.04 (XFCE) x68_64
• Manjaro (XFCE) Aarch64 (pinebook pro)

Additional context The search in list can be disabled by overloading the method keyboardSearch in CueListView :

diff --git a/lisp/layouts/list_layout/cue_list_view.py b/lisp/layouts/list_layout/cue_list_view.py
index 8a190df8..dca8d07c 100644
--- a/lisp/layouts/list_layout/cue_list_view.py
+++ b/lisp/layouts/list_layout/cue_list_view.py
@@ -111,6 +111,10 @@ class CueListView(QTreeWidget):
         else:
             super().contextMenuEvent(event)

+    def keyboardSearch(self, search):
+        # disable scroll to letter match
+        pass
+
     def keyPressEvent(self, event):
         self.key_event.emit(event)

but you may want to keep the behavior if the shortcut is not configured.

[FrancescoCeruti]

Thanks! I confirm for 0.5.x versions. The "develop" branch apparently is not affected by the same issue.

IMHO it's an unwanted behavior. Feel free to open a pull request with a fix for the master branch.