• Type: Improvement
• Resolution: Fixed
• Priority: Major
• Fix Version/s: 3.3
• Affects Version/s: 3.3
• Component/s: None
• Labels:

  None

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

Raphaël Franchet made changes - 24/Feb/12 08:53

Status

Original: Resolved [ 5 ]

New: Closed [ 6 ]

Laurence Aumeunier made changes - 17/Feb/12 19:13

Resolution		New: Fixed [ 1 ]
Status	Original: Reopened [ 4 ]	New: Resolved [ 5 ]

Collapse comment: Raphaël Franchet added a comment - 17/Feb/12 16:23

Raphaël Franchet added a comment - 17/Feb/12 16:23

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

Expand comment: Raphaël Franchet added a comment - 17/Feb/12 16:23

Raphaël Franchet added a comment - 17/Feb/12 16:23 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

Collapse comment: Laurence Aumeunier added a comment - 17/Feb/12 15:23

Laurence Aumeunier added a comment - 17/Feb/12 15:23

which classname are talking about ?

Expand comment: Laurence Aumeunier added a comment - 17/Feb/12 15:23

Laurence Aumeunier added a comment - 17/Feb/12 15:23 which classname are talking about ?

Laurence Aumeunier made changes - 13/Feb/12 12:09

Assignee

New: Raphaël Franchet [ raphael ]

Raphaël Franchet made changes - 10/Feb/12 10:46

Resolution	Original: Fixed [ 1 ]
Status	Original: Closed [ 6 ]	New: Reopened [ 4 ]

Collapse comment: Raphaël Franchet added a comment - 10/Feb/12 10:46

Raphaël Franchet added a comment - 10/Feb/12 10:46

The classname variable should also be moved

Expand comment: Raphaël Franchet added a comment - 10/Feb/12 10:46

Raphaël Franchet added a comment - 10/Feb/12 10:46 The classname variable should also be moved

Raphaël Franchet made changes - 10/Feb/12 10:45

Status

Original: Resolved [ 5 ]

New: Closed [ 6 ]

Raphaël Franchet made changes - 31/Jan/12 10:24

Resolution		New: Fixed [ 1 ]
Status	Original: Open [ 1 ]	New: Resolved [ 5 ]

Raphaël Franchet created issue - 31/Jan/12 10:11