tabula_rasa 0.0.2 → 0.0.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 77d9e0c24755a3555875efcb476b45574faff791
-  data.tar.gz: ab243bfe91d4bd59df5fc0e9ab9245b59f3d6b93
+  metadata.gz: 85b6138d31f80d0ef643af81a02b77bcea0fc4b2
+  data.tar.gz: 787437faf4bd0631fd23cc2ec3e64e8e48c2079a
 SHA512:
-  metadata.gz: e27ca7c5d3570874a0c6ccf8262941d43814f3bca5e7bba1823f0551d8e129b9d424af58c58fcaa8df71ec5e45a48198ff76bf8099a315d7314c2bbbf473cee7
-  data.tar.gz: 68a9fa9df16526c3b8d3440721815163c2a6ac95acbbb6421b9ed848396ccd0f9690ffd67670011957d3853212bacff644cf380d36c4d5dc2fca93a163e71dd5
+  metadata.gz: 0647a914557c4b474c43013f1693e3454826feed995410445d39165f3dc270f07aba2805b9addf321aacfe634f57bc4a81dce0ac421d38348b6352ed12940c20
+  data.tar.gz: 86b608afad417372b5eb3610823e6625e7cc8c6c42ae2a087e02ce6caa1ac72df3c4bd88397f4baf3b56c7130cc2251390337bf26f0461e2b8815aac9bc84bc3

data/lib/tabula_rasa/base.rb

@@ -1,10 +1,11 @@
+require 'tabula_rasa/row'
 require 'tabula_rasa/column'
 
 module TabulaRasa
   class Base
-    attr_reader :collection, :view, :klass, :columns, :options
+    attr_reader :collection, :options
 
-    delegate :capture, :content_tag, :safe_join, :truncate, to: :view
+    delegate :content_tag, :cycle, :dom_id, :dom_class, :safe_join, to: :view
 
     def initialize(collection, view, options = {}, &block)
       raise ArgumentError, 'TabulaRasa only works on ActiveRecord Relation instances' unless collection.is_a?(ActiveRecord::Relation)
@@ -14,6 +15,7 @@ module TabulaRasa
       @klass = collection.klass
       @columns = []
       yield self if block_given?
+      ensure_row
     end
 
     def render
@@ -25,8 +27,15 @@ module TabulaRasa
       @columns << Column.new(self, *args, &block)
     end
 
+    def row(options = {}, &block)
+      raise ArgumentError, 'Row definition cannot be called more than once' if @row.present?
+      @row = Row.new(self, options, &block)
+    end
+
   private
 
+    attr_reader :view, :klass, :columns
+
     def content
       safe_join [thead, tbody]
     end
@@ -47,7 +56,7 @@ module TabulaRasa
 
     def collection_body
       rows = collection.map do |member|
-        content_tag :tr do
+        content_tag :tr, @row.options_for(member) do
           cells = columns.map do |column|
             column.body_content_for member
           end
@@ -57,8 +66,20 @@ module TabulaRasa
       safe_join rows
     end
 
+    def empty_body
+      content_tag :tr, class: 'empty' do
+        content_tag :td, colspan: columns.size do
+          "No #{klass.table_name.downcase} present"
+        end
+      end
+    end
+
     def table_options
       options.except :head, :body
     end
+
+    def ensure_row
+      @row ||= Row.new(self)
+    end
   end
 end

data/lib/tabula_rasa/column.rb

@@ -1,6 +1,6 @@
 module TabulaRasa
   class Column
-    delegate :capture, :content_tag, :truncate, to: :base
+    delegate :content_tag, to: :base
 
     def initialize(base, *args, &block)
       @base = base
@@ -23,7 +23,7 @@ module TabulaRasa
       content_tag :td, body_value_for(instance), options[:body]
     end
 
-    def body(&block)
+    def value(&block)
       @body_value_option = block if block_given?
     end
 
@@ -61,9 +61,9 @@ module TabulaRasa
     def body_value_for(instance)
       value = body_value_option || instance.send(attribute)
       if value.respond_to?(:call)
-        value.call(instance)
+        value.call instance
       else
-        truncate value.to_s
+        value.to_s
       end
     end
   end

data/lib/tabula_rasa/row.rb

@@ -0,0 +1,78 @@
+module TabulaRasa
+  class Row
+    # NOTE: Need to handle ALL valid HTML5 attributes
+    ATTRIBUTES = [:id, :class]
+
+    delegate :cycle, :dom_id, :dom_class, to: :base
+
+    def initialize(base, options = {}, &block)
+      @base = base
+      @options = options
+      yield self if block_given?
+    end
+
+    def options_for(instance)
+      {
+        id: [id_value_for(instance), dom_id_for(instance)].compact.join(' ').squish,
+        class: [class_value_for(instance), zebrastripe, dom_class_for(instance)].compact.join(' ').squish
+      }.merge data_values_for(instance)
+    end
+
+    def data(key, &block)
+      options[:data] ||= {}
+      options[:data][key] = block
+    end
+
+    ATTRIBUTES.each do |attribute|
+      define_method attribute do |&block|
+        options[attribute] = block
+      end
+    end
+
+  private
+
+    attr_reader :base, :options
+
+    def zebrastripe
+      case base.options[:zebra]
+      when false
+        # Nothing
+      when nil
+        cycle *%w{even odd}
+      when Array
+        cycle *base.options[:zebra]
+      else
+        raise ArgumentError, "Invalid value for options[:zebra]: #{base.options[:zebra]}. Should be array."
+      end
+    end
+
+    def dom_id_for(instance)
+      return unless base.options[:dom_id] == true
+      dom_id(instance) unless id_value_for(instance).present?
+    end
+
+    def dom_class_for(instance)
+      dom_class(instance) if base.options[:dom_class] == true
+    end
+
+    def data_values_for(instance)
+      return {} if options[:data].blank?
+      options[:data].inject({}) do |m, x|
+        key, value = x
+        m["data-#{key}"] = value.call(instance)
+        m
+      end
+    end
+
+    ATTRIBUTES.each do |attribute|
+      define_method "#{attribute}_value_for" do |instance|
+        value = options[attribute]
+        if value.respond_to?(:call)
+          value.call instance
+        else
+          value.to_s
+        end
+      end
+    end
+  end
+end

data/lib/tabula_rasa/version.rb

@@ -1,3 +1,3 @@
 module TabulaRasa
-  VERSION = "0.0.2"
+  VERSION = "0.0.3"
 end

data/spec/config/spec.rb

@@ -3,6 +3,7 @@ class TabulaRasa::Spec < MiniTest::Spec
   include ActionView::Helpers::TagHelper
   include ActionView::Helpers::TextHelper
   include ActionView::Helpers::OutputSafetyHelper
+  include ActionView::RecordIdentifier
   include TabulaRasa::Helpers
 
   attr_accessor :output_buffer

data/spec/tabula_rasa/helper_body_spec.rb

@@ -35,8 +35,156 @@ describe TabulaRasa::Helpers, 'Head Specs' do
     end
   end
 
-  # NOTE: For now, I don't see the point of adding attributes on TR element.
-  # I've had varying success styling on that and tend to style on the TH elements instead.
+  describe 'TR attributes' do
+    it 'zebrastripes by default' do
+      captured = capture do
+        tabula_rasa @survivors do |t|
+          t.column :first_name
+        end
+      end
+      rows = extract_all('table tbody tr', captured)
+
+      rows[0].attribute('class').value.must_equal 'even'
+      rows[1].attribute('class').value.must_equal 'odd'
+      rows[2].attribute('class').value.must_equal 'even'
+    end
+
+    it 'can disable zebrastripes' do
+      captured = capture do
+        tabula_rasa @survivors, zebra: false do |t|
+          t.column :first_name
+        end
+      end
+      rows = extract_all('table tbody tr', captured)
+
+      rows.each do |row|
+        row.attribute('class').value.must_be_empty
+      end
+    end
+
+    it 'can customize zebrastripes' do
+      captured = capture do
+        tabula_rasa @survivors, zebra: %w{foo bar} do |t|
+          t.column :first_name
+        end
+      end
+      rows = extract_all('table tbody tr', captured)
+
+      rows[0].attribute('class').value.must_match /foo/
+      rows[1].attribute('class').value.must_match /bar/
+      rows[2].attribute('class').value.must_match /foo/
+    end
+
+    it 'raises ArgumentError when invalid zebra option given' do
+      proc {
+        tabula_rasa @survivors, zebra: 'string' do |t|
+          t.column :first_name
+        end
+      }.must_raise ArgumentError
+    end
+
+    it 'can set dom_id via option on the main method' do
+      captured = capture do
+        tabula_rasa @survivors, dom_id: true do |t|
+          t.column :first_name
+        end
+      end
+      rows = extract_all('table tbody tr', captured)
+
+      rows.each_with_index do |row, n|
+        row.attribute('id').value.must_equal "survivor_#{@survivors[n].id}"
+      end
+    end
+
+    it 'overrides dom_id option when id is explicitly set on row' do
+      captured = capture do
+        tabula_rasa @survivors, dom_id: true do |t|
+          t.column :first_name
+          t.row do |r|
+            r.id do |instance|
+              "manual_#{instance.id}"
+            end
+          end
+        end
+      end
+      rows = extract_all('table tbody tr', captured)
+
+      rows.each_with_index do |row, n|
+        row.attribute('id').value.must_equal "manual_#{@survivors[n].id}"
+      end
+    end
+
+    it 'can set dom_class via option on the main method' do
+      captured = capture do
+        tabula_rasa @survivors, dom_class: true do |t|
+          t.column :first_name
+        end
+      end
+      rows = extract_all('table tbody tr', captured)
+
+      rows.each_with_index do |row, n|
+        row.attribute('class').value.must_match dom_class(@survivors[n])
+      end
+    end
+
+    it 'can set attributes via options on internal row method' do
+      captured = capture do
+        tabula_rasa @survivors do |t|
+          t.column :first_name
+          t.row class: 'useless_row_class_for_testing'
+        end
+      end
+      rows = extract_all('table tbody tr', captured)
+      rows.each do |row|
+        row.attribute('class').value.must_match /useless_row_class_for_testing/
+      end
+
+      # Zebrastripes should still work as well
+      rows[0].attribute('class').value.must_match /even/
+      rows[1].attribute('class').value.must_match /odd/
+      rows[2].attribute('class').value.must_match /even/
+    end
+
+    it 'can set attributes via block on internal row method' do
+      captured = capture do
+        tabula_rasa @survivors do |t|
+          t.column :first_name
+          t.row do |r|
+            r.id do |instance|
+              "survivor_id_#{instance.id}"
+            end
+            r.class do |instance|
+              "survivor_class_#{instance.id}"
+            end
+            r.data :foo do |instance|
+              "foo_#{instance.id}"
+            end
+          end
+        end
+      end
+      rows = extract_all('table tbody tr', captured)
+
+      rows.each_with_index do |row, n|
+        row.attribute('id').value.must_match "survivor_id_#{@survivors[n].id}"
+        row.attribute('class').value.must_match "survivor_class_#{@survivors[n].id}"
+        row.attribute('data-foo').value.must_match "foo_#{@survivors[n].id}"
+      end
+
+      # Zebrastripes should still work as well
+      rows[0].attribute('class').value.must_match /even/
+      rows[1].attribute('class').value.must_match /odd/
+      rows[2].attribute('class').value.must_match /even/
+    end
+
+    it 'raises ArgumentError if row is defined more than once' do
+      proc {
+        tabula_rasa @survivors do |t|
+          t.row class: 'foo'
+          t.row class: 'bar'
+        end
+      }.must_raise(ArgumentError)
+    end
+  end
 
   describe 'TD attributes' do
     it 'sets no attributes by default' do
@@ -124,11 +272,11 @@ describe TabulaRasa::Helpers, 'Head Specs' do
       body.attributes.must_be_empty
     end
 
-    it 'can override first argument via a Proc argument for internal column method' do
+    it 'can override first argument via a block for internal column method' do
       captured = capture do
         tabula_rasa @survivors do |t|
           t.column :whole_name do |c|
-            c.body do |instance|
+            c.value do |instance|
               "#{instance.first_name} #{instance.last_name}"
             end
           end
@@ -142,11 +290,11 @@ describe TabulaRasa::Helpers, 'Head Specs' do
       end
     end
 
-    it 'can override first argument via a Proc argument for internal column method while setting attributes via options[:body]' do
+    it 'can override first argument via a block for internal column method while setting attributes via options[:body]' do
       captured = capture do
         tabula_rasa @survivors do |t|
           t.column :whole_name, body: {id: 'id_value', class: 'class_value', arbitrary: 'arbitrary_value', value: 'Overridden Value'} do |c|
-            c.body do |instance|
+            c.value do |instance|
               "#{instance.first_name} #{instance.last_name}"
             end
           end
@@ -169,5 +317,23 @@ describe TabulaRasa::Helpers, 'Head Specs' do
       table.attributes.must_be_empty
       body.attributes.must_be_empty
     end
+
+    it 'indicates the table is empty when collection is empty' do
+      @survivors = Survivor.none
+      captured = capture do
+        tabula_rasa @survivors do |t|
+          t.column :first_name
+        end
+      end
+      table = extract_first('table', captured)
+      body = extract_first('table tbody', captured)
+      cells = extract_all('table tbody td', captured)
+
+      cells.size.must_equal 1
+      cell = cells.first
+      cell.children.size.must_equal 1
+      cell.children.first.must_be :text?
+      cell.text.must_equal 'No survivors present'
+    end
   end
 end

data/spec/tabula_rasa/helper_head_spec.rb

@@ -35,8 +35,9 @@ describe TabulaRasa::Helpers, 'Head Specs' do
     end
   end
 
-  # NOTE: For now, I don't see the point of adding attributes on TR element.
+  # NOTE: For now, I don't see the point of adding attributes on TR element on THEAD.
   # I've had varying success styling on that and tend to style on the TH elements instead.
+  # Also, unlike TBODY, there's only one row so... it's THEAD TR. Done!
 
   describe 'TH attributes' do
     it 'sets no attributes by default' do
@@ -115,7 +116,7 @@ describe TabulaRasa::Helpers, 'Head Specs' do
       head.attributes.must_be_empty
     end
 
-    it 'raises ArgumentError if attempting to override content via a proc [as body can]' do
+    it 'raises ArgumentError if attempting to override content via a block [as body can]' do
       proc {
         tabula_rasa @survivors do |t|
           t.column :first_name do |c|

data/spec/tabula_rasa/helper_integrated_spec.rb

@@ -26,11 +26,11 @@ describe TabulaRasa::Helpers, 'Integrated [Head and Body] Specs' do
     end
   end
 
-  it 'can override body values via proc while using attribute argument for head value' do
+  it 'can override body values via a block while using attribute argument for head value' do
     captured = capture do
       tabula_rasa @survivors do |t|
         t.column :whole_name do |c|
-          c.body do |instance|
+          c.value do |instance|
             "#{instance.first_name} #{instance.last_name}"
           end
         end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: tabula_rasa
 version: !ruby/object:Gem::Version
-  version: 0.0.2
+  version: 0.0.3
 platform: ruby
 authors:
 - RSL
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-10-30 00:00:00.000000000 Z
+date: 2013-11-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -125,6 +125,7 @@ files:
 - lib/tabula_rasa/base.rb
 - lib/tabula_rasa/column.rb
 - lib/tabula_rasa/helpers.rb
+- lib/tabula_rasa/row.rb
 - lib/tabula_rasa/version.rb
 - spec/config/active_record.rb
 - spec/config/spec.rb