Changeset 50807 in vbox

Timestamp:

Mar 18, 2014 9:35:35 AM (11 years ago)

Author:

vboxsync

Message:

FE/Qt: Runtime UI: Mouse-handling: Do not try to activate hovered machine-window instantly, it's causing races and conflicts with Apple HIG.

Location:

trunk/src/VBox/Frontends/VirtualBox/src/runtime

Files:

• UIMachineWindow.h (modified) (1 diff)
• UIMouseHandler.cpp (modified) (2 diffs)

trunk/src/VBox/Frontends/VirtualBox/src/runtime/UIMachineWindow.h

@@ -74 +74 @@
 
 protected slots:
+
+    /** Sets the top-level widget containing this widget to be the active window. */
+    void sltActivateWindow() { activateWindow(); }
 
     /* Session event-handlers: */

trunk/src/VBox/Frontends/VirtualBox/src/runtime/UIMouseHandler.cpp

@@ -22 +22 @@
 #include <QMouseEvent>
 #include <QTouchEvent>
+#include <QTimer>
 
 /* GUI includes: */
@@ -597 +598 @@
                         QApplication::activeWindow() != pWatchedWidget->window())
                     {
-                        /* Activating hovered machine window: */
-                        pWatchedWidget->window()->activateWindow();
+                        /* Activating hovered machine-window in 300msec: */
+                        QTimer::singleShot(300, pWatchedWidget->window(), SLOT(sltActivateWindow()));
 #ifdef Q_WS_X11
                         /* On X11 its not enough to just activate window if you