Remove assertion in QGroupBoxPrivate::shouldHandleKeyEvent()

A disabled QGroupBox doesn't receive key events in any screnario implemented within Qt. It can, however, receive such an event directly in an external implementation. Remove the assertion and return false instead. This amends 5d397a2. Task-number: QTBUG-129095 Change-Id: I7b7167ac49b24844712c27b9fd4395a81687481f Reviewed-by: Volker Hilsheimer <[email protected]>
qt · Oct 31, 2024 · 070d21a · 070d21a
1 parent d91bfbc
commit 070d21a
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/src/widgets/widgets/qgroupbox.cpp b/src/widgets/widgets/qgroupbox.cpp
@@ -727,9 +727,8 @@ void QGroupBox::mouseReleaseEvent(QMouseEvent *event)
 bool QGroupBoxPrivate::shouldHandleKeyEvent(const QKeyEvent *keyEvent) const
 {
     Q_Q(const QGroupBox);
-    Q_ASSERT(q->isEnabled());
 
-    if (!q->isCheckable() || keyEvent->isAutoRepeat())
+    if (!q->isEnabled() || !q->isCheckable() || keyEvent->isAutoRepeat())
         return false;
 
     const QList<Qt::Key> buttonPressKeys = QGuiApplicationPrivate::platformTheme()