Pluto
  1. Pluto
  2. PLUTO-475

Default Portlet Skin and WindowStateAnchorTag emit invalid markup

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Trivial Trivial
    • Resolution: Fixed
    • Affects Version/s: 1.1.5, 2.0.0
    • Fix Version/s: 1.1.5, 2.0.0
    • Component/s: portal driver
    • Labels:
      None

      Description

      The portlet skin and WindowStateAnchorTag both create invalid markup. The "id" attribute in the portlet skin contains the "/" character, and the WindowStateAnchorTag create an extra closing </span> tag.

      1. portal.patch
        1 kB
        Benjamin Gould
      2. portal-driver.patch
        0.8 kB
        Benjamin Gould

        Activity

        Hide
        Elliot Metsger added a comment -

        Done! Thanks Ben - applied to trunk and 1.1.x.

        Show
        Elliot Metsger added a comment - Done! Thanks Ben - applied to trunk and 1.1.x.
        Hide
        Benjamin Gould added a comment -

        This patch modifies the WindowStateAnchorTag so that it does not create the extra closing tag in its markup.

        Show
        Benjamin Gould added a comment - This patch modifies the WindowStateAnchorTag so that it does not create the extra closing tag in its markup.
        Hide
        Benjamin Gould added a comment -

        This patch modifies the WindowStateAnchorTag so that it does not create the extra closing tag in its markup.

        Show
        Benjamin Gould added a comment - This patch modifies the WindowStateAnchorTag so that it does not create the extra closing tag in its markup.
        Hide
        Benjamin Gould added a comment -

        This patch removes the "id" from the default portlet skin, and some extra window state links that do not seem to be displayed anyhow.

        Show
        Benjamin Gould added a comment - This patch removes the "id" from the default portlet skin, and some extra window state links that do not seem to be displayed anyhow.

          People

          • Assignee:
            Unassigned
            Reporter:
            Benjamin Gould
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development