[MGNLSTK-618] correct navigation config detail Created: 10/May/10 Updated: 30/Sep/10 Resolved: 30/Sep/10 |
|
| Status: | Closed |
| Project: | Magnolia Standard Templating Kit (closed) |
| Component/s: | templates |
| Affects Version/s: | 1.3.1 |
| Fix Version/s: | 1.3.5 |
| Type: | Bug | Priority: | Minor |
| Reporter: | Bert Leunis | Assignee: | Ondrej Chytil |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Attachments: |
|
||||||||
| Issue Links: |
|
||||||||
| Template: |
|
||||||||
| Patch included: |
Yes
|
||||||||
| Acceptance criteria: |
Empty
|
||||||||
| Date of First Response: | |||||||||
| Description |
|
When working with the navigation in the STK I found that one property for the navigation has the wrong name. At config/modules/standard-templating-kit/config/site/templates/prototype/navigation/horizontal (and also ../vertical) the property "openAll" is defined. In the corresponding class (info.magnolia.module.templatingkit.navigation.SubNavigation) this property is named "allOpen". So setting the property has no effect. It should be renamed in one of these places. I suggest to change the config instead of the class. |
| Comments |
| Comment by Bert Leunis [ 23/Jul/10 ] |
|
|
| Comment by Magnolia International [ 23/Jul/10 ] |
|
bootstrap file fixed with r35931 ( |
| Comment by Jan Haderka [ 30/Sep/10 ] |
|
The update task needs to update all site definitions. |