Tapestry 5
  1. Tapestry 5
  2. TAP5-1128

A space is added to a HTML start comment sequence making it impossible to add IE conditional comments

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 5.2.0
    • Component/s: tapestry-core
    • Labels:
      None

      Description

      I tried to add an Internet Explorer conditional style to solve some rendering differences between IE and FF. To way to add conditional comments is by using the following syntax

      <!--[if IE]>
      Special instructions for IE here
      <![endif]-->

      The problem is that Tapestry adds a space between <!-- and [if IE]

      example: <Unable to render embedded object: File (--[if IE]> becomes <) not found.-- [if IE]> (note the added space)

      IE only accepts the conditional comment if there is no space between <!-- and [if IE]

        Activity

        Howard M. Lewis Ship made changes -
        Status In Progress [ 3 ] Closed [ 6 ]
        Fix Version/s 5.2.0 [ 12314122 ]
        Resolution Fixed [ 1 ]
        Howard M. Lewis Ship made changes -
        Status Reopened [ 4 ] In Progress [ 3 ]
        Howard M. Lewis Ship made changes -
        Project Tapestry [ 10573 ] Tapestry 5 [ 12310833 ]
        Key TAPESTRY-2503 TAP5-1128
        Affects Version/s 5.0.13 [ 12313205 ]
        Component/s tapestry-core [ 12312470 ]
        Component/s Framework [ 11280 ]
        Howard M. Lewis Ship made changes -
        Resolution Won't Fix [ 2 ]
        Status Closed [ 6 ] Reopened [ 4 ]
        Howard M. Lewis Ship made changes -
        Field Original Value New Value
        Status Open [ 1 ] Closed [ 6 ]
        Assignee Howard M. Lewis Ship [ hlship ]
        Resolution Won't Fix [ 2 ]
        Martijn Brinkers created issue -

          People

          • Assignee:
            Howard M. Lewis Ship
            Reporter:
            Martijn Brinkers
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development