JAL-591 ensure slider hidden if text input field only
authorgmungoc <g.m.carstairs@dundee.ac.uk>
Mon, 23 Apr 2018 08:41:59 +0000 (09:41 +0100)
committergmungoc <g.m.carstairs@dundee.ac.uk>
Mon, 23 Apr 2018 08:47:54 +0000 (09:47 +0100)
src/jalview/gui/OptsAndParamsPage.java

index fcb293c..483ba3d 100644 (file)
@@ -314,6 +314,8 @@ public class OptsAndParamsPage
 
     boolean isIntegerParameter;
 
+    boolean isStringParameter;
+
     boolean adjusting;
 
     JComboBox<String> choicebox;
@@ -358,12 +360,14 @@ public class OptsAndParamsPage
       {
         ValueType type = validator.getType();
         isIntegerParameter = type == ValueType.Integer;
+        isStringParameter = type == ValueType.String;
 
         /*
          * ensure slider has an integer range corresponding to
          * the min-max range of the parameter
          */
-        if (validator.getMin() != null && !isIntegerParameter)
+        if (validator.getMin() != null && !isIntegerParameter
+                && !isStringParameter)
         {
           double min = validator.getMin().doubleValue();
           double max = validator.getMax().doubleValue();
@@ -752,7 +756,7 @@ public class OptsAndParamsPage
      */
     Object updateSliderFromValueField()
     {
-      if (validator == null)
+      if (validator == null || isStringParameter)
       {
         if (isChoiceParameter)
         {