AbstractTree XHTML Strict validation
------------------------------------

                 Key: WICKET-1856
                 URL: https://issues.apache.org/jira/browse/WICKET-1856
             Project: Wicket
          Issue Type: Improvement
          Components: wicket
    Affects Versions: 1.4-M2
            Reporter: Kaspar Fischer


[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.
-
You can reply to this email to add a comment to the issue online.

Reply via email to