Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 5.2.0, 5.1.0.5
    • Fix Version/s: 5.3, 5.2.5
    • Component/s: tapestry-core
    • Labels:
      None

      Description

      According to the HTML5 Working Draft [1], the doctype to be used in the case you are using the HTML syntax is:

      <!DOCTYPE HTML> case-insensitive

      Unfortunately the class org.apache.tapestry5.dom.DTD doesn't allow doctypes with empty public and system identifier. It is also not possible to output the DOCTYPE legacy string <!DOCTYPE HTML SYSTEM "about:legacy-compat">, because some parser internal to Tapestry tries to resolve the system identifier. The best thing, I was able to output is the full HTML 4.01 strict doctype:

      <!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01//EN" "http://www.w3.org/TR/html4/strict.dtd">

      It is allowed in HTML5 as obsolete permitted DOCTYPE string, but not desired.

      It would be very simple to allow for what empty HTML5 doctype in org.apache.tapestry5.dom.DTD.

      [1]: http://www.w3.org/TR/2010/WD-html5-20100304/

        Activity

        Howard M. Lewis Ship made changes -
        Fix Version/s 5.3 [ 12316024 ]
        Fix Version/s 5.3.0 [ 12316023 ]
        Josh Canfield made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Josh Canfield made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Fix Version/s 5.3.0 [ 12316023 ]
        Fix Version/s 5.2.5 [ 12315565 ]
        Resolution Fixed [ 1 ]
        Josh Canfield made changes -
        Assignee Josh Canfield [ joshcanfield ]
        Alexander Kiel made changes -
        Field Original Value New Value
        Attachment added_ability_to_print_the_HTML5_DOCTYPE.patch [ 12437745 ]
        Alexander Kiel created issue -

          People

          • Assignee:
            Josh Canfield
            Reporter:
            Alexander Kiel
          • Votes:
            2 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development