Bug 92833

Summary:	usability improvements to datasource config UIs
Product:	[Applications] kst	Reporter:	Nicolas Brisset <nicolas.brisset>
Component:	general	Assignee:	George Staikos <staikos>
Status:	RESOLVED FIXED
Severity:	wishlist
Priority:	NOR
Version First Reported In:	1.x
Target Milestone:	---
Platform:	unspecified
OS:	Linux
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:

Description Nicolas Brisset 2004-11-07 00:24:47 UTC

Version:           1.0.0_pre1 (using KDE 3.2 BRANCH >= 20040204, Mandrake Linux Cooker i586 - Cooker)
Compiler:          gcc version 3.3.2 (Mandrake Linux 10.0 3.3.2-6mdk)
OS:                Linux (i686) release 2.6.3-7mdk

It is nice to have the possibility to configure datasources, but I believe we could and should make it nicer/more usable by:
1) moving the "Configure..." button in the datawizard below, next to the label stating the type of source instead of right to the file selector
2) in the kst settings/Data Sources tab, displaying a text field like "No configuration options available for this data source" when there is none (when "Configure" is inactive right now) in the empty space below the combo box, and embedding the config dialog in the same empty space when it exists (hoping there won't ever be such a complex config dialog that embedding it looks bad !)

Comment 1 George Staikos 2004-12-06 05:45:28 UTC

#2 is WONTFIX (redundant, and the embedding idea is too slow)

Comment 2 George Staikos 2004-12-06 05:58:47 UTC

CVS commit by staikos: 

move the configure button down one row
FEATURE: 92833


  M +38 -36    datawizard.ui   1.68


--- kdeextragear-2/kst/kst/datawizard.ui  #1.67:1.68
@@ -80,5 +80,5 @@
                 </property>
             </spacer>
-            <widget class="KURLRequester" row="2" column="1">
+            <widget class="KURLRequester" row="2" column="1" colspan="2" rowspan="1">
                 <property name="name">
                     <cstring>_url</cstring>
@@ -93,23 +93,4 @@
                 </property>
             </widget>
-            <widget class="QPushButton" row="2" column="2">
-                <property name="name">
-                    <cstring>_configureSource</cstring>
-                </property>
-                <property name="enabled">
-                    <bool>false</bool>
-                </property>
-                <property name="text">
-                    <string>Con&amp;figure...</string>
-                </property>
-                <property name="sizePolicy">
-                    <sizepolicy>
-                        <hsizetype>5</hsizetype>
-                        <vsizetype>5</vsizetype>
-                        <horstretch>1</horstretch>
-                        <verstretch>0</verstretch>
-                    </sizepolicy>
-                </property>
-            </widget>
             <spacer row="2" column="3">
                 <property name="name">
@@ -129,20 +110,41 @@
                 </property>
             </spacer>
-            <widget class="QLabel" row="3" column="1" colspan="2" rowspan="1">
-                <property name="name">
-                    <cstring>_fileType</cstring>
-                </property>
-                <property name="alignment">
-                    <set>WordBreak|AlignTop</set>
-                </property>
-                <property name="sizePolicy">
-                    <sizepolicy>
-                        <hsizetype>5</hsizetype>
-                        <vsizetype>5</vsizetype>
-                        <horstretch>1</horstretch>
-                        <verstretch>0</verstretch>
-                    </sizepolicy>
-                </property>
-            </widget>
+            <hbox row="3" column="1" colspan="2" rowspan="1">
+                <widget class="QLabel">
+                    <property name="name">
+                        <cstring>_fileType</cstring>
+                    </property>
+                    <property name="alignment">
+                        <set>WordBreak|AlignTop</set>
+                    </property>
+                    <property name="sizePolicy">
+                        <sizepolicy>
+                            <hsizetype>5</hsizetype>
+                            <vsizetype>5</vsizetype>
+                            <horstretch>2</horstretch>
+                            <verstretch>0</verstretch>
+                        </sizepolicy>
+                    </property>
+                </widget>
+                <widget class="QPushButton">
+                    <property name="name">
+                        <cstring>_configureSource</cstring>
+                    </property>
+                    <property name="enabled">
+                        <bool>false</bool>
+                    </property>
+                    <property name="text">
+                        <string>Con&amp;figure...</string>
+                    </property>
+                    <property name="sizePolicy">
+                        <sizepolicy>
+                            <hsizetype>5</hsizetype>
+                            <vsizetype>5</vsizetype>
+                            <horstretch>0</horstretch>
+                            <verstretch>0</verstretch>
+                        </sizepolicy>
+                    </property>
+                </widget>
+            </hbox>
             <spacer row="4" column="1">
                 <property name="name">