Changeset 101239 in vbox for trunk

Timestamp:

Sep 22, 2023 2:51:16 PM (17 months ago)

Author:

vboxsync

Message:

FE/Qt: bugref:10513: UIAdvancedSettingsDialog: Adjusting sticky scrolling rules to avoid wheel events being passed to types of widgets which we don't really want these events for.

File:

• trunk/src/VBox/Frontends/VirtualBox/src/settings/UIAdvancedSettingsDialog.cpp (modified) (1 diff)

trunk/src/VBox/Frontends/VirtualBox/src/settings/UIAdvancedSettingsDialog.cpp

@@ -480 +480 @@
     }
 
+    /* Unconditionally and for good
+     * redirect wheel event for widgets of following types to m_pScrollViewport: */
+    if (   qobject_cast<QAbstractButton*>(pObject)
+        || qobject_cast<QAbstractSpinBox*>(pObject)
+        || qobject_cast<QAbstractSpinBox*>(pObject->parent())
+        || qobject_cast<QComboBox*>(pObject)
+        || qobject_cast<QSlider*>(pObject)
+        || qobject_cast<QTabWidget*>(pObject)
+        || qobject_cast<QTabWidget*>(pObject->parent()))
+    {
+        /* Check if redirected event was really handled, otherwise give it back: */
+        if (QCoreApplication::sendEvent(m_pScrollViewport, pEvent))
+            return true;
+    }
+
     /* While 'sticky scrolling timer' is active
      * redirect wheel event for widgets of following types to m_pScrollViewport: */
     if (   m_pScrollingTimer->isActive()
-        && (   qobject_cast<QAbstractButton*>(pObject)
-            || qobject_cast<QAbstractScrollArea*>(pObject)
-            || qobject_cast<QAbstractScrollArea*>(pObject->parent())
-            || qobject_cast<QAbstractSpinBox*>(pObject)
-            || qobject_cast<QAbstractSpinBox*>(pObject->parent())
-            || qobject_cast<QComboBox*>(pObject)
-            || qobject_cast<QSlider*>(pObject)
-            || qobject_cast<QTabWidget*>(pObject)
-            || qobject_cast<QTabWidget*>(pObject->parent())))
+        && (   qobject_cast<QAbstractScrollArea*>(pObject)
+            || qobject_cast<QAbstractScrollArea*>(pObject->parent())))
     {
         /* Check if redirected event was really handled, otherwise give it back: */