Move base styles Context method in a constant

Author: blocknotes

lib/prawn_html/context.rb

@@ -2,7 +2,9 @@
 
 module PrawnHtml
   class Context < Array
-    DEF_FONT_SIZE = 16 * PX
+    DEFAULT_STYLES = {
+      size: 16 * PX
+    }.freeze
 
     attr_reader :previous_tag
     attr_accessor :last_text_node
@@ -54,7 +56,7 @@ def block_styles
     # @return [Hash] the hash of merged styles
     def merged_styles
       @merged_styles ||=
-        each_with_object(base_styles) do |element, res|
+        each_with_object(DEFAULT_STYLES.dup) do |element, res|
           evaluate_element_styles(element, res)
           element.update_styles(res)
         end
@@ -71,12 +73,6 @@ def remove_last
 
     private
 
-    def base_styles
-      {
-        size: DEF_FONT_SIZE
-      }
-    end
-
     def evaluate_element_styles(element, res)
       styles = element.styles.slice(*Attributes::STYLES_APPLY[:text_node])
       styles.each do |key, val|

lib/prawn_html/document_renderer.rb

@@ -129,7 +129,7 @@ def apply_callbacks(buffer)
     def adjust_leading(buffer, leading)
       return leading if leading
 
-      (buffer.map { |item| item[:size] || Context::DEF_FONT_SIZE }.max * 0.055).round(4)
+      (buffer.map { |item| item[:size] || Context::DEFAULT_STYLES[:size] }.max * 0.055).round(4)
     end
 
     def bounds(buffer, options, block_styles)

lib/prawn_html/tags/small.rb

@@ -6,7 +6,7 @@ class Small < Tag
       ELEMENTS = [:small].freeze
 
       def update_styles(context_styles)
-        size = (context_styles[:size] || Context::DEF_FONT_SIZE) * 0.85
+        size = (context_styles[:size] || Context::DEFAULT_STYLES[:size]) * 0.85
         context_styles[:size] = size
         super(context_styles)
       end

spec/integrations/blocks_spec.rb

@@ -2,7 +2,7 @@
 
 RSpec.describe 'Blocks' do
   let(:pdf) { Prawn::Document.new(page_size: 'A4', page_layout: :portrait) }
-  let(:size) { PrawnHtml::Context::DEF_FONT_SIZE }
+  let(:size) { PrawnHtml::Context::DEFAULT_STYLES[:size] }
 
   before do
     PrawnHtml.append_html(pdf, html)

spec/support/test_utils.rb

@@ -3,7 +3,7 @@
 module TestUtils
   extend self
 
-  def adjust_leading(size = PrawnHtml::Context::DEF_FONT_SIZE)
+  def adjust_leading(size = PrawnHtml::Context::DEFAULT_STYLES[:size])
     (size * 0.055).round(4)
   end
 
@@ -16,7 +16,7 @@ def default_font_family
   end
 
   def default_font_size
-    PrawnHtml::Context::DEF_FONT_SIZE
+    PrawnHtml::Context::DEFAULT_STYLES[:size]
   end
 
   def font_ascender(font_family: 'Helvetica', font_size: default_font_size)

spec/units/prawn_html/context_spec.rb

@@ -121,7 +121,7 @@ def on_context_remove(context)
     subject(:merged_styles) { context.merged_styles }
 
     context 'with no elements' do
-      it { is_expected.to eq(size: PrawnHtml::Context::DEF_FONT_SIZE) }
+      it { is_expected.to eq(size: PrawnHtml::Context::DEFAULT_STYLES[:size]) }
     end
 
     context 'with some elements' do

spec/units/prawn_html/tags/small_spec.rb

@@ -27,7 +27,7 @@
     include_context 'with pdf wrapper'
 
     let(:html) { '<small>Some sample content</small>' }
-    let(:size) { PrawnHtml::Context::DEF_FONT_SIZE * 0.85 }
+    let(:size) { PrawnHtml::Context::DEFAULT_STYLES[:size] * 0.85 }
 
     before { append_html_to_pdf(html) }