Uploaded image for project: 'Wicket'
  1. Wicket
  2. WICKET-5237

Wicket generates invalid HTML by expanding col tags

Attach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Bug
    • Status: Resolved
    • Minor
    • Resolution: Fixed
    • 6.8.0
    • 6.9.0, 7.0.0-M1
    • None

    Description

      hi,

      I just noticed that wicket expands col tags, even though the (x)html specifications forbids it.

      take this markup as an example:

      <table>
      <colgroup>
      <col width="20%" />
      <col width="80%" />
      </colgroup>
      <tbody>
      <tr>
      <td>I take a fifth of the available space</td>
      <td>I take four fifth of the available space</td>
      </tr>
      </tbody>
      </table>

      Instead of return this as-is, it get's converted to:

      <table>
      <colgroup>
      <col width="20%"></col>
      <col width="80%"></col>
      </colgroup>
      <tbody>
      <tr>
      <td>I take a fifth of the available space</td>
      <td>I take four fifth of the available space</td>
      </tr>
      </tbody>
      </table>

      But the specifications mention that col tags must not have end tags. This may be related to WICKET-2765, as this seems to be the point when col was added to the OpenCloseTagExpander class. Note that it is ok to have a non closing col tag in html (self-closing in xhtml). It's all about generating a separated end tag.

      This happens in wicket 6.8, but I guess it's relevant to all versions down to wicket 1.4.

      Specs for reference:

      http://www.w3.org/TR/1999/REC-html401-19991224/struct/tables.html#edef-COL
      http://www.w3.org/TR/html-markup/col.html

      Kind regards,

      Konrad

      Attachments

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            mgrigorov Martin Tzvetanov Grigorov
            kmohrf Konrad Mohrfeldt
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment