svn commit: r516668 - /ofbiz/trunk/framework/widget/src/org/ofbiz/widget/html/HtmlFormRenderer.java

[jacopoc]

Author: jacopoc
Date: Sat Mar 10 01:10:31 2007
New Revision: 516668

URL: http://svn.apache.org/viewvc?view=rev&rev=516668
Log:
Applied patch from Adrian Crum (OFBIZ-795) with misc layout improvement for form widgets.
Some minor formatting fixes.

Modified:
    ofbiz/trunk/framework/widget/src/org/ofbiz/widget/html/HtmlFormRenderer.java

Modified: ofbiz/trunk/framework/widget/src/org/ofbiz/widget/html/HtmlFormRenderer.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/widget/src/org/ofbiz/widget/html/HtmlFormRenderer.java?view=diff&rev=516668&r1=516667&r2=516668
==============================================================================
--- ofbiz/trunk/framework/widget/src/org/ofbiz/widget/html/HtmlFormRenderer.java (original)
+++ ofbiz/trunk/framework/widget/src/org/ofbiz/widget/html/HtmlFormRenderer.java Sat Mar 10 01:10:31 2007
@@ -1067,13 +1067,9 @@
             this.appendWhitespace(buffer);
             buffer.append("<script language=\"JavaScript\" type=\"text/javascript\">");
             this.appendWhitespace(buffer);
-            buffer.append("<!--//");
-            this.appendWhitespace(buffer);
             buffer.append("document." + modelForm.getCurrentFormName(context) + ".");
             buffer.append(focusFieldName + ".focus();");
             this.appendWhitespace(buffer);
-            buffer.append("//-->");
-            this.appendWhitespace(buffer);
             buffer.append("</script>");
         }
         this.appendWhitespace(buffer);
@@ -1130,16 +1126,15 @@
 
         buffer.append("<!-- begin form widget -->");
         this.appendWhitespace(buffer);
-         if(UtilValidate.isNotEmpty(modelForm.getDefaultTableStyle())) {
-             buffer.append(" <table");
-             buffer.append(" class=\"");
-             buffer.append(modelForm.getDefaultTableStyle());
-             buffer.append("\"");
-             buffer.append("cellspacing=\"0\">");
-         } else {
-             buffer.append(" <table cellspacing=\"0\" class=\"basic-table dark-grid\">");
-             // DEJ 20050101 removed the width=\"100%\", doesn't look very good with CSS float: left based side "columns"
-         }
+        if(UtilValidate.isNotEmpty(modelForm.getDefaultTableStyle())) {
+            buffer.append(" <table");
+            buffer.append(" class=\"");
+            buffer.append(modelForm.getDefaultTableStyle());
+            buffer.append("\" cellspacing=\"0\">");
+        } else {
+            buffer.append(" <table cellspacing=\"0\" class=\"basic-table form-widget-table dark-grid\">");
+            // DEJ 20050101 removed the width=\"100%\", doesn't look very good with CSS float: left based side "columns"
+        }
 
         this.appendWhitespace(buffer);
     }