Merge pull request #65 from pepijnve/icon_size

Add option to interpret size option as icon height rather than font size

Author: jessedoyle

lib/prawn/icon/interface.rb

@@ -225,7 +225,16 @@ def format_hash
 
     def render
       @pdf.font(@data.path) do
-        @pdf.text @unicode, @options
+        opts = @options
+
+        if @options.fetch(:size_mode, :font_size) == :icon_height
+          requested_size = @options[:size] || @pdf.font_size
+          actual_height = @pdf.font.height_at(requested_size)
+          adjusted_size = requested_size / actual_height * requested_size
+          opts = opts.merge(size: adjusted_size)
+        end
+
+        @pdf.text @unicode, opts
       end
     end
 

spec/integration/icon_spec.rb

@@ -18,6 +18,13 @@
 
           expect(text.font_settings.first[:size]).to eq(60)
         end
+
+        it 'should handle text options (icon height)' do
+          pdf.icon 'far-address-book', size: 60, size_mode: :icon_height
+          text = PDF::Inspector::Text.analyze(pdf.render)
+
+          expect(text.font_settings.first[:size]).to be_within(0.0001).of(58.25243)
+        end
       end
 
       context 'inline_format: true' do