Fixed 'v-required-field-indicator' classname and * character

diff --git a/src/com/vaadin/terminal/gwt/client/ui/VBoxLayout.java b/src/com/vaadin/terminal/gwt/client/ui/VBoxLayout.java
index cc4150e9fe31f6a213d7694ad05142c3eb3bef38..b102fdb22316963909bc9ddfa0a6b2047d89f8bf 100644 (file)
--- a/src/com/vaadin/terminal/gwt/client/ui/VBoxLayout.java
+++ b/src/com/vaadin/terminal/gwt/client/ui/VBoxLayout.java
@@ -224,6 +224,8 @@ public class VBoxLayout extends FlowPanel {
             return captionPosition;
         }
 
+        // TODO refactor VCaption and use that instead: creates a tight coupling
+        // between this layout and Vaadin, but it's already coupled
         public void setCaption(String captionText, String iconUrl,
                 List<String> styles, String error, boolean required) {
 
@@ -292,7 +294,9 @@ public class VBoxLayout extends FlowPanel {
             if (required) {
                 if (requiredIcon == null) {
                     requiredIcon = DOM.createSpan();
-                    requiredIcon.setClassName("v-required-indicator");
+                    // TODO decide something better
+                    requiredIcon.setInnerHTML("*");
+                    requiredIcon.setClassName("v-required-field-indicator");
                 }
                 caption.appendChild(requiredIcon);
             } else if (requiredIcon != null) {

diff --git a/tests/testbench/com/vaadin/tests/components/orderedlayout/BoxLayoutTest.java b/tests/testbench/com/vaadin/tests/components/orderedlayout/BoxLayoutTest.java
index ca67ed64352faabbb3554cf631c734a1391968a4..e52979267af3d4a5c697646c03027c3a4b01286b 100644 (file)
--- a/tests/testbench/com/vaadin/tests/components/orderedlayout/BoxLayoutTest.java
+++ b/tests/testbench/com/vaadin/tests/components/orderedlayout/BoxLayoutTest.java
@@ -13,6 +13,7 @@ import com.vaadin.terminal.UserError;
 import com.vaadin.terminal.WrappedRequest;
 import com.vaadin.tests.components.AbstractTestRoot;
 import com.vaadin.ui.AbstractComponent;
+import com.vaadin.ui.AbstractField;
 import com.vaadin.ui.AbstractOrderedLayout;
 import com.vaadin.ui.Alignment;
 import com.vaadin.ui.Button;
@@ -43,6 +44,7 @@ public class BoxLayoutTest extends AbstractTestRoot {
     protected NativeSelect componentIcon;
     protected TextField componentDescription;
     protected CheckBox componentError;
+    protected CheckBox componentRequired;
 
     protected NativeSelect align;
     protected CheckBox expand;
@@ -113,7 +115,8 @@ public class BoxLayoutTest extends AbstractTestRoot {
                         grid.addComponent(new Label("Grid cell 2"));
                         grid.addComponent(grow);
                         grid.addComponent(new Label("Grid cell 4"));
-                        l.addComponent(grid);
+                        // l.addComponent(grid);
+                        l.addComponent(new TextField("Some field"));
                     }
                 });
         header.addComponent(addComponent);
@@ -328,6 +331,19 @@ public class BoxLayoutTest extends AbstractTestRoot {
         });
         component.addComponent(componentError);
 
+        componentRequired = new CheckBox("Required");
+        componentRequired.setImmediate(true);
+        componentRequired.setEnabled(false);
+        componentRequired.addListener(new ValueChangeListener() {
+            public void valueChange(ValueChangeEvent event) {
+                if (target != null && target instanceof AbstractField) {
+                    ((AbstractField<?>) target).setRequired(componentRequired
+                            .getValue());
+                }
+            }
+        });
+        component.addComponent(componentRequired);
+
         return root;
     }
 
@@ -361,6 +377,8 @@ public class BoxLayoutTest extends AbstractTestRoot {
                 componentIcon.setEnabled(target != null);
                 componentDescription.setEnabled(target != null);
                 componentError.setEnabled(target != null);
+                componentRequired.setEnabled(target != null
+                        && target instanceof AbstractField);
                 align.setEnabled(target != null);
                 expand.setEnabled(target != null);
                 if (target != null) {
@@ -391,6 +409,10 @@ public class BoxLayoutTest extends AbstractTestRoot {
                     }
                     componentDescription.setValue(target.getDescription());
                     componentError.setValue(target.getComponentError() != null);
+                    if (target instanceof AbstractField) {
+                        componentRequired.setValue(((AbstractField<?>) target)
+                                .isRequired());
+                    }
                 }
             }
         });