https://bugs.kde.org/show_bug.cgi?id=438197

--- Comment #2 from Oded Arbel <o...@geek.co.il> ---
(In reply to Nate Graham from comment #1)
> This is a core behavior of QMenu from Qt which cannot be changed by us. The
> real solution is probably to just fix Bug 438198. :)

This assertion doesn't seem to be correct. For once, QAction::triggered()
documentation - https://doc.qt.io/qt-5/qaction.html#triggered - notes that
triggered() is not emitted by toggle(). But also notably, kwin's relevant code
- here:
https://invent.kde.org/odeda/kwin/-/blob/770875a76e662a1ed7728da769f8f7d019623701/src/useractions.cpp#L585
- connects the checkbox's clicked() signal to the QAction:triggered() slot. It
didn't have to do that - that signal can be connected to something in
UserActionsMenu that understands that a checkbox was clicked and not trigger
the QAction.

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to