The new menu builder WayFinder is ready for Testing

Quote from: rthrash at Jul 22, 2006, 12:49 PM

I’ve updated the hover zone drop down menu to work fine in IE now, too. By hover zone, I mean that you can errantly hover off the menus a bit without loosing your menu pop-up, and it’s all done with some clever CSS and innerTpls.

Looks great, makes the usability of the menu alot better when there is some space around the menu to hover on without it disappearing..

Don’t know if this would be commonly needed feature, but one thing I could of used on those cssplay menus is that the "parent" class would not be appended to the wf.classes on a folder item if the &level limit is reached. Not by default this way ofcourse, but an parameter to enable this would be great.

Also, would there be a need to have a selfClass parameter what would be added to the wf.classes on the current item? Don’t know if it really needed, because it can be done in hereTpl by setting it as a id to the list element or link tag or wherever.. but I guess someone might want/need it to be in the li element as a class with the rest of em, don’t know.

Quote from: doze at Jul 22, 2006, 02:01 PM

Don’t know if this would be commonly needed feature, but one thing I could of used on those cssplay menus is that the "parent" class would not be appended to the wf.classes on a folder item if the &level limit is reached. Not by default this way ofcourse, but an parameter to enable this would be great.

Actually, I like that as a default behavior. It makes sense to me anyway.

Also, would there be a need to have a selfClass parameter what would be added to the wf.classes on the current item? Don’t know if it really needed, because it can be done in hereTpl by setting it as a id to the list element or link tag or wherever.. but I guess someone might want/need it to be in the li element as a class with the rest of em, don’t know.

I see no reason not to include a here class either.

One more thing, I think it would be good to add this feature to Wayfinder by default. This can be also done with &startId=`[[UltimateParent]]` what I did in the mollio secondary menu example, but it’s so commonly used that might be good if it would be built-in. Also a possibility to define how many levels it should go up the tree (and maybe also reversed, to what level starting from root).

What do you think?

LOL... I sent Kyle an email about that earlier!

Naah, I dont believe you j/k. Great to know! BTW, looking forward to test the next version with these latests updates implemented..

Have now had chance to play with the 3rd release, and am glad to see my previous concerns have been addressed. This thing rocks.

Quote from: doze at Jul 22, 2006, 12:38 PM

EDIT: also, there is some minor flaws in the comment texts. In parentRowTpl there is a example to use [+wf.nestedmenu+] placeholder, but it should be [+wf.wrapper+] right?

This is also referenced in hereTpl. It is correct in rowTpl.

This confused me briefly too. I at first used parentRowTpl, mainly because the debug inferred I should set one, but I soon found that I was better off putting the wrapper in the rowTpl and hereTpl chunks, because there is no hereParentRowTpl.

I think there *should* be a hereParentRowTpl (or parentHereRowTpl), not least because the current parentRowTpl says it inherits from rowTpl when in fact it inherits from either rowTpl or hereTpl as appropriate, and it would just be consistent.

What about TV’s..? Should there be filtering options like Ditto has?

Hi everyone, thanks for giving this a good testing. I have been in las vegas over the last 3 days and have not had time to look at the updates. I will try to get through them asap, I need to get caught up at work first though.

I have updated the wayfinder snippet to include some of the previously mentioned additions. If I missed any let me know. Doze, if you want to update your css play stuff to work with these additions I will add it to the package as a good example on how to use it. I also am going to be working on removing the documentation into a seperate file. It is just getting way to long in the snippet header.

Here is a list of changes:

• Added ultimate parent option with new parameters (doze can you make sure the functionality is correct?)
• Added ability to load js/css from a chunk
• Added a few items to the debug output
• Set default class for first to ’’
• Set default class for here to ’active’
• I updated the molio example so it will work correctly

Anymore comments/suggestions are appreciated

The js/css loading requires this fix to the parser: http://modxcms.com/forums/index.php/topic,1942.msg18783.html#msg18783

I’m running into some issues where the outer row is not getting the right classes (e.g., none, and I need the last class to work).

Call:


Row Template:


Result:
http://stanginlizard.com/

Expected Result:
The top-level last LI (for Login) would have a class of .last as well; it doesn’t.