big change: helper 'tabs_for' now requires a block

Author: Russell Jones

README

@@ -13,21 +13,21 @@ Maybe you will find them useful as well.
 These include ... (just one helper for now)
 
 
-TabsRenderer
+TabsHelper
 ----------------------------------------------------------------	
 
 This helper simplifies the code required to use JQuery UIs Tabs plugin.
 
-There are 3 steps to rendering a tab ... initialize, create and render.
+Example usage ...
 
-  <% tabs = TabsRenderer.new(self) %>
-  	<% tabs.create('tab_one', 'Tab 1') do %>
+  <% tabs_for do |tab| %>
+  	<% tab.create('tab_one', 'Tab 1') do %>
       # ... insert tab contents
   	<% end %>
-  	<% tabs.create('tab_two', 'Tab 2') do %>
+  	<% tab.create('tab_two', 'Tab 2') do %>
       # ... insert tab contents
   	<% end %>
-  <%= tabs.render %>
+	<% end %>
 
 The above will generate this HTML in your view:
 
@@ -49,43 +49,21 @@ Tabs will be rendered in the order you create them.
 You can easily render a tab conditionally by appending your condition to the end of 
 the 'create' block as such ...
 
-  <% tabs.create('profile_tab', 'Your Profile') do %>
+  <% tab.create('profile_tab', 'Your Profile') do %>
   	# ... insert tab contents
   <% end unless @current_user.nil? %>
 
 You can pass HTML options to either the parent DIV or any individual tab's
 DIV as you like ...
 
-  <% tabs = TabsRenderer.new(self, :class = 'flora') %>
-  	<% tabs.create('tab_one', 'Tab 1', :style => 'background: #FFF') do %>
+  <% tabs_for(:class => 'zippy') do |tab| %>
+  	<% tab.create('tab_one', 'Tab 1', :style => 'background: #FFF') do %>
       # ... insert tab contents
   	<% end %>
-  <%= tabs.render %>
+  <% end %>
 
 The default DOM ID for the parent div is ... id="tabs" ... unless you pass in an HTML
 option with a different value.
 
-Also optional, you can now call TabsRenderer.new with a block parameter, and create
-the tabs that way.
 
-  <% tabs = TabsRenderer.new(self) do |tab| %>
-  	<% tab.create('tab_one', 'Tab 1') do %>
-      # ... insert tab contents
-  	<% end %>
-  	<% tab.create('tab_two', 'Tab 2') do %>
-      # ... insert tab contents
-  	<% end %>
-	<% end %>
-  <%= tabs.render %>
-  
-##########################################################################
-Now works as:
-  <% tabs_for do |tab| %>
-  	<% tab.create('tab_one', 'Tab 1') do %>
-      # ... insert tab contents
-  	<% end %>
-  	<% tab.create('tab_two', 'Tab 2') do %>
-      # ... insert tab contents
-  	<% end %>
-	<% end %>
 

helpers/tabs_helper.rb

@@ -1,51 +1,51 @@
 module TabsHelper
 
-  def tabs_for( *args, &proc )
+  def tabs_for( *options, &block )
     raise ArgumentError, "Missing block" unless block_given?
 
-    tabs = TabsHelper::TabsRenderer.new( &proc )
+    tabs = TabsHelper::TabsRenderer.new( *options, &block )
     tabs_html = tabs.render
-    concat tabs_html, proc.binding
+    concat tabs_html, block.binding
   end
 
   class TabsRenderer
 
-    def initialize( options={}, &proc )
+    def initialize( options={}, &block )
       raise ArgumentError, "Missing block" unless block_given?
 
-      @template = eval( 'self', proc.binding )
+      @template = eval( 'self', block.binding )
       @options = options
       @tabs = []
 
       yield self
     end
 
-    def create(tab_id, tab_text, options={}, &block)
+    def create( tab_id, tab_text, options={}, &block )
       raise "Block needed for TabsRenderer#CREATE" unless block_given?
-      @tabs << [tab_id, tab_text, options, block]
+      @tabs << [ tab_id, tab_text, options, block ]
     end
 
     def render
-      content_tag(:div, (render_tabs + render_bodies), {:id => :tabs}.merge(@options))
+      content_tag( :div, ( render_tabs + render_bodies ), { :id => :tabs }.merge( @options ) )
     end
 
     private #  ---------------------------------------------------------------------------
 
     def render_tabs
       content_tag :ul do
         @tabs.collect do |tab|
-          content_tag(:li, link_to(content_tag(:span, tab[1]), "##{tab[0]}") )
+          content_tag( :li, link_to( content_tag( :span, tab[1] ), "##{tab[0]}" ) )
         end
       end
     end
 
     def  render_bodies
       @tabs.collect do |tab| 
-        content_tag(:div, capture(&tab[3]), tab[2].merge(:id => tab[0])) 
+        content_tag( :div, capture( &tab[3] ), tab[2].merge( :id => tab[0] ) ) 
       end.to_s
     end
 
-    def method_missing(*args, &block)
+    def method_missing( *args, &block )
       @template.send( *args, &block )
     end