Changeset 69670 in vbox

Timestamp:

Nov 13, 2017 12:03:55 PM (7 years ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

119038

Message:

FE/Qt: bugref:8400: Few fixes for medium details widget: 1. Disable location choose button is no medium is chosen; 2. Manually validate details on loading to handle it if nothing loaded; 3. Adjust validation for location to treat only chosen mediums.

File:

• trunk/src/VBox/Frontends/VirtualBox/src/medium/UIMediumDetailsWidget.cpp (modified) (3 diffs)

trunk/src/VBox/Frontends/VirtualBox/src/medium/UIMediumDetailsWidget.cpp

@@ -678 +678 @@
     m_pLabelLocation->setEnabled(m_newData.m_fValid);
     m_pEditorLocation->setEnabled(m_newData.m_fValid);
+    m_pButtonLocation->setEnabled(m_newData.m_fValid);
     m_pEditorLocation->setText(m_newData.m_options.m_strLocation);
 
@@ -693 +694 @@
     m_pEditorSize->setMediumSize(m_newData.m_options.m_uLogicalSize);
     sltSizeValueChanged(m_pEditorSize->mediumSize());
+
+    /* Revalidate: */
+    revalidate();
 }
 
@@ -728 +732 @@
     if (!pWidget || pWidget == m_pErrorPaneLocation)
     {
-        /* Always valid for now: */
-        const bool fError = m_newData.m_options.m_strLocation.isEmpty();
+        /* If medium is valid itself, details are valid only is location is set: */
+        const bool fError = m_newData.m_fValid && m_newData.m_options.m_strLocation.isEmpty();
         m_pErrorPaneLocation->setVisible(fError);
         if (fError)