Changeset 4090

Timestamp:

Dec 23, 2010, 10:49:15 AM (12 years ago)

Author:

olle

Message:

Refs #711. Refs #668. Field set show/hide toggle updated to allow the feature for single field sets, if specifically called for:

1. Class/file gui/form/Fieldset.java in client/servlet/ updated with new private instance variable Boolean singleFieldsetToggleAllowed with public accessor methods. The default value of singleFieldsetToggleAllowed is false, and will therefore not change the appearance of forms if not specifically called for.

2. Class/file gui/web/GUIConverter.java in client/servlet/ updated in public method Tag convert(org.proteios.gui.form.Form w) to allow the show/hide toggle feature for a single field set, if value of singleFieldsetToggleAllowed is true.

Location:

trunk/client/servlet/src/org/proteios/gui

Files:

• form/Fieldset.java (modified) (2 diffs)
• web/GUIConverter.java (modified) (1 diff)

trunk/client/servlet/src/org/proteios/gui/form/Fieldset.java

@@ -45 +45 @@
     */
     private Boolean open = true;
+
+    /*
+      Set the flag to true to allow a single fieldset to have the show/hide toggle feature.
+     */
+    private Boolean singleFieldsetToggleAllowed = false;
 
     public Fieldset()
@@ -126 +131 @@
         return this.open;
     }
+
+    public void setSingleFieldsetToggleAllowed(Boolean singleFieldsetToggleAllowed) 
+    {
+        this.singleFieldsetToggleAllowed = singleFieldsetToggleAllowed;
+    }
+
+    public Boolean isSingleFieldsetToggleAllowed() 
+    {
+        return this.singleFieldsetToggleAllowed;
+    }
 }

trunk/client/servlet/src/org/proteios/gui/web/GUIConverter.java

@@ -1239 +1239 @@
             for (org.proteios.gui.form.Fieldset fs : fieldsets)
             {
-                Tag fieldset = convertFieldset(fs, form, fieldsets.size() > 1 ? true : false);
+              // Allow fieldset show/hide toggle if several fieldsets, or specifically allowed
+                Tag fieldset = convertFieldset(fs, form, (fieldsets.size() > 1 || (fieldsets.size() == 1 && fieldsets.get(0).isSingleFieldsetToggleAllowed() == true)) ? true : false);
                 form.add(fieldset);
             }