[
https://issues.apache.org/jira/browse/WICKET-4153?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13133663#comment-13133663
]
Bertrand Guay-Paquet commented on WICKET-4153:
----------------------------------------------
@Sander: your patch does produce valid markup. However, when there are no
records, the headers become part of the implicit tbody section instead ot thead
which can wreck CSS styling.
@All: It's too bad that my patch is deemed too intrusive. I can't just ignore
the rendering bug present in Firefox until this is fixed when an easy
workaround that works for me is available. I'll have to keep on using my own
version of datatable.
Would putting the no-records row in the tbody instead of tfoot be acceptable? I
very well might be doing such a modification to my DataTable anyway so I could
provide a patch.
> The tbody section of a DataTable is empty when no records are returned by the
> provider.
> ---------------------------------------------------------------------------------------
>
> Key: WICKET-4153
> URL: https://issues.apache.org/jira/browse/WICKET-4153
> Project: Wicket
> Issue Type: Bug
> Components: wicket-extensions
> Affects Versions: 1.5.1
> Reporter: Bertrand Guay-Paquet
> Priority: Trivial
> Labels: datatable
> Fix For: 1.5.3
>
> Attachments: DataTable-usePlainTR.diff, DataTable.patch,
> DataTableTest-usePlainTR.diff, good.png, with_patch.png, without_patch.png
>
>
> When a DataTable is rendered without records, the tbody section is empty.
> This violates the html spec.
> From the spec:
> "When present, each THEAD, TFOOT, and TBODY contains a row group. Each row
> group must contain at least one row, defined by the TR element."
> and
> "The THEAD, TFOOT, and TBODY sections must contain the same number of
> columns."
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
