[TYPO3-dev] Navi HMENU dropdown issue

[Thomas Petersen]

Hi Stefan
Im new to TS...But thx..

U have fix the 1. issue, and its also fixed with colors THX :-)

Now i only gat the 2. issue, if i select a link in the dropdown, the top link is getting black, and not yellow, if we select the toplink.
How much can i clean it up, if i # something out, them im getting some errors.

Hope U or another can help with the 2. issue.

Code right now at www.codem.dk

lib.mainNavi = HMENU
lib.mainNavi.entryLevel=0
lib.mainNavi {
    1 = TMENU
    1 {
    wrap = <ul id="navigation"> | </ul>
    #Expane All - 0=NO, 1=YES
    expAll = 1
    NO = 1
    NO.wrapItemAndSub = <li class="drop"> | </li>
    NO.stdWrap.wrap = <span>|</span>
    RO < .NO 
    CUR < .NO 
    CUR.wrapItemAndSub = <li class="current_page_item">|</li>
    CUR.stdWrap.wrap = <span>|</span>
    IFSUB = 1
    IFSUB {
    wrapItemAndSub = <li class="drop"> | </li>
    stdWrap.wrap = <span>|</span>
    }
    ACT < .NO
    ACTIFSUB < .IFSUB
    }
    2 = TMENU
    2 {
    wrap = <ul> | </ul>
    expAll = 1
    NO.wrapItemAndSub = <li class="drop"> | </li>
    IFSUB = 1
    IFSUB {
    wrapItemAndSub = <li class="drop"> | </li>
    }
}



Quote: Stefan Reichelt (Songyu) wrote on Wed, 18 September 2013 15:01
----------------------------------------------------
> Hello Thomas,
> 
> On 18/09/2013 14:12, Thomas Petersen wrote:
> > Can someone guide me here ?
> 
> I think we'd need to see the output in the frontend to get a better
> understanding of what goes wrong.
> 
> 
> Anyway, this might be one (second one probably) of the problems, not
> sure. You've written:
> CUR.allWrap = <li class="current_page_item">|</li>
> 
> Since you copied .NO into .CUR then .CUR already has wrapItemAndSub set
> with the <li>-tag in it. So with the allWrap you add it a second time.
> You could change it to:
> CUR.wrapItemAndSub= <li class="drop current_page_item">|</li>
> 
> To have both classes in the LI-Tag.
> 
> And little bit of cleaning up:
> 1. You can remove all the items which have no difference to .NO. This
> includes .ACT .RO and all .IFSUB types. Only add them if you truly need
> them to do something different than .NO. If they are not defined, .NO
> will be used anyway.
> 
> 2. You can avoid all the '.ACT = 1' & '.CUR = 1' and so forth by setting
> '.NO = 1' and when you copy .NO, it copies the activation too.
> 
> 
> Kind regards
> Stefan
----------------------------------------------------