CMS MADE SIMPLE FORGE

CMS Made Simple Core

 

[#4211] simple_navigation.tpl broken

avatar
Created By: Alex Dittrich (faglork)
Date Submitted: Mon Oct 19 15:35:42 -0400 2009

Assigned To:
Version: 1.6.5
CMSMS Version: None
Severity: Major
Resolution: Fixed
State: Closed
Summary:
simple_navigation.tpl broken
Detailed Description:
Hi!

The simple_navigation.tpl shipped with CMSMS 1.6.x has been changed without any
notice in the release notes.

The bug: an "inactive parent" should be coded h3, actually it gets an a href -
the same as an "active parent". That means that a page which is an inactive
parent links to itself. You even cannot "soften" the issue with CSS, since now
(with 1.6.x) both nodes get the same classes.

Since the standard templates can't be edited, the only solution is to copy the
old 1.5.x simple_navigation.tpl under a new name and change ALL page templates
accordingly.

a) this is clearly a bug and should be fixed
b) changes in the standard navigation templates should be announced in the
release notes
c) users should be warned NOT TO USE the standard navigation templates since
future changes will break their sites on upgrade

On a side note, it would be nice to know *why* this has changed, since so far
nobody could find an explanation: it just makes no sense.
If there is a logic behind this, it would have been better to introduce a new
additional standard template.

Cheers,
Alex


History

Comments
avatar
Date: 2010-05-30 13:59
Posted By: Rolf (rolf1)

Fixed in SVN
      
avatar
Date: 2010-07-06 14:28
Posted By: Rolf (rolf1)

1.8 is released
      
Updates

Updated: 2010-07-06 14:28
state: Open => Closed

Updated: 2010-05-30 13:59
resolution_id: => 7