You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by mg...@apache.org on 2011/04/18 19:14:51 UTC

svn commit: r1094661 - /wicket/trunk/wicket-core/src/main/java/org/apache/wicket/markup/html/tree/AbstractTree.java

Author: mgrigorov
Date: Mon Apr 18 17:14:51 2011
New Revision: 1094661

URL: http://svn.apache.org/viewvc?rev=1094661&view=rev
Log:
WICKET-1856 AbstractTree XHTML Strict validation

Add empty table body to the already hidden <table> when there is nothing to show.

Modified:
    wicket/trunk/wicket-core/src/main/java/org/apache/wicket/markup/html/tree/AbstractTree.java

Modified: wicket/trunk/wicket-core/src/main/java/org/apache/wicket/markup/html/tree/AbstractTree.java
URL: http://svn.apache.org/viewvc/wicket/trunk/wicket-core/src/main/java/org/apache/wicket/markup/html/tree/AbstractTree.java?rev=1094661&r1=1094660&r2=1094661&view=diff
==============================================================================
--- wicket/trunk/wicket-core/src/main/java/org/apache/wicket/markup/html/tree/AbstractTree.java (original)
+++ wicket/trunk/wicket-core/src/main/java/org/apache/wicket/markup/html/tree/AbstractTree.java Mon Apr 18 17:14:51 2011
@@ -45,6 +45,7 @@ import org.apache.wicket.model.IDetachab
 import org.apache.wicket.model.IModel;
 import org.apache.wicket.model.Model;
 import org.apache.wicket.request.IRequestHandler;
+import org.apache.wicket.request.Response;
 import org.apache.wicket.request.resource.JavaScriptResourceReference;
 import org.apache.wicket.request.resource.ResourceReference;
 import org.apache.wicket.util.string.AppendingStringBuffer;
@@ -216,9 +217,14 @@ public abstract class AbstractTree exten
 				// yes, write empty div with id
 				// this is necessary for createElement js to work correctly
 				String tagName = ((ComponentTag)getMarkup().get(0)).getName();
-				getResponse().write(
-					"<" + tagName + " style=\"display:none\" id=\"" + getMarkupId() + "\"></" +
-						tagName + ">");
+				Response response = getResponse();
+				response.write("<" + tagName + " style=\"display:none\" id=\"" + getMarkupId() +
+					"\">");
+				if ("table".equals(tagName))
+				{
+					response.write("<tbody><tr><td></td></tr></tbody>");
+				}
+				response.write("</" + tagName + ">");
 			}
 			else
 			{