r2527 - people/synrg/xjig

[Ben Armstrong]

Author: synrg
Date: 2007-04-04 19:05:41 +0000 (Wed, 04 Apr 2007)
New Revision: 2527

Modified:
   people/synrg/xjig/xjig-menu
Log:
Kdialog re-enabled for testing

Modified: people/synrg/xjig/xjig-menu
===================================================================
--- people/synrg/xjig/xjig-menu	2007-04-04 18:38:39 UTC (rev 2526)
+++ people/synrg/xjig/xjig-menu	2007-04-04 19:05:41 UTC (rev 2527)
@@ -150,7 +150,7 @@
          dialog_answer="$(kdialog --error "${2}")"
          ;;
       text)
-			# FIXME: in testing, dimensions did not match 40x1
+         # FIXME: in testing, dimensions did not match 40x1
          dialog_answer="$(kdialog --title "${2}" --textinputbox "${3}" "" 40 1)"
          ;;
       file)
@@ -288,14 +288,12 @@
 Validate ()
 {
    case "$dialog" in
-# FIXME: In tests on a non-KDE desktop kdialog was so slow it was
-#			unusable.  Disabled until I can figure out why and fix it.
-#      kdialog)
-#         if [ ! -x /usr/bin/kdialog ]; then
-#            dialog="none"
-#            Dialog error "${PROGRAM}: Missing /usr/bin/kdialog"
-#         fi
-#         ;;
+      kdialog)
+         if [ ! -x /usr/bin/kdialog ]; then
+            dialog="none"
+            Dialog error "${PROGRAM}: Missing /usr/bin/kdialog"
+         fi
+         ;;
       zenity)
          if [ ! -x /usr/bin/zenity ]; then
             dialog="none"
@@ -304,14 +302,16 @@
          ;;
       auto)
 # FIXME: We currently prefer to use kdialog over zenity because it provides
-#			image previews.  But ideally the dialog chosen by 'auto' should
-#			match the user's session type (kdialog for KDE, zenity for
-#			GNOME & XFCE).  If zenity ever supports previews, do this kind
-#			of autodetection of the user's desktop.
+#        image previews.  But ideally the dialog chosen by 'auto' should
+#        match the user's session type (kdialog for KDE, zenity for
+#        GNOME & XFCE).  If zenity ever supports previews, do this kind
+#        of autodetection of the user's desktop.
+# FIXME: In tests on a non-KDE desktop kdialog was so slow it was
+#        unusable.  Disabled until I can figure out why and fix it.
 #         if [ -x /usr/bin/kdialog ]; then
 #            dialog="kdialog"
 #         elif [ -x /usr/bin/zenity ]; then
-			if [ -x /usr/bin/zenity ]; then
+         if [ -x /usr/bin/zenity ]; then
             dialog="zenity"
          else
             dialog="none"