Attempt a better layout for the download dialog

This seems to work well and provides a lot less wasted space.

Signed-off-by: Dirk Hohndel <dirk@hohndel.org>

1 files changed, 74 insertions, 78 deletions

diff --git a/qt-ui/downloadfromdivecomputer.ui b/qt-ui/downloadfromdivecomputer.ui
index 004895038..8d3f5c1dc 100644
--- a/qt-ui/downloadfromdivecomputer.ui
+++ b/qt-ui/downloadfromdivecomputer.ui
@@ -20,109 +20,136 @@
   </property>
   <layout class="QVBoxLayout" name="vendorProductLabelLayout">
    <item>
-    <layout class="QGridLayout" name="vendorProductLayout">
-     <item row="0" column="0">
-      <widget class="QLabel" name="label">
-       <property name="text">
-        <string>Vendor</string>
-       </property>
-      </widget>
-     </item>
-     <item row="0" column="1">
-      <widget class="QLabel" name="label_2">
-       <property name="text">
-        <string>Dive computer</string>
-       </property>
-      </widget>
-     </item>
-     <item row="1" column="0">
-      <widget class="QComboBox" name="vendor"/>
-     </item>
-     <item row="1" column="1">
-      <widget class="QComboBox" name="product"/>
-     </item>
-    </layout>
-   </item>
-   <item>
-    <layout class="QHBoxLayout" name="mainDialogLayout">
+    <layout class="QHBoxLayout" name="mainDialogLayout" stretch="0,1">
      <item>
       <layout class="QVBoxLayout" name="leftOfMainLayout">
        <item>
         <layout class="QGridLayout" name="choicesLayout">
-         <item row="0" column="0">
+         <item row="4" column="0">
           <widget class="QLabel" name="label_3">
            <property name="text">
             <string>Device or mount point</string>
            </property>
           </widget>
          </item>
-         <item row="1" column="0">
+         <item row="5" column="0">
           <widget class="QComboBox" name="device">
            <property name="editable">
             <bool>true</bool>
            </property>
           </widget>
          </item>
-         <item row="1" column="1">
+         <item row="5" column="1">
           <widget class="QToolButton" name="search">
            <property name="text">
             <string>...</string>
            </property>
           </widget>
          </item>
-         <item row="2" column="0">
+         <item row="6" column="0">
           <widget class="QCheckBox" name="forceDownload">
            <property name="text">
             <string>Force download of all dives</string>
            </property>
           </widget>
          </item>
-         <item row="3" column="0">
+         <item row="7" column="0">
           <widget class="QCheckBox" name="preferDownloaded">
            <property name="text">
             <string>Always prefer downloaded dives</string>
            </property>
           </widget>
          </item>
-         <item row="4" column="0">
+         <item row="8" column="0">
           <widget class="QCheckBox" name="createNewTrip">
            <property name="text">
             <string>Download into new trip</string>
            </property>
           </widget>
          </item>
-         <item row="5" column="0">
+         <item row="9" column="0">
           <widget class="QCheckBox" name="logToFile">
            <property name="text">
             <string>Save libdivecomputer logfile</string>
            </property>
           </widget>
          </item>
-         <item row="5" column="1">
+         <item row="9" column="1">
           <widget class="QToolButton" name="chooseLogFile">
            <property name="text">
             <string>...</string>
            </property>
           </widget>
          </item>
-         <item row="6" column="0">
+         <item row="10" column="0">
           <widget class="QCheckBox" name="dumpToFile">
            <property name="text">
             <string>Save libdivecomputer dumpfile</string>
            </property>
           </widget>
          </item>
-         <item row="6" column="1">
+         <item row="10" column="1">
           <widget class="QToolButton" name="chooseDumpFile">
            <property name="text">
             <string>...</string>
            </property>
           </widget>
          </item>
+         <item row="0" column="0" colspan="2">
+          <widget class="QLabel" name="label">
+           <property name="text">
+            <string>Vendor</string>
+           </property>
+          </widget>
+         </item>
+         <item row="1" column="0" colspan="2">
+          <widget class="QComboBox" name="vendor"/>
+         </item>
+         <item row="2" column="0" colspan="2">
+          <widget class="QLabel" name="label_2">
+           <property name="text">
+            <string>Dive computer</string>
+           </property>
+          </widget>
+         </item>
+         <item row="3" column="0" colspan="2">
+          <widget class="QComboBox" name="product"/>
+         </item>
+        </layout>
+       </item>
+       <item>
+        <layout class="QHBoxLayout" name="downloadCancelRetryLayout">
+         <item>
+          <spacer name="horizontalSpacer_2">
+           <property name="orientation">
+            <enum>Qt::Horizontal</enum>
+           </property>
+           <property name="sizeHint" stdset="0">
+            <size>
+             <width>40</width>
+             <height>20</height>
+            </size>
+           </property>
+          </spacer>
+         </item>
+         <item>
+          <widget class="QPushButton" name="downloadCancelRetryButton">
+           <property name="text">
+            <string>Download</string>
+           </property>
+          </widget>
+         </item>
         </layout>
        </item>
        <item>
-        <spacer name="verticalSpacer">
+        <widget class="QProgressBar" name="progressBar">
+         <property name="value">
+          <number>0</number>
+         </property>
+        </widget>
+       </item>
+       <item>
+        <spacer name="aboveOKCancelSpacer">
          <property name="orientation">
           <enum>Qt::Vertical</enum>
          </property>
@@ -135,9 +162,9 @@
         </spacer>
        </item>
        <item>
-        <layout class="QHBoxLayout" name="downloadCancelRetryLayout">
+        <layout class="QHBoxLayout" name="buttonBoxLayout">
          <item>
-          <spacer name="horizontalSpacer_2">
+          <spacer name="horizontalSpacer">
            <property name="orientation">
             <enum>Qt::Horizontal</enum>
            </property>
@@ -150,9 +177,16 @@
           </spacer>
          </item>
          <item>
-          <widget class="QPushButton" name="downloadCancelRetryButton">
+          <widget class="QPushButton" name="ok">
            <property name="text">
-            <string>Download</string>
+            <string>OK</string>
+           </property>
+          </widget>
+         </item>
+         <item>
+          <widget class="QPushButton" name="cancel">
+           <property name="text">
+            <string>Cancel</string>
            </property>
           </widget>
          </item>
@@ -211,44 +245,6 @@
      </item>
     </layout>
    </item>
-   <item>
-    <layout class="QHBoxLayout" name="buttonBoxLayout">
-     <item>
-      <spacer name="horizontalSpacer">
-       <property name="orientation">
-        <enum>Qt::Horizontal</enum>
-       </property>
-       <property name="sizeHint" stdset="0">
-        <size>
-         <width>40</width>
-         <height>20</height>
-        </size>
-       </property>
-      </spacer>
-     </item>
-     <item>
-      <widget class="QPushButton" name="ok">
-       <property name="text">
-        <string>OK</string>
-       </property>
-      </widget>
-     </item>
-     <item>
-      <widget class="QPushButton" name="cancel">
-       <property name="text">
-        <string>Cancel</string>
-       </property>
-      </widget>
-     </item>
-    </layout>
-   </item>
-   <item>
-    <widget class="QProgressBar" name="progressBar">
-     <property name="value">
-      <number>0</number>
-     </property>
-    </widget>
-   </item>
   </layout>
  </widget>
  <resources/>