From 6ec830cefec859423da84ab8f3e4fb23302f3adc Mon Sep 17 00:00:00 2001
From: Sven Augustin <sven.augustin@psi.ch>
Date: Fri, 30 Dec 2022 01:20:11 +0100
Subject: [PATCH] use simpler QAction logic

---
 grum/menus/barmenu.py | 13 ++-----------
 1 file changed, 2 insertions(+), 11 deletions(-)

diff --git a/grum/menus/barmenu.py b/grum/menus/barmenu.py
index 4f528f5..5007021 100644
--- a/grum/menus/barmenu.py
+++ b/grum/menus/barmenu.py
@@ -8,19 +8,10 @@ class BarMenu(MenuBase):
     def __init__(self, bar, name):
         super().__init__()
         self.qmenu = qmenu = bar.addMenu(name)
-        qmenu.triggered[QAction].connect(self.on_select)
-        self.actions = {}
-
-    def on_select(self, p):
-        txt = p.text()
-        if txt in self.checkboxes: #TODO: allow checkboxes with callback?
-            return
-        func = self.actions[txt]
-        func()
 
     def addAction(self, name, func):
-        self.qmenu.addAction(name)
-        self.actions[name] = func
+        action = QAction(name, triggered=func, parent=self.qmenu) # here, parent needs to be set
+        self.qmenu.addAction(action)