Merge pull request #24 from prawnpdf/hb_issue_22

fixes #22 - unnecessary page break with centered tables

CHANGELOG.md

@@ -1,6 +1,6 @@
 ## 0.1.2 (not released yet)
 
-...
+* fixed unnecessary page breaks with centered tables (#22, #23)
 
 ## 0.1.1
 

lib/prawn/table.rb

@@ -473,7 +473,8 @@ def initial_row_on_initial_page
       needed_height = row(0..number_of_header_rows).height
 
       # have we got enough room to fit the first row (including header row(s))
-      return -1 if fits_on_page?(needed_height)
+      use_reference_bounds = true
+      return -1 if fits_on_page?(needed_height, use_reference_bounds)
 
       # If there isn't enough room left on the page to fit the first data row
       # (including the header), start the table on the next page.
@@ -484,8 +485,13 @@ def initial_row_on_initial_page
     end
 
     # do we have enough room to fit a given height on to the current page?
-    def fits_on_page?(needed_height)
-      needed_height < @pdf.y - (@pdf.bounds.absolute_bottom - Prawn::FLOAT_PRECISION)
+    def fits_on_page?(needed_height, use_reference_bounds = false)
+      if use_reference_bounds
+        bounds = @pdf.reference_bounds
+      else
+        bounds = @pdf.bounds
+      end
+      needed_height < @pdf.y - (bounds.absolute_bottom - Prawn::FLOAT_PRECISION)
     end
 
     # return the header rows

spec/table_spec.rb

@@ -1593,4 +1593,12 @@
     end
     pdf.render
   end
+
+  it 'illustrates issue #20 (2) and #22', issue: 22 do
+    pdf = Prawn::Document.new
+    pdf.table [['one', 'two']], position: :center
+    pdf.table [['three', 'four']], position: :center
+    pdf.render
+    pdf.page_count.should == 1
+  end
 end