[ 
http://jira.magnolia.info/browse/MAGNOLIA-523?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Gregory Joseph reopened MAGNOLIA-523:
-------------------------------------


reopening to unset fix version...

> SimpleNavigationTag showing empty <ul> when no visible children
> ---------------------------------------------------------------
>
>                 Key: MAGNOLIA-523
>                 URL: http://jira.magnolia.info/browse/MAGNOLIA-523
>             Project: Magnolia
>          Issue Type: Bug
>          Components: taglibs
>    Affects Versions: 2.1 Final
>            Reporter: John Olival
>            Assignee: Philipp Bracher
>            Priority: Minor
>
> When displaying the navigation for elements that have children and if all the 
> elements' children are set to hideInNav, an empty ul: <ul 
> class="levelx"></ul> is shown. In CSS drop-down navigation menus, this 
> displays as an empty block, but also it is syntactically incorrect HTML. This 
> is only a problem for elements which are located anywhere in the branch of a 
> selected element. E.g. In the tree
> Home
>   + Section 1
> ...
>   + Section 2
>     + Subsection 2.1
>     + Subsection 2.2
>       + Page 2.2.1
>       + Page 2.2.2
>       + Page 2.2.3
>     + Subsection 2.3
>       + Page 2.3.1
>       + Page 2.3.2
>       + Page 2.3.3
> ...
> Pages under 2.2 are marked as hideInNav; if Page 2.2.1 is selected so is the 
> currently active page, then an empty ul element is created.
> Having looked at the code, I reckon the fix is here (line 249):
>             if (activePage.getHandle().equals(child.getHandle())) {
>                 // self
>                 // check if child has visible children
>                 if (hasVisibleChildren(child)) {
>                     showChildren = true;
>                 }
>                 self = true;
>                 cssClasses.add("active"); //$NON-NLS-1$
>             }
> It hasn't been tested.
> John

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://jira.magnolia.info/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

----------------------------------------------------------------
for list details see
http://documentation.magnolia.info/
----------------------------------------------------------------

Reply via email to