Added support for blocks in menu_item, to be more inline with link_to's capabilities.

Author: jakew

app/helpers/navbar_helper.rb

@@ -15,10 +15,12 @@ def menu_group(options={}, &block)
     content_tag(:ul, :class => "nav #{pull_class}", &block)
   end
 
-  def menu_item(name, path="#", *args)
+  def menu_item(name=nil, path="#", *args, &block)
+    path = name || path if block_given?
     options = args.extract_options!
     content_tag :li, :class => is_active?(path) do
-      link_to name, path, options
+      name, path = path, options if block_given?
+      link_to name, path, options, &block
     end
   end
 

spec/lib/twitter_bootstrap_rails/navbar_helper_spec.rb

@@ -101,6 +101,18 @@
       self.stub!("uri_state").and_return(:active)
       menu_item("Log out", "/users/sign_out", :class => "home_link", :method => :delete).should eql('<li class="active"><a href="/users/sign_out" class="home_link" data-method="delete" rel="nofollow">Log out</a></li>')
     end
+    it "should pass a block but no name if a block is present" do
+      self.stub!("current_page?").and_return(false)
+      menu_item("/"){content_tag("i", "", :class => "icon-home") + " Home"}.should eql('<li><a href="/"><i class="icon-home"></i> Home</a></li>')
+    end
+    it "should work with just a block" do
+      self.stub!("current_page?").and_return(false)
+      menu_item{ content_tag("i", "", :class => "icon-home") + " Home" }.should eql('<li><a href="#"><i class="icon-home"></i> Home</a></li>')
+    end
+    it "should return the link with class 'active' if on current page with a block" do
+      self.stub!("uri_state").and_return(:active)
+      menu_item("/"){ content_tag("i", "", :class => "icon-home") + " Home" }.should eql('<li class="active"><a href="/"><i class="icon-home"></i> Home</a></li>')
+    end
   end
 
   describe "drop_down" do