Menu.builder li_item being applied to all?

[Richard Dallaway]

I've been running "mvn -o" for a while, but didn't today and noticed a
change in behaviour for the menu builder (1.1 SNAPSHOT). I'm seeing
all my menus being given the selected item class, not just the one
that's selected for the page you're on. I'm using...

<lift:Menu.builder li_item:class="current_tab" />

I've cut my site map down to this...

val menus = Menu(Loc("one", new Link("index" :: Nil, false), "Page 1")) ::
Menu(Loc("two", new Link("index2" :: Nil, false), "Page 2")) :: Nil

LiftRules.setSiteMap(SiteMap(menus:_*))

...and I see the following output:

<ul><li class="current_tab"><a href="/index">Page 1</a></li><li
class="current_tab"><span>Page 2</span></li></ul>

I'd expect only one of the li elements to have the class attribute.

Am I doing something wrong here?

Thank you
Richard

[george]

Maybe the fix for this issue has caused the problem..?

http://groups.google.com/group/liftweb/browse_thread/thread/aca3b12c397e3b87
http://github.com/dpp/liftweb/issues/closed/#issue/53

[David Pollak]

I think I've fixed my bone-headed attempt to fix another issue.

--
Lift, the simply functional web framework http://liftweb.net
Beginning Scala http://www.apress.com/book/view/1430219890
Follow me: http://twitter.com/dpp
Git some: http://github.com/dpp

[Timothy Perrett]

Whenever I find myself having thoughts such as these its categorically
time to start writing better tests ;-)

Cheers, Tim

On Aug 25, 9:55 pm, David Pollak <feeder.of.the.be...@gmail.com>
wrote:

[Richard Dallaway]

I can confirm that your fix of the fix has fixed my issue (which means
I've grabbed the latest 1.1 snapshot and it's all good again)

Thank you
Richard