The new menu builder WayFinder is ready for Testing

Hi

I’m sorry.

I have deleted the first row with the php tag but I do not add à comment tag /* at the beginning of the snippet.

Sorry.

Just one thing I’ve noticed is that the ’id ’attribute is being repeated for all top level <ul> tags which is not good for xhtml compliance.

garry, the reason the id is being duplicated is this: if the there is no template set for innerTpl it uses the outerTpl template to process the nesting. So if you don’t want the id to repeat, just remove it from the chunk, or create a new template for innerTpl and it should take care of it. Let me know if that helps.

Okay, thanks for that. That helps

My, my, this is just awesome !

I am off to testing this new baby

Hmm.. a problem. I have made up a tree structure like this:



When I call Wayfinder using:


It outputs:

* Page 1
o Page 1.1
o Page 1.2
+ Page 1.2.1
+ Page 1.2.2
+ Page 1.2.3
o Page 1.3
* Page 2

So the content’s of Page 2 folder is totally missing. I copy pasted the wayfinder templates from the snippet comments. The debug when I click onto Page 2 says:

WayFinder Debug Output:
Docids for ’Here’ class checking
44

Chunk Checks
No chunk specified for innerTpl

No chunk specified for innerRowTpl

No chunk specified for innerHereTpl

Output Info
Item Processed: (1) Page 1 inserted into chunk: wfRow | class applied:
Item Processed: (55) Page 1.1 inserted into chunk: wfRow | class applied:
Item Processed: (56) Page 1.2 inserted into chunk: wfRow | class applied:
Item Processed: (58) Page 1.2.1 inserted into chunk: wfRow | class applied:
Item Processed: (60) Page 1.2.2 inserted into chunk: wfRow | class applied:
Item Processed: (61) Page 1.2.3 inserted into chunk: wfRow | class applied:
Level decreased previous items inserted into chunk: wfOuter | class applied:
Item Processed: (57) Page 1.3 inserted into chunk: wfRow | class applied:
Level decreased previous items inserted into chunk: wfOuter | class applied:
Item Processed: (44) Page 2 inserted into chunk: wfHere | class applied:
Item Processed: (49) Page 2.1 inserted into chunk: wfRow | class applied:
Item Processed: (50) Page 2.1.1 inserted into chunk: wfRow | class applied:
Item Processed: (52) Page 2.1.1.1 inserted into chunk: wfRow | class applied:
Item Processed: (53) Page 2.1.1.2 inserted into chunk: wfRow | class applied:
Level decreased previous items inserted into chunk: wfOuter | class applied:
Item Processed: (51) Page 2.1.2 inserted into chunk: wfRow | class applied:
Level decreased previous items inserted into chunk: wfOuter | class applied:
Item Processed: (38) Page 2.2 inserted into chunk: wfRow | class applied:
Item Processed: (54) Page 2.2.1 inserted into chunk: wfRow | class applied:
End of list reached items inserted into chunk: wfOuter | class applied:

So the debug says that they are included into the chunk, but trust me, it’s not coming to the HTML source.

EDIT:

Quote from: rthrash at Jul 14, 2006, 06:43 PM

Lady and Gentlemen: (sottwell is the only female with access here currently...)

Wendy doesn’t have access here?

EDIT 2:

Also when I click to Page 1.1 OR Page 1.3, the output changes to:

* Page 1
o Page 1.1
o Page 1.2
o Page 1.3
* Page 2

So the Page 1.2 childs are missing..

Wendy doesn’t have access here?

Hehe, Wendy is, err, as a much a gentleman as you and me.

Doze,
I found the error, it was not adding the submenus on if it hit the end of the menu and was not at level 1. I have fixed this and have added a new zip to my first post.

Thanks,
Kyle

Kyle,

Ok, it fixed that, then to the next error

If I click the root level links "Page 1" or "Page 2", the link changes to normal text, but if I click a link in somewhere in the tree, they doesn’t change.

Doze,
I have updated the first post with the fix. Thanks for finding these and let me know if you find any more

It will now use the hereTpl chunk to format all here items unless the innerHereTpl chunk is specified.