Try to fix some OpenDialog sizing issues, reported by trax (no ticket) and hopefully fix #1538

modules/gui/qt4/components/open_panels.cpp

@@ -86,6 +86,7 @@ FileOpenPanel::FileOpenPanel( QWidget *_parent, intf_thread_t *_p_intf ) :
 
     /* Add a tooltip */
     dialogBox->setToolTip( qtr( "Select one or multiple files" ) );
+    dialogBox->setMinimumHeight( 250 );
 
     // But hide the two OK/Cancel buttons. Enable them for debug.
     QDialogButtonBox *fileDialogAcceptBox =

modules/gui/qt4/ui/open.ui

@@ -9,8 +9,8 @@
    <rect>
     <x>0</x>
     <y>0</y>
-    <width>520</width>
-    <height>668</height>
+    <width>537</width>
+    <height>640</height>
    </rect>
   </property>
   <property name="sizePolicy" >
@@ -28,24 +28,11 @@
   <property name="windowTitle" >
    <string>Dialog</string>
   </property>
-  <layout class="QVBoxLayout" >
-   <item>
-    <widget class="QTabWidget" name="Tab" >
-     <property name="sizePolicy" >
-      <sizepolicy vsizetype="MinimumExpanding" hsizetype="Expanding" >
-       <horstretch>0</horstretch>
-       <verstretch>1</verstretch>
-      </sizepolicy>
-     </property>
-     <property name="minimumSize" >
-      <size>
-       <width>500</width>
-       <height>400</height>
-      </size>
-     </property>
-    </widget>
+  <layout class="QGridLayout" name="gridLayout" >
+   <item row="0" column="0" colspan="4" >
+    <widget class="QTabWidget" name="Tab" />
    </item>
-   <item>
+   <item row="1" column="0" colspan="4" >
     <widget class="QCheckBox" name="advancedCheckBox" >
      <property name="toolTip" >
       <string>_("Show extended options")</string>
@@ -55,9 +42,9 @@
      </property>
     </widget>
    </item>
-   <item>
+   <item row="2" column="0" colspan="4" >
     <widget class="QGroupBox" name="advancedFrame" >
-     <layout class="QGridLayout" >
+     <layout class="QGridLayout" name="gridLayout_2" >
       <item row="0" column="0" >
        <widget class="QLabel" name="cacheLabel" >
         <property name="text" >
@@ -136,17 +123,17 @@
         </property>
        </spacer>
       </item>
-      <item row="1" column="1" colspan="6" >
-       <widget class="Line" name="line" />
-      </item>
-      <item row="2" column="0" colspan="7" >
+      <item row="1" column="0" >
        <widget class="QCheckBox" name="slaveCheckbox" >
         <property name="text" >
          <string>_("Play another media synchronously (extra audio file, ...)")</string>
         </property>
        </widget>
       </item>
-      <item row="3" column="0" >
+      <item row="1" column="1" colspan="6" >
+       <widget class="Line" name="line" />
+      </item>
+      <item row="2" column="0" >
        <widget class="QLabel" name="slaveLabel" >
         <property name="text" >
          <string>_("Extra media")</string>
@@ -156,10 +143,10 @@
         </property>
        </widget>
       </item>
-      <item row="3" column="1" colspan="4" >
+      <item row="2" column="1" colspan="4" >
        <widget class="QLineEdit" name="slaveText" />
       </item>
-      <item row="3" column="5" colspan="2" >
+      <item row="2" column="5" colspan="2" >
        <widget class="QPushButton" name="slaveBrowseButton" >
         <property name="toolTip" >
          <string>_("Select the file")</string>
@@ -169,10 +156,7 @@
         </property>
        </widget>
       </item>
-      <item row="4" column="1" colspan="6" >
-       <widget class="Line" name="line" />
-      </item>
-      <item row="5" column="0" >
+      <item row="4" column="0" >
        <widget class="QLabel" name="advancedLabel" >
         <property name="text" >
          <string>_("Customize")</string>
@@ -182,7 +166,10 @@
         </property>
        </widget>
       </item>
-      <item row="5" column="1" colspan="6" >
+      <item row="3" column="1" colspan="6" >
+       <widget class="Line" name="line" />
+      </item>
+      <item row="4" column="1" colspan="6" >
        <widget class="QLineEdit" name="advancedLineInput" >
         <property name="toolTip" >
          <string>_("Complete MRL for VLC internal")</string>
@@ -192,12 +179,7 @@
      </layout>
     </widget>
    </item>
-   <item>
-    <layout class="QHBoxLayout" >
-     <property name="spacing" >
-      <number>0</number>
-     </property>
-     <item>
+   <item row="3" column="0" >
     <spacer>
      <property name="orientation" >
       <enum>Qt::Horizontal</enum>
@@ -210,7 +192,7 @@
      </property>
     </spacer>
    </item>
-     <item>
+   <item row="3" column="1" >
     <widget class="QPushButton" name="menuButton" >
      <property name="sizePolicy" >
       <sizepolicy vsizetype="Minimum" hsizetype="Fixed" >
@@ -235,7 +217,7 @@
      </property>
     </widget>
    </item>
-     <item>
+   <item row="3" column="2" >
     <widget class="QPushButton" name="playButton" >
      <property name="sizePolicy" >
       <sizepolicy vsizetype="Minimum" hsizetype="Minimum" >
@@ -257,7 +239,7 @@
      </property>
     </widget>
    </item>
-     <item>
+   <item row="3" column="3" >
     <widget class="QDialogButtonBox" name="buttonsBox" >
      <property name="sizePolicy" >
       <sizepolicy vsizetype="Fixed" hsizetype="Maximum" >
@@ -270,9 +252,7 @@
      </property>
     </widget>
    </item>
-    </layout>
-   </item>
-   <item>
+   <item row="4" column="0" colspan="4" >
     <spacer>
      <property name="orientation" >
       <enum>Qt::Vertical</enum>