From: Joonas Lehtinen <joonas.lehtinen@itmill.com>
Date: Sat, 14 Jun 2008 19:12:47 +0000 (+0000)
Subject: Fixed #1807 : Required-fields should have "i-required" classname for better styling
X-Git-Tag: 6.7.0.beta1~4588
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=970f6581027e522321b822f74aa3f4a4835758a5;p=vaadin-framework.git

Fixed #1807 : Required-fields should have "i-required" classname for better styling

svn changeset:4888/svn branch:trunk
---

diff --git a/src/com/itmill/toolkit/terminal/gwt/client/ApplicationConnection.java b/src/com/itmill/toolkit/terminal/gwt/client/ApplicationConnection.java
index 835d89ad65..9bb3647638 100755
--- a/src/com/itmill/toolkit/terminal/gwt/client/ApplicationConnection.java
+++ b/src/com/itmill/toolkit/terminal/gwt/client/ApplicationConnection.java
@@ -43,6 +43,9 @@ import com.itmill.toolkit.terminal.gwt.client.ui.Notification.HideEvent;
  */
 public class ApplicationConnection {
     private static final String MODIFIED_CLASSNAME = "i-modified";
+
+    private static final String REQUIRED_CLASSNAME_EXT = "-required";
+
     private static final String ERROR_CLASSNAME_EXT = "-error";
 
     public static final String VAR_RECORD_SEPARATOR = "\u001e";
@@ -90,13 +93,13 @@ public class ApplicationConnection {
      * 
      * @SuppressWarnings
      */
-    private JavaScriptObject ttClientWrapper = null;
+    private final JavaScriptObject ttClientWrapper = null;
 
     private int activeRequests = 0;
 
-    private ApplicationConfiguration configuration;
+    private final ApplicationConfiguration configuration;
 
-    private Vector pendingVariableBursts = new Vector();
+    private final Vector pendingVariableBursts = new Vector();
 
     public ApplicationConnection(WidgetSet widgetSet,
             ApplicationConfiguration cnf) {
@@ -815,6 +818,13 @@ public class ApplicationConnection {
             tooltipInfo.setErrorUidl(null);
         }
 
+        // add required style to required components
+        if (uidl.hasAttribute("required")) {
+            styleBuf.append(" ");
+            styleBuf.append(primaryName);
+            styleBuf.append(REQUIRED_CLASSNAME_EXT);
+        }
+
         // Styles + disabled & readonly
         component.setStyleName(styleBuf.toString());