Changeset 105586 in vbox for trunk/src/VBox/Frontends

Timestamp:

Aug 5, 2024 2:28:46 PM (6 months ago)

Author:

vboxsync

Message:

FE/Qt: bugref:10665, bugref:10744: UINativeWizard: Make sure wizard is cleanup up if aborted (by Escape key, Cancel button or by closing the window).

Location:

trunk/src/VBox/Frontends/VirtualBox/src/wizards

Files:

• UINativeWizard.cpp (modified) (5 diffs)
• UINativeWizard.h (modified) (3 diffs)

trunk/src/VBox/Frontends/VirtualBox/src/wizards/UINativeWizard.cpp

@@ -5 +5 @@
 
 /*
- * Copyright (C) 2009-2023 Oracle and/or its affiliates.
+ * Copyright (C) 2009-2024 Oracle and/or its affiliates.
  *
  * This file is part of VirtualBox base platform packages, as
@@ -99 +99 @@
     , m_strHelpKeyword(strHelpKeyword)
     , m_iLastIndex(-1)
+    , m_fAborted(true)
     , m_fClosed(false)
     , m_pLabelPixmap(0)
@@ -234 +235 @@
 void UINativeWizard::keyPressEvent(QKeyEvent *pEvent)
 {
+    // WORKAROUND:
+    // In non-modal case we'll have to handle Escape button ourselves.
+    // In modal case QDialog does this itself internally by unwinding the event-loop.
+
     /* Different handling depending on current modality: */
     const Qt::WindowModality enmModality = windowHandle()->modality();
@@ -276 +281 @@
         {
             m_fClosed = true;
+            if (m_fAborted)
+                cleanWizard();
             emit sigClose(m_enmType);
         }
@@ -374 +381 @@
         /* Different handling depending on current modality: */
         if (windowHandle()->modality() == Qt::NonModal)
+        {
+            m_fAborted = false;
             close();
+        }
         else
             accept();

trunk/src/VBox/Frontends/VirtualBox/src/wizards/UINativeWizard.h

@@ -5 +5 @@
 
 /*
- * Copyright (C) 2009-2023 Oracle and/or its affiliates.
+ * Copyright (C) 2009-2024 Oracle and/or its affiliates.
  *
  * This file is part of VirtualBox base platform packages, as
@@ -155 +155 @@
 
 protected slots:
+
     /** Handles translation event. */
     virtual void sltRetranslateUI();
@@ -210 +211 @@
     /** Holds the set of invisible pages. */
     QSet<int>   m_invisiblePages;
+    /** Holds whether user has requested to abort wizard. */
+    bool        m_fAborted;
     /** Holds whether the dialod had emitted signal to be closed. */
     bool        m_fClosed;