Uploaded image for project: 'CMS'
  1. CMS
  2. CMS-3360

Update XSL : move code for main view from generic to -main xsl

XMLWordPrintable

    • Icon: Improvement Improvement
    • Resolution: Fixed
    • Icon: Major Major
    • 3.3
    • 3.3
    • None
    • None

      currently, article.xsl have the code for main view.
      This is disturbing and lead to strange behavior when overriding all this

          [CMS-3360] Update XSL : move code for main view from generic to -main xsl

          Raphaël Franchet added a comment -

          for every content type we are using the common.xsl
          this xsl suppose to have a xsl variable named common-content-css-class-name

          e.g. on article, currently we have :

          • article.xsl defining <xsl:variable name="common-content-css-class-name">article view-main</xsl:variable>
          • article-main.xsl defining nothing
          • article-abstract.xsl defining <xsl:variable name="common-content-css-class-name">article view-abstract</xsl:variable>
          • article-link.xsl defining <xsl:variable name="common-content-css-class-name">article view-link</xsl:variable>

          To be clean, the definition in article.xsl should be moved to article-main.xsl

          Raphaël Franchet added a comment - for every content type we are using the common.xsl this xsl suppose to have a xsl variable named common-content-css-class-name e.g. on article, currently we have : article.xsl defining <xsl:variable name="common-content-css-class-name">article view-main</xsl:variable> article-main.xsl defining nothing article-abstract.xsl defining <xsl:variable name="common-content-css-class-name">article view-abstract</xsl:variable> article-link.xsl defining <xsl:variable name="common-content-css-class-name">article view-link</xsl:variable> To be clean, the definition in article.xsl should be moved to article-main.xsl

          Laurence Aumeunier added a comment -

          which classname are talking about ?

          Laurence Aumeunier added a comment - which classname are talking about ?

          Raphaël Franchet added a comment -

          The classname variable should also be moved

          Raphaël Franchet added a comment - The classname variable should also be moved

            raphael Raphaël Franchet
            raphael Raphaël Franchet
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

              Created:
              Updated:
              Resolved: