[Jifty-commit] r4865 - in jifty/trunk: lib/Jifty/Web

[jifty-commit at lists.jifty.org]

Author: trs
Date: Wed Jan 16 10:35:50 2008
New Revision: 4865

Modified:
   jifty/trunk/   (props changed)
   jifty/trunk/lib/Jifty/Web/Menu.pm

Log:
 r31001 at zot:  tom | 2008-01-16 10:35:01 -0500
 Support menu grouping for YUI


Modified: jifty/trunk/lib/Jifty/Web/Menu.pm
==============================================================================
--- jifty/trunk/lib/Jifty/Web/Menu.pm	(original)
+++ jifty/trunk/lib/Jifty/Web/Menu.pm	Wed Jan 16 10:35:50 2008
@@ -4,7 +4,7 @@
 use URI;
 use Scalar::Util qw(weaken);
 
-__PACKAGE__->mk_accessors(qw(label _parent sort_order link target escape_label class));
+__PACKAGE__->mk_accessors(qw(label _parent sort_order link target escape_label class group));
 
 =head1 NAME
 
@@ -67,7 +67,6 @@
 Gets or sets the sort order of the item, as it will be displayed under
 the parent.  This defaults to adding onto the end.
 
-
 =head2 link
 
 Gets or set a Jifty::Web::Link object that represents this menu item. If
@@ -85,6 +84,11 @@
 Gets or sets the CSS class the link should have in addition to the default
 classes.  This is only used if C<link> isn't specified.
 
+=head2 group [STRING]
+
+Gets or sets the menu group this menu item is in.  Only used when rendering
+as a YUI menubar for now.  Groups are sorted by this value as well.
+
 =head2 url
 
 Gets or sets the URL that the menu's link goes to.  If the link
@@ -207,6 +211,27 @@
     return wantarray ? @kids : \@kids;
 }
 
+=head2 grouped_children
+
+Returns the children of this menu item in grouped sorted order; as an array
+of array refs in array context, or as an array ref of array refs in scalar.
+
+=cut
+
+sub grouped_children {
+    my $self = shift;
+    
+    my %group;
+    for my $kid ( $self->children ) {
+        push @{ $group{$kid->group} }, $kid;
+    }
+
+    my @grouped = map  { $group{$_} }
+                  sort { $a cmp $b }
+                  keys %group;
+
+    return wantarray ? @grouped : \@grouped;
+}
 
 =head2 render_as_menu
 
@@ -351,22 +376,27 @@
 
 sub _render_as_yui_menu_item {
     my ($self, $class, $id) = @_;
-    my @kids = $self->children 
-        or return;
+    my @grouped = $self->grouped_children or return;
     
     Jifty->web->out(
         qq{<div}
         . ($id ? qq{ id="$id"} : "")
-        . qq{ class="$class"><div class="bd"><ul>}
+        . qq{ class="$class"><div class="bd">}
     );
-    for (@kids) {
-        Jifty->web->out( qq{<li class="${class}item }
-        . ($_->active? 'active' : '') . qq{">});
-        Jifty->web->out( $_->as_link );
-        $_->_render_as_yui_menu_item("yuimenu");
-        Jifty->web->out( qq{</li>});
+    my $count = 1;
+    for my $group ( @grouped ) {
+        Jifty->web->out( $count == 1 ? '<ul class="first-of-type">' : '<ul>' );
+        for ( @$group ) {
+            Jifty->web->out( qq{<li class="${class}item }
+            . ($_->active? 'active' : '') . qq{">});
+            Jifty->web->out( $_->as_link );
+            $_->_render_as_yui_menu_item("yuimenu");
+            Jifty->web->out( qq{</li>});
+        }
+        Jifty->web->out('</ul>');
+        $count++;
     }
-    Jifty->web->out(qq{</ul></div></div>});
+    Jifty->web->out(qq{</div></div>});
 }
 
 =head2 as_link