You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by "Mikk Lauringson (JIRA)" <ji...@apache.org> on 2011/03/23 11:29:05 UTC
[jira] [Commented] (WICKET-1856) AbstractTree XHTML Strict
validation
[ https://issues.apache.org/jira/browse/WICKET-1856?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13010050#comment-13010050 ]
Mikk Lauringson commented on WICKET-1856:
-----------------------------------------
This problem is still present in latest wicket version 1.4.16. Why is it still unresolved several years later, if a working fix is known?
> AbstractTree XHTML Strict validation
> ------------------------------------
>
> Key: WICKET-1856
> URL: https://issues.apache.org/jira/browse/WICKET-1856
> Project: Wicket
> Issue Type: Improvement
> Components: wicket-core
> Affects Versions: 1.4-M2
> Reporter: Kaspar Fischer
> Assignee: Matej Knopp
> Priority: Trivial
>
> [Notice: Adapted from http://markmail.org/message/afuxccwazkzwc7bz ]
> I have a rootless BaseTree and see it output
> <table style="display:none" id="tree1f_0"></table><table class="wicket-tree-content" id="tree1f_1"><tr>...
> which according to http://validator.w3.org is invalid strict XHTML. Looking at the
> comment in the code, AbstractTree.onRender(),
> // is this root and tree is in rootless mode?
> if (this == rootItem && isRootLess() == true)
> {
> // yes, write empty div with id
> // this is necessary for createElement js to work correctly
> String tagName = ((ComponentTag)markupStream.get()).getName();
> getResponse().write(
> "<" + tagName + " style=\"display:none\" id=\"" + getMarkupId() + "\"></" +
> tagName + ">");
> markupStream.skipComponent();
> }
> it appears from the comment that the indention was to output a div and not a table. With
> a div, we would get valid markup:
> <div style="display:none" id="tree1f_0"></div><table class="wicket-tree-content" id="tree1f_1"><tr>...
> Is there a particular reason to fetch the tagName from the markupStream instead of just hardcoding
> it to "div"?
> Thanks,
> Kaspar
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira