Bug 92833 - usability improvements to datasource config UIs
Summary: usability improvements to datasource config UIs
Status: RESOLVED FIXED
Alias: None
Product: kst
Classification: Applications
Component: general (show other bugs)
Version: 1.x
Platform: unspecified Linux
: NOR wishlist
Target Milestone: ---
Assignee: George Staikos
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-11-07 00:24 UTC by Nicolas Brisset
Modified: 2004-12-06 05:58 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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">