Please don't shoot me for bringing this up again. I still don't think the current behavior of Menu.item makes sense. If the user doesn't want that link to appear on the page, then they shouldn't include <lift:Menu.item name="foo1" /> in its source. Most people put text around their link:
<lift:Menu.item name="foo1" /> | <lift:Menu.item name="foo2" /> | <lift:Menu.item name="foo3" /> which, on foo2, would render as _foo1_ | | _foo3_ or For a good time <lift:Menu.item name="foo1">click here</ llift:Menu.item>. For a not so good time <lift:Menu.item name="foo2">click here</llift:Menu.item>. which, on foo2, would render as For a good time _click here_. For a not so good time . Regards, Bradford On Mar 25, 7:53 am, bradford <fingerm...@gmail.com> wrote: > David, you're right that needing to surround the element text of a > with span is a unique case and should be a custom snippet. I've > removed the span now and think that Derek's addition of "always" would > be just what I need. Adding group="foo" to Menu.builder would suffice > as well. > > Thanks for the tips, Chas. > > Derek, if you do add "always" can you please let me know so that I can > update my code. > > Thanks, > Bradford > > On Mar 24, 10:37 pm, Derek Chen-Becker <dchenbec...@gmail.com> wrote: > > > The general case is that a page won't link to itself, I think, which is why > > the default isn't to show it when the page matches. Unless anyone has > > objections I can add an "always" attribute. As for #1, the Menu.item makes a > > link using whatever the contents of the Menu.item tag are for the link text: > > > <lift:Menu.item name="foo"><span>Go here</span></lift:Menu.item> > > > should become > > > <a href={foo location}><span>Go here</span></a> > > > Am I misunderstanding what you're looking for there? As for #2, you should > > be able to add a class using the prefixed attribute: > > > <lift:Menu.builder li_item:class="bar" /> > > > In this context, li_item is the menu item that matches the current page. > > With Menu.group, you can specify the binding template: > > > <ul> > > <lift:Menu.group group="help"> > > <li class="bar"><menu:bind /></li> > > </lift:Menu.group> > > </ul> > > > But there's no provision to do anything special for the current page. > > > Let me know if that's not sufficient or if I'm misunderstanding your > > requirement. > > > Derek > > > On Tue, Mar 24, 2009 at 3:50 PM, bradford <fingerm...@gmail.com> wrote: > > > > Thanks for the clarification, David, and for your snippet, Derek. > > > > I think adding an "always" attribute to Menu.item would be very > > > beneficial. I still don't understand why that's not its default > > > behavior. > > > > It looks like I will not be able to use any of lift's Menu tags at > > > this time, because 1) I need to surround the item text with span and > > > 2) I need a way to add class="active" to the li_item. Both are not > > > possible with Menu.item, Menu.group, or Menu.builder. Let me know if > > > I am mistaken. If I am not not, may I put in a feature request for > > > these items. For the time being I will just hard code it as follows > > > (which is not a big deal to me at this time): > > > > <ul class="menu"> > > > <li><a href="/foo1" class="active"><span>Foo1</span></foo> > > > <li><a href="/foo2"><span>Foo2</span></foo> > > > <li><a href="/foo3"><span>Foo3</span></foo> > > > </ul> > > > > Thanks again for the great support :) > > > > Bradford > > > > On Mar 24, 12:08 pm, David Pollak <feeder.of.the.be...@gmail.com> > > > wrote: > > > > On Tue, Mar 24, 2009 at 9:02 AM, Charles F. Munat <c...@munat.com> > > > wrote: > > > > > > David Pollak wrote: > > > > > > What's the best practice: > > > > > > > For more information about <lift:Menu.item name="foo1" />. For > > > more > > > > > > information about <lift:Menu.item name="foo2" />. > > > > > > > Or > > > > > > > For more information about <a href="/foo1">foo1</a>. For more > > > > > > information about <a href="/foo2">foo2</a>. > > > > > > > The latter. This allows you to move the pages around on the > > > filesystem > > > > > > without having to grep through all you source files looking for what > > > > > > needs to be changed. > > > > > > Am I missing something, or did you mean the former? > > > > > D'oh! That brain-finger connection is always getting messed up. I > > > > meant > > > > the former. Thanks for correcting me! > > > > > > Chas. > > > > > -- > > > > Lift, the simply functional web frameworkhttp://liftweb.net > > > > Beginning Scalahttp://www.apress.com/book/view/1430219890 > > > > Follow me:http://twitter.com/dpp > > > > Git some:http://github.com/dpp --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Lift" group. To post to this group, send email to liftweb@googlegroups.com To unsubscribe from this group, send email to liftweb+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/liftweb?hl=en -~----------~----~----~----~------~----~------~--~---