ICEfaces
  1. ICEfaces
  2. ICE-2393

XHTML: standard JSF components, <img> should be self-closed

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 1.6.2
    • Fix Version/s: 1.7Beta1, 1.7
    • Component/s: Framework
    • Labels:
      None
    • Environment:
      All

      Description

      The <ice:graphicImage> component and the <ice:dataTable> component using the scrollable="true" attribute-value pair render <img> elements that have separate </img> closing tags, but the DTD defines <img> as an empty element. So, it is more appropriate to render these components with self-closed <input /> elements.

        Issue Links

          Activity

          Hide
          Mark Collette added a comment -

          ICE-2530 has a code patch to accomplish this.

          Show
          Mark Collette added a comment - ICE-2530 has a code patch to accomplish this.
          Hide
          Greg Dick added a comment -

          I checked the code with 2530, it seems correct, and addresses a number of problems in the existing code. I checked the xhtml documentation and found these tags that seem candidates for short form tags.

          "img, input, br, hr, meta, base, link, frame, col, area"

          The patch allows for attributes of <br> to be encoded properly.

          Show
          Greg Dick added a comment - I checked the code with 2530, it seems correct, and addresses a number of problems in the existing code. I checked the xhtml documentation and found these tags that seem candidates for short form tags. "img, input, br, hr, meta, base, link, frame, col, area" The patch allows for attributes of <br> to be encoded properly.

            People

            • Assignee:
              Unassigned
              Reporter:
              Arturo Zambrano
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: