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>");
}