Changeset 104547 in vbox

Timestamp:

May 8, 2024 12:25:11 PM (7 months ago)

Author:

vboxsync

Message:

FE/Qt: bugref:10670: Accessibility fixes for New Cloud VM wizard; Expert mode should have proper labels for text-fields it uses, it's required for JAWS screen-reader.

Location:

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

Files:

• UIWizardNewCloudVMPageExpert.cpp (modified) (5 diffs)
• UIWizardNewCloudVMPageExpert.h (modified) (2 diffs)

trunk/src/VBox/Frontends/VirtualBox/src/wizards/newcloudvm/UIWizardNewCloudVMPageExpert.cpp

@@ -27 +27 @@
 
 /* Qt includes: */
+#include <QGridLayout>
 #include <QHeaderView>
+#include <QLabel>
 #include <QListWidget>
 #include <QPushButton>
@@ -53 +55 @@
 UIWizardNewCloudVMPageExpert::UIWizardNewCloudVMPageExpert()
     : m_pToolBox(0)
+    , m_pLayoutProvider(0)
+    , m_pProviderLabel(0)
     , m_pProviderComboBox(0)
+    , m_pProfileLabel(0)
     , m_pProfileComboBox(0)
     , m_pProfileToolButton(0)
@@ -78 +83 @@
                     pLayoutLocation->setContentsMargins(0, 0, 0, 0);
 
-                    /* Prepare provider combo-box: */
-                    m_pProviderComboBox = new QIComboBox(pWidgetLocation);
-                    if (m_pProviderComboBox)
-                        pLayoutLocation->addWidget(m_pProviderComboBox);
-
-                    /* Prepare profile layout: */
-                    QHBoxLayout *pLayoutProfile = new QHBoxLayout;
-                    if (pLayoutProfile)
+                    /* Prepare provider layout: */
+                    m_pLayoutProvider = new QGridLayout;
+                    if (m_pLayoutProvider)
                     {
-                        pLayoutProfile->setContentsMargins(0, 0, 0, 0);
-                        pLayoutProfile->setSpacing(1);
-
-                        /* Prepare profile combo-box: */
-                        m_pProfileComboBox = new QIComboBox(pWidgetLocation);
-                        if (m_pProfileComboBox)
-                            pLayoutProfile->addWidget(m_pProfileComboBox);
-
-                        /* Prepare profile tool-button: */
-                        m_pProfileToolButton = new QIToolButton(pWidgetLocation);
-                        if (m_pProfileToolButton)
+                        m_pLayoutProvider->setContentsMargins(0, 0, 0, 0);
+                        m_pLayoutProvider->setColumnStretch(0, 0);
+                        m_pLayoutProvider->setColumnStretch(1, 1);
+
+                        /* Prepare provider label: */
+                        m_pProviderLabel = new QLabel(this);
+                        if (m_pProviderLabel)
+                            m_pLayoutProvider->addWidget(m_pProviderLabel, 0, 0, Qt::AlignRight);
+
+                        /* Prepare provider combo-box: */
+                        m_pProviderComboBox = new QIComboBox(pWidgetLocation);
+                        if (m_pProviderComboBox)
                         {
-                            m_pProfileToolButton->setIcon(UIIconPool::iconSet(":/cloud_profile_manager_16px.png",
-                                                                              ":/cloud_profile_manager_disabled_16px.png"));
-                            pLayoutProfile->addWidget(m_pProfileToolButton);
+                            m_pProviderLabel->setBuddy(m_pProviderComboBox);
+                            m_pLayoutProvider->addWidget(m_pProviderComboBox, 0, 1);
                         }
 
+                        /* Prepare profile label: */
+                        m_pProfileLabel = new QLabel(this);
+                        if (m_pProfileLabel)
+                            m_pLayoutProvider->addWidget(m_pProfileLabel, 1, 0, Qt::AlignRight);
+
+                        /* Prepare profile layout: */
+                        QHBoxLayout *pLayoutProfile = new QHBoxLayout;
+                        if (pLayoutProfile)
+                        {
+                            pLayoutProfile->setContentsMargins(0, 0, 0, 0);
+                            pLayoutProfile->setSpacing(1);
+
+                            /* Prepare profile combo-box: */
+                            m_pProfileComboBox = new QIComboBox(pWidgetLocation);
+                            if (m_pProfileComboBox)
+                            {
+                                m_pProfileLabel->setBuddy(m_pProfileComboBox);
+                                pLayoutProfile->addWidget(m_pProfileComboBox);
+                            }
+
+                            /* Prepare profile tool-button: */
+                            m_pProfileToolButton = new QIToolButton(pWidgetLocation);
+                            if (m_pProfileToolButton)
+                            {
+                                m_pProfileToolButton->setIcon(UIIconPool::iconSet(":/cloud_profile_manager_16px.png",
+                                                                                  ":/cloud_profile_manager_disabled_16px.png"));
+                                pLayoutProfile->addWidget(m_pProfileToolButton);
+                            }
+
+                            /* Add into layout: */
+                            m_pLayoutProvider->addLayout(pLayoutProfile, 1, 1);
+                        }
+
                         /* Add into layout: */
-                        pLayoutLocation->addLayout(pLayoutProfile);
+                        pLayoutLocation->addLayout(m_pLayoutProvider);
                     }
                 }
@@ -213 +246 @@
     }
 
+    /* Translate provider label: */
+    if (m_pProviderLabel)
+        m_pProviderLabel->setText(UIWizardNewCloudVM::tr("&Location:"));
     /* Translate received values of Location combo-box.
      * We are enumerating starting from 0 for simplicity: */
@@ -230 +266 @@
 
     /* Translate profile stuff: */
+    if (m_pProfileLabel)
+        m_pProfileLabel->setText(UIWizardNewCloudVM::tr("&Profile:"));
     if (m_pProfileToolButton)
         m_pProfileToolButton->setToolTip(UIWizardNewCloudVM::tr("Open Cloud Profile Manager..."));

trunk/src/VBox/Frontends/VirtualBox/src/wizards/newcloudvm/UIWizardNewCloudVMPageExpert.h

@@ -37 +37 @@
 
 /* Forward declarations: */
+class QGridLayout;
+class QLabel;
 class UIToolBox;
 class UIWizardNewCloudVM;
@@ -97 +99 @@
     UIToolBox *m_pToolBox;
 
+    /** Holds the location layout instance. */
+    QGridLayout  *m_pLayoutProvider;
+    /** Holds the provider type label instance. */
+    QLabel       *m_pProviderLabel;
     /** Holds the location type combo-box instance. */
     QIComboBox   *m_pProviderComboBox;
+    /** Holds the profile label instance. */
+    QLabel       *m_pProfileLabel;
     /** Holds the profile combo-box instance. */
     QIComboBox   *m_pProfileComboBox;