Priority: Normal
Current State: Closed
Released In: 1.1.4
Target Release: patch
The core table code in Render.pm doesn't have any unit test coverage.
Created unit tests and noted issues:
- <th> and <strong> are redundant in the table header. Should not need to generate the strong markup.
- Code doesn't support generating table footers.
Since this behavior has been around forever, and is overridden by the
TablePlugin, setting as enhancement, target Release 1.2
--
GeorgeClark - 13 Oct 2011
Found another issue - A list adjacent to a table doesn't terminate the list, so the table html runs together with the list markup. Fix to both were rather simple. Committing fixes to 1.2 - should I also apply to 1.1.4? Unit tests pass. Changed to normal. These are bugs, but masked by
TablePlugin.
--
GeorgeClark - 13 Oct 2011
Between George and I we have fixed the
core table rendering. However the
TablePlugin remains at fault.
--
CrawfordCurrie - 14 Oct 2011
Will sync over to 1.1.4 - given that
TablePlugin generally masks this issue, fix is low risk.
--
GeorgeClark - 14 Oct 2011