[Lift] Menu builder group and li_item

12 views
Skip to first unread message

george

unread,
Aug 22, 2009, 2:47:43 PM8/22/09
to Lift
I'm trying to work with a SiteMap which has some different LocGroups.

When I do:

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

I can render a menu for the whole sitemap and the selected item has
the class attribute applied.

But, when I do:

<lift:Menu.builder group="site" li_item:class="selected" />

I would expect the same behavior with a subset of items, but it just
doesn't work. The selected item doesn't get the class attribute.

However, if I do:

<lift:Menu.builder group="site" li:class="hello" />

Every item gets the class attribute.

To me this seems like a bug, or am I missing something?

Derek Chen-Becker

unread,
Aug 24, 2009, 3:56:29 PM8/24/09
to lif...@googlegroups.com
I think that that's a bug. I don't have time to look at it today, so please file an issue.

Derek

george

unread,
Aug 24, 2009, 4:41:02 PM8/24/09
to Lift

David Pollak

unread,
Aug 24, 2009, 6:08:23 PM8/24/09
to lif...@googlegroups.com
A fix is pushed to the GitHub repo.  Hudson is chewing through a new build.


On Mon, Aug 24, 2009 at 1:41 PM, george <geo...@mattandgeorge.com> wrote:

Issue is filed here:

http://github.com/dpp/liftweb/issues#issue/53




--
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

george

unread,
Aug 25, 2009, 12:36:42 PM8/25/09
to Lift
Thanks for looking into this so quickly. I haven't had a chance to
test it yet, but I notice another user is now having a slightly
different problem. Not sure if it's related.

http://groups.google.com/group/liftweb/browse_thread/thread/12415c178ee7b1ee/75d38df3617f77b6#75d38df3617f77b6

David Pollak

unread,
Aug 25, 2009, 4:54:36 PM8/25/09
to lif...@googlegroups.com
Yeah... I broke things rather than fixing them... I've checked in something that I believe addresses your issue and fixes the other user's issue.

george

unread,
Aug 25, 2009, 5:55:24 PM8/25/09
to Lift
awesome thanks!
Reply all
Reply to author
Forward
0 new messages