prawn_hebrew 0.1.3 → 0.1.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c85ea02d489bf50f5f0c8facaab7928485a9042f324a54bfb06cd974708e5324
-  data.tar.gz: 21a63093c22ffe1baf7c9b646a000138ef05694448315f077642a0e7f72acf43
+  metadata.gz: bb9a4a47a50c88ba630e62138ad995627b797d769643e90cb88457eb10b690a6
+  data.tar.gz: e0dc4ef750250930ddd517905ae5930032bcc69d43f850106b14a2fa9050fd57
 SHA512:
-  metadata.gz: c4ab1e0ce09de9756667400c9b77b43d2f859734056adce3252ffd327af19c1fc97d79ec8c4832c794ef1020a971980e32b9dbd4adf3d4b1960772e06e2edcc7
-  data.tar.gz: dc6e7cd6daae94a23eb250652dd9d1ec754439e921cd3d0707d2b3060cf6515d1e8859e5e32502e46ba0fdbfc8a55fdf62665a0b9089e4c92871a542ea75c66c
+  metadata.gz: c1e2bb75e635bc6a04056fc91517f48eedaa07bdc15c7c74bb5dbfbd5eef4ddb1793a0f3bf2e6cb76af41f71e9411968f31892711aee5068f81132a87871ad5c
+  data.tar.gz: b6f983a0f591078eb3efc7f024cc83e0e4694daf3fb2f3f545275efc7c32f772eb227035e09e90952be9b0fde735900a5ed5fb0e0a47db1fda8bff9c9c6b3898

data/lib/prawn_hebrew.rb

@@ -160,64 +160,56 @@ module PrawnHebrew
                                    char_spacing, leading, min_font_size, rotation, box_opts)
       min_size = min_font_size || 5
       current_size = initial_size
+      fitting_size = nil
       
-      # Try rendering with progressively smaller font sizes until it fits
+      # Find the largest font size that fits by testing with dry_run
       while current_size >= min_size
-        # Create a copy of box_opts to test rendering
-        test_opts = box_opts.dup
-        
-        # Try rendering at current size
         fragments = hebrew_formatted_text(text, size: current_size, style: style,
                                           hebrew_font: hebrew_font,
                                           english_font: english_font)
         
+        test_opts = box_opts.dup
         test_opts[:leading] = leading if leading > 0
         
-        # Test if it fits by attempting to render
-        success = character_spacing(char_spacing) do
+        # Test if it fits without actually rendering
+        overflow_text = character_spacing(char_spacing) do
           if rotation != 0
             rotate(rotation, origin: test_opts[:at] || [0, 0]) do
-              result = formatted_text_box(fragments, test_opts.merge(dry_run: true))
-              result.empty? || result == ""
+              formatted_text_box(fragments, test_opts.merge(dry_run: true))
             end
           else
-            result = formatted_text_box(fragments, test_opts.merge(dry_run: true))
-            result.empty? || result == ""
+            formatted_text_box(fragments, test_opts.merge(dry_run: true))
           end
         end
         
-        # If it fits, render it for real and return
-        if success
-          character_spacing(char_spacing) do
-            if rotation != 0
-              rotate(rotation, origin: box_opts[:at] || [0, 0]) do
-                formatted_text_box(fragments, box_opts.merge(leading: leading > 0 ? leading : nil).compact)
-              end
-            else
-              formatted_text_box(fragments, box_opts.merge(leading: leading > 0 ? leading : nil).compact)
-            end
-          end
-          return
+        # formatted_text_box returns an array when there's overflow, empty array when it fits
+        if overflow_text.is_a?(Array) && overflow_text.empty?
+          fitting_size = current_size
+          break
         end
         
         # Reduce font size and try again
         current_size -= 0.5
       end
       
-      # If we've reached min size, render at min size anyway
-      fragments = hebrew_formatted_text(text, size: min_size, style: style,
+      # Use the fitting size, or min_size if nothing fit
+      final_size = fitting_size || min_size
+      
+      # Now render once with the final size
+      fragments = hebrew_formatted_text(text, size: final_size, style: style,
                                         hebrew_font: hebrew_font,
                                         english_font: english_font)
       
-      box_opts[:leading] = leading if leading > 0
+      render_opts = box_opts.dup
+      render_opts[:leading] = leading if leading > 0
       
       character_spacing(char_spacing) do
         if rotation != 0
-          rotate(rotation, origin: box_opts[:at] || [0, 0]) do
-            formatted_text_box(fragments, box_opts)
+          rotate(rotation, origin: render_opts[:at] || [0, 0]) do
+            formatted_text_box(fragments, render_opts)
           end
         else
-          formatted_text_box(fragments, box_opts)
+          formatted_text_box(fragments, render_opts)
         end
       end
     end

data/lib/version.rb

@@ -1,3 +1,3 @@
 module PrawnHebrew
-  VERSION = "0.1.3"
+  VERSION = "0.1.5"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: prawn_hebrew
 version: !ruby/object:Gem::Version
-  version: 0.1.3
+  version: 0.1.5
 platform: ruby
 authors:
 - Ben Lite