From cdbdbab24331c25ea0ea201011ddb695b5a5be35 Mon Sep 17 00:00:00 2001
From: Bert Gijsbers <gijsbers@science.uva.nl>
Date: Tue, 2 Jul 2024 21:30:18 +0200
Subject: [PATCH] When a key event is handled, don't propagate it further.

---
 src/wmwinlist.cc | 56 +++++++++++++++++++-----------------------------
 1 file changed, 22 insertions(+), 34 deletions(-)

diff --git a/src/wmwinlist.cc b/src/wmwinlist.cc
index 5b1a27b5c..840ed946c 100644
--- a/src/wmwinlist.cc
+++ b/src/wmwinlist.cc
@@ -188,46 +188,34 @@ void WindowListBox::actionPerformed(YAction action, unsigned int modifiers) {
 }
 
 bool WindowListBox::handleKey(const XKeyEvent &key) {
+    const KeySym k = keyCodeToKeySym(key.keycode);
+    const int m = KEY_MODMASK(key.state);
+
     if (key.type == KeyPress) {
-        KeySym k = keyCodeToKeySym(key.keycode);
-        int m = KEY_MODMASK(key.state);
         fKeyPressed = k;
-
-        switch (k) {
-        case XK_F10:
-        case XK_Menu:
-            if (k != XK_F10 || m == ShiftMask) {
-                if (hasSelection()) {
-                    YMenu* windowListPopup = windowList->getWindowListPopup();
-                    enableCommands(windowListPopup);
-                    windowListPopup->popup(nullptr, nullptr, nullptr,
-                                           key.x_root, key.y_root,
-                                           YPopupWindow::pfCanFlipVertical |
-                                           YPopupWindow::pfCanFlipHorizontal |
-                                           YPopupWindow::pfPopupMenu);
-                } else {
-                    YMenu* windowListAllPopup = windowList->getWindowListAllPopup();
-                    windowListAllPopup->popup(nullptr, nullptr, nullptr,
-                                              key.x_root, key.y_root,
-                                              YPopupWindow::pfCanFlipVertical |
-                                              YPopupWindow::pfCanFlipHorizontal |
-                                              YPopupWindow::pfPopupMenu);
-                }
-            }
-            break;
-        case XK_Delete:
-            {
-                if (m & ShiftMask)
-                    actionPerformed(actionKill, key.state);
-                else
-                    actionPerformed(actionClose, key.state);
+        if (k == XK_Menu || (k == XK_F10 && m == ShiftMask)) {
+            YMenu* menu;
+            if (hasSelection()) {
+                menu = windowList->getWindowListPopup();
+                enableCommands(menu);
+            } else {
+                menu = windowList->getWindowListAllPopup();
             }
-            break;
+            menu->popup(nullptr, nullptr, nullptr, key.x_root, key.y_root,
+                        YPopupWindow::pfCanFlipVertical |
+                        YPopupWindow::pfCanFlipHorizontal |
+                        YPopupWindow::pfPopupMenu);
+            return true;
+        }
+        else if (k == XK_Delete) {
+            if (m & ShiftMask)
+                actionPerformed(actionKill, key.state);
+            else
+                actionPerformed(actionClose, key.state);
+            return true;
         }
     }
     else if (key.type == KeyRelease) {
-        KeySym k = keyCodeToKeySym(key.keycode);
-        int m = KEY_MODMASK(key.state);
         if (k == XK_Escape && k == fKeyPressed && m == 0) {
             windowList->handleClose();
             return true;