Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 4.0
    • Fix Version/s: 4.1
    • Component/s: Contrib, Framework
    • Labels:
      None

      Description

      I think the Tapestry components should generate valid XHTML 1.0 Strict. Components that generate XHTML can still be used to build HTML 4.01 pages, so XHTML 1.0 Strict seems to be the best choice.

      Currently there are the following XHTML issues with Tapestry:

      *) some tag attributes are not quoted properly. For example <td
      colspan=2> instead of <td colspan="2">.

      *) some attributes are no longer supported in XHTML 1.0 Strict. For
      example the img tag no longer has a border attribute. This
      affects the border parameter of the Image component as well. People who
      create HTML 4.01 pages can easily remove the border using Cascading Style Sheets.

      *) some tag attributes have changed. For example <script language=...> has
      been replaced by <script type=...>

      1. xhtml-1.patch
        6 kB
        Markus Wiederkehr

        Activity

        Hide
        Markus Wiederkehr added a comment -

        This patch adds double quotes to tag attributes where necessary. It also replaces an occurance of valign=center by valign="middle".

        Show
        Markus Wiederkehr added a comment - This patch adds double quotes to tag attributes where necessary. It also replaces an occurance of valign=center by valign="middle".
        Hide
        Howard M. Lewis Ship added a comment -

        I'm trying to hit a few other likely problems.

        Changing Image and Rollover to not output a "border" attribute (except as an informal).

        Changeing Border to not output a "language" attribute for <script>.

        Fixing the bad templates that did not quote some attributes.

        Show
        Howard M. Lewis Ship added a comment - I'm trying to hit a few other likely problems. Changing Image and Rollover to not output a "border" attribute (except as an informal). Changeing Border to not output a "language" attribute for <script>. Fixing the bad templates that did not quote some attributes.
        Hide
        Howard M. Lewis Ship added a comment -

        Also, change FormSupportImpl to write out the hidden fields inside a <div>.

        Show
        Howard M. Lewis Ship added a comment - Also, change FormSupportImpl to write out the hidden fields inside a <div>.
        Hide
        Robert Hannebauer added a comment -

        The Form component generates an attribute "name" but the xhtml validator complains about an unknown attribute.

        Error Line 53 column 53: there is no attribute "name".
        <form method="post" action="/test/app" name="Form" id="Form">

        Show
        Robert Hannebauer added a comment - The Form component generates an attribute "name" but the xhtml validator complains about an unknown attribute. Error Line 53 column 53: there is no attribute "name". <form method="post" action="/test/app" name="Form" id="Form">
        Hide
        Jesse Kuhnert added a comment -

        Ensure and test that XHTML support really is present throughout the framework.

        Show
        Jesse Kuhnert added a comment - Ensure and test that XHTML support really is present throughout the framework.
        Hide
        Jesse Kuhnert added a comment -

        Removed form name attribute being written. Let us know if anything else comes up.

        Show
        Jesse Kuhnert added a comment - Removed form name attribute being written. Let us know if anything else comes up.

          People

          • Assignee:
            Jesse Kuhnert
            Reporter:
            Markus Wiederkehr
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development