You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by ja...@apache.org on 2007/10/20 11:45:51 UTC

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

Author: jacopoc
Date: Sat Oct 20 02:45:50 2007
New Revision: 586690

URL: http://svn.apache.org/viewvc?rev=586690&view=rev
Log:
Applied (slightli different version od) patch from Marco Risaliti (OFBIZ-1355) to fix the ugly layout for empty cells in IE.

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?rev=586690&r1=586689&r2=586690&view=diff
==============================================================================
--- ofbiz/trunk/framework/widget/src/org/ofbiz/widget/html/HtmlFormRenderer.java (original)
+++ ofbiz/trunk/framework/widget/src/org/ofbiz/widget/html/HtmlFormRenderer.java Sat Oct 20 02:45:50 2007
@@ -984,7 +984,14 @@
                 buffer.append(modelFormField.getTitleStyle());
                 buffer.append("\">");
             }
-            renderHyperlinkTitle(buffer, context, modelFormField, titleText);         
+            if (" ".equals(titleText)) {
+                // If the title content is just a blank then render it colling renderFormatEmptySpace:
+                // the method will set its content to work fine in most browser
+                this.renderFormatEmptySpace(buffer, context, modelFormField.getModelForm());
+            } else {
+                renderHyperlinkTitle(buffer, context, modelFormField, titleText);
+            }
+
             if (UtilValidate.isNotEmpty(modelFormField.getTitleStyle())) {
                 buffer.append("</span>");
             }