prawn-html 0.4.2 → 0.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3eec6595a60748790725f32e92e7b749d92086e5331d8a572d287a3a90f9d83a
-  data.tar.gz: 89bcbe740f84c4ecdea2877e96d0f677a25bf424325b41fa40caf3d1351fba96
+  metadata.gz: e82e0a464e39fd4c47c399cb043ceefe090524b29ead1c06795a484d4c648388
+  data.tar.gz: e124a7fb9800a434f42d403b478c6a6cb30b4e7d1e7e540d015028dec6daaa59
 SHA512:
-  metadata.gz: 9b34f31b07aaac30910fd1bcb2118b9cbe92bbdaf4ac80850520887e7c3803a6bb1217764551baeeb1a88d695d7b26c4885a4d3065bbd1f20527d37a5723c7bf
-  data.tar.gz: 6ed594bd070e4e4cf20519c28094f7d57eeb779f60c4867f380aab22946d20401bac307e17db47c0560b2cf1cdc591d80e07c3ff54999db92bd3040e19cc18d7
+  metadata.gz: c08e1252e2c9c8f1591840179549d3e14483d91361b1a9f56d96f23cfa30829b98d8d2ace33808a73a9467e1ff37054d7f462d1c7170db273a4ff83b203d807f
+  data.tar.gz: '0979bd9a66e463ee8e4ea3c141a2d4e2fcc0e3b1bc82397aeda2e357c10c4518b00b1e1e4fc3f2ec786070c6d229241423cd5933a89fbc9137bdccc211d22a44'

data/lib/prawn-html.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 module PrawnHtml
-  PX = 0.66 # conversion constant for pixel sixes
+  PX = 0.6 # conversion constant for pixel sixes
 
   COLORS = {
     'aliceblue' => 'f0f8ff',

data/lib/prawn_html/attributes.rb

@@ -7,16 +7,15 @@ module PrawnHtml
     attr_reader :styles
 
     STYLES_APPLY = {
-      block: %i[align leading left margin_left padding_left position top],
+      block: %i[align bottom leading left margin_left padding_left position right top],
       tag_close: %i[margin_bottom padding_bottom break_after],
       tag_open: %i[margin_top padding_top break_before],
-      text_node: %i[background callback character_spacing color font link list_style_type size styles white_space]
+      text_node: %i[callback character_spacing color font link list_style_type size styles white_space]
     }.freeze
 
     STYLES_LIST = {
       # text node styles
-      'background' => { key: :background, set: :convert_color },
-      'callback' => { key: :callback, set: :copy_value },
+      'background' => { key: :callback, set: :callback_background },
       'color' => { key: :color, set: :convert_color },
       'font-family' => { key: :font, set: :unquote },
       'font-size' => { key: :size, set: :convert_size },
@@ -25,7 +24,7 @@ module PrawnHtml
       'href' => { key: :link, set: :copy_value },
       'letter-spacing' => { key: :character_spacing, set: :convert_float },
       'list-style-type' => { key: :list_style_type, set: :unquote },
-      'text-decoration' => { key: :styles, set: :append_styles },
+      'text-decoration' => { key: :styles, set: :append_text_decoration },
       'vertical-align' => { key: :styles, set: :append_styles },
       'white-space' => { key: :white_space, set: :convert_symbol },
       # tag opening styles
@@ -37,13 +36,15 @@ module PrawnHtml
       'margin-bottom' => { key: :margin_bottom, set: :convert_size },
       'padding-bottom' => { key: :padding_bottom, set: :convert_size },
       # block styles
-      'left' => { key: :left, set: :convert_size },
+      'bottom' => { key: :bottom, set: :convert_size, options: :height },
+      'left' => { key: :left, set: :convert_size, options: :width },
       'line-height' => { key: :leading, set: :convert_size },
       'margin-left' => { key: :margin_left, set: :convert_size },
       'padding-left' => { key: :padding_left, set: :convert_size },
       'position' => { key: :position, set: :convert_symbol },
+      'right' => { key: :right, set: :convert_size, options: :width },
       'text-align' => { key: :align, set: :convert_symbol },
-      'top' => { key: :top, set: :convert_size }
+      'top' => { key: :top, set: :convert_size, options: :height }
     }.freeze
 
     STYLES_MERGE = %i[margin_left padding_left].freeze
@@ -67,9 +68,10 @@ module PrawnHtml
     # Merge text styles
     #
     # @param text_styles [String] styles to parse and process
-    def merge_text_styles!(text_styles)
+    # @param options [Hash] options (container width/height/etc.)
+    def merge_text_styles!(text_styles, options: {})
       hash_styles = Attributes.parse_styles(text_styles)
-      process_styles(hash_styles) unless hash_styles.empty?
+      process_styles(hash_styles, options: options) unless hash_styles.empty?
     end
 
     class << self
@@ -100,21 +102,33 @@ module PrawnHtml
 
     private
 
-    def apply_rule!(result, rule, value)
-      return unless rule
+    def process_styles(hash_styles, options:)
+      hash_styles.each do |key, value|
+        rule = evaluate_rule(key, value)
+        apply_rule!(merged_styles: @styles, rule: rule, value: value, options: options)
+      end
+      @styles
+    end
 
-      if rule[:set] == :append_styles
-        (result[rule[:key]] ||= []) << Utils.normalize_style(value)
-      else
-        result[rule[:key]] = Utils.send(rule[:set], value)
+    def evaluate_rule(rule_key, attr_value)
+      rule = STYLES_LIST[rule_key]
+      if rule && rule[:set] == :append_text_decoration
+        return { key: :callback, set: :callback_strike_through } if attr_value == 'line-through'
+
+        return { key: :styles, set: :append_styles }
       end
+      rule
     end
 
-    def process_styles(hash_styles)
-      hash_styles.each do |key, value|
-        apply_rule!(@styles, STYLES_LIST[key], value)
+    def apply_rule!(merged_styles:, rule:, value:, options:)
+      return unless rule
+
+      if rule[:set] == :append_styles
+        (merged_styles[rule[:key]] ||= []) << Utils.normalize_style(value)
+      else
+        opts = rule[:options] ? options[rule[:options]] : nil
+        merged_styles[rule[:key]] = Utils.send(rule[:set], value, options: opts)
       end
-      @styles
     end
   end
 end

data/lib/prawn_html/callbacks/{highlight.rb → background.rb}

@@ -2,12 +2,12 @@
 
 module PrawnHtml
   module Callbacks
-    class Highlight
+    class Background
       DEF_HIGHLIGHT = 'ffff00'
 
-      def initialize(pdf, item)
+      def initialize(pdf, color = nil)
         @pdf = pdf
-        @color = item.delete(:background) || DEF_HIGHLIGHT
+        @color = color || DEF_HIGHLIGHT
       end
 
       def render_behind(fragment)

data/lib/prawn_html/context.rb

@@ -2,15 +2,17 @@
 
 module PrawnHtml
   class Context < Array
-    DEF_FONT_SIZE = 10.3
+    DEF_FONT_SIZE = 16 * PX
 
-    attr_accessor :last_margin, :last_text_node
+    attr_reader :previous_tag
+    attr_accessor :last_text_node
 
     # Init the Context
     def initialize(*_args)
       super
-      @last_margin = 0
       @last_text_node = false
+      @merged_styles = nil
+      @previous_tag = nil
     end
 
     # Add an element to the context
@@ -25,6 +27,7 @@ module PrawnHtml
       element.parent = last
       push(element)
       element.on_context_add(self) if element.respond_to?(:on_context_add)
+      @merged_styles = nil
       self
     end
 
@@ -49,11 +52,21 @@ module PrawnHtml
     # Merge the context styles for text nodes
     #
     # @return [Hash] the hash of merged styles
-    def text_node_styles
-      each_with_object(base_styles) do |element, res|
-        evaluate_element_styles(element, res)
-        element.update_styles(res) if element.respond_to?(:update_styles)
-      end
+    def merged_styles
+      @merged_styles ||=
+        each_with_object(base_styles) do |element, res|
+          evaluate_element_styles(element, res)
+          element.update_styles(res) if element.respond_to?(:update_styles)
+        end
+    end
+
+    # Remove the last element from the context
+    def remove_last
+      last.on_context_remove(self) if last.respond_to?(:on_context_remove)
+      @merged_styles = nil
+      @last_text_node = false
+      @previous_tag = last.tag
+      pop
     end
 
     private

data/lib/prawn_html/document_renderer.rb

@@ -11,6 +11,7 @@ module PrawnHtml
     def initialize(pdf)
       @buffer = []
       @context = Context.new
+      @last_margin = 0
       @pdf = pdf
     end
 
@@ -20,8 +21,7 @@ module PrawnHtml
     def on_tag_close(element)
       render_if_needed(element)
       apply_tag_close_styles(element)
-      context.last_text_node = false
-      context.pop
+      context.remove_last
     end
 
     # On tag open callback
@@ -35,8 +35,9 @@ module PrawnHtml
       tag_class = Tag.class_for(tag_name)
       return unless tag_class
 
-      tag_class.new(tag_name, attributes: attributes, element_styles: element_styles).tap do |element|
-        setup_element(element)
+      options = { width: pdf.page_width, height: pdf.page_height }
+      tag_class.new(tag_name, attributes: attributes, options: options).tap do |element|
+        setup_element(element, element_styles: element_styles)
       end
     end
 
@@ -48,7 +49,7 @@ module PrawnHtml
     def on_text_node(content)
       return if content.match?(/\A\s*\Z/)
 
-      buffer << context.text_node_styles.merge(text: prepare_text(content))
+      buffer << context.merged_styles.merge(text: prepare_text(content))
       context.last_text_node = true
       nil
     end
@@ -59,19 +60,20 @@ module PrawnHtml
 
       output_content(buffer.dup, context.block_styles)
       buffer.clear
-      context.last_margin = 0
+      @last_margin = 0
     end
 
     alias_method :flush, :render
 
     private
 
-    attr_reader :buffer, :context, :pdf
+    attr_reader :buffer, :context, :last_margin, :pdf
 
-    def setup_element(element)
+    def setup_element(element, element_styles:)
       add_space_if_needed unless render_if_needed(element)
-      apply_tag_open_styles(element)
       context.add(element)
+      element.process_styles(element_styles: element_styles)
+      apply_tag_open_styles(element)
       element.custom_render(pdf, context) if element.respond_to?(:custom_render)
     end
 
@@ -89,14 +91,14 @@ module PrawnHtml
 
     def apply_tag_close_styles(element)
       tag_styles = element.tag_close_styles
-      context.last_margin = tag_styles[:margin_bottom].to_f
-      pdf.advance_cursor(context.last_margin + tag_styles[:padding_bottom].to_f)
+      @last_margin = tag_styles[:margin_bottom].to_f
+      pdf.advance_cursor(last_margin + tag_styles[:padding_bottom].to_f)
       pdf.start_new_page if tag_styles[:break_after]
     end
 
     def apply_tag_open_styles(element)
       tag_styles = element.tag_open_styles
-      move_down = (tag_styles[:margin_top].to_f - context.last_margin) + tag_styles[:padding_top].to_f
+      move_down = (tag_styles[:margin_top].to_f - last_margin) + tag_styles[:padding_top].to_f
       pdf.advance_cursor(move_down) if move_down > 0
       pdf.start_new_page if tag_styles[:break_before]
     end
@@ -111,24 +113,41 @@ module PrawnHtml
     def output_content(buffer, block_styles)
       apply_callbacks(buffer)
       left_indent = block_styles[:margin_left].to_f + block_styles[:padding_left].to_f
-      options = block_styles.slice(:align, :leading, :mode, :padding_left)
-      options[:indent_paragraphs] = left_indent if left_indent > 0
-      pdf.puts(buffer, options, bounding_box: bounds(block_styles))
+      options = block_styles.slice(:align, :indent_paragraphs, :leading, :mode, :padding_left)
+      options[:leading] = adjust_leading(buffer, options[:leading])
+      pdf.puts(buffer, options, bounding_box: bounds(buffer, options, block_styles), left_indent: left_indent)
     end
 
     def apply_callbacks(buffer)
       buffer.select { |item| item[:callback] }.each do |item|
-        callback = Tag::CALLBACKS[item[:callback]]
-        item[:callback] = callback.new(pdf, item)
+        callback, arg = item[:callback]
+        callback_class = Tag::CALLBACKS[callback]
+        item[:callback] = callback_class.new(pdf, arg)
       end
     end
 
-    def bounds(block_styles)
+    def adjust_leading(buffer, leading)
+      return leading if leading
+
+      (buffer.map { |item| item[:size] || Context::DEF_FONT_SIZE }.max * 0.055).round(4)
+    end
+
+    def bounds(buffer, options, block_styles)
       return unless block_styles[:position] == :absolute
 
-      y = pdf.bounds.height - (block_styles[:top] || 0)
-      w = pdf.bounds.width - (block_styles[:left] || 0)
-      [[block_styles[:left] || 0, y], { width: w }]
+      x = if block_styles.include?(:right)
+            x1 = pdf.calc_buffer_width(buffer) + block_styles[:right]
+            x1 < pdf.page_width ? (pdf.page_width - x1) : 0
+          else
+            block_styles[:left] || 0
+          end
+      y = if block_styles.include?(:bottom)
+            pdf.calc_buffer_height(buffer, options) + block_styles[:bottom]
+          else
+            pdf.page_height - (block_styles[:top] || 0)
+          end
+
+      [[x, y], { width: pdf.page_width - x }]
     end
   end
 end

data/lib/prawn_html/pdf_wrapper.rb

@@ -6,7 +6,7 @@ module PrawnHtml
   class PdfWrapper
     extend Forwardable
 
-    def_delegators :@pdf, :bounds, :start_new_page
+    def_delegators :@pdf, :start_new_page
 
     # Wrapper for Prawn PDF Document
     #
@@ -24,6 +24,46 @@ module PrawnHtml
       pdf.move_down(move_down)
     end
 
+    # Calculate the height of a buffer of items
+    #
+    # @param buffer [Array] Buffer of items
+    # @param options [Hash] Output options
+    #
+    # @return [Float] calculated height
+    def calc_buffer_height(buffer, options)
+      pdf.height_of_formatted(buffer, options)
+    end
+
+    # Calculate the width of a buffer of items
+    #
+    # @param buffer [Array] Buffer of items
+    #
+    # @return [Float] calculated width
+    def calc_buffer_width(buffer)
+      width = 0
+      buffer.each do |item|
+        font_family = item[:font] || pdf.font.name
+        pdf.font(font_family, size: item[:size] || pdf.font_size) do
+          width += pdf.width_of(item[:text], inline_format: true)
+        end
+      end
+      width
+    end
+
+    # Height of the page
+    #
+    # @return [Float] height
+    def page_height
+      pdf.bounds.height
+    end
+
+    # Width of the page
+    #
+    # @return [Float] width
+    def page_width
+      pdf.bounds.width
+    end
+
     # Draw a rectangle
     #
     # @param x [Float] left position of the rectangle
@@ -66,12 +106,12 @@ module PrawnHtml
     # @param buffer [Array] array of text items
     # @param options [Hash] hash of options
     # @param bounding_box [Array] bounding box arguments, if bounded
-    def puts(buffer, options, bounding_box: nil)
-      return pdf.formatted_text(buffer, options) unless bounding_box
+    def puts(buffer, options, bounding_box: nil, left_indent: 0)
+      return output_buffer(buffer, options, left_indent: left_indent) unless bounding_box
 
       current_y = pdf.cursor
       pdf.bounding_box(*bounding_box) do
-        pdf.formatted_text(buffer, options)
+        output_buffer(buffer, options, left_indent: left_indent)
       end
       pdf.move_cursor_to(current_y)
     end
@@ -90,5 +130,12 @@ module PrawnHtml
     private
 
     attr_reader :pdf
+
+    def output_buffer(buffer, options, left_indent:)
+      formatted_text = proc { pdf.formatted_text(buffer, options) }
+      return formatted_text.call if left_indent == 0
+
+      pdf.indent(left_indent, 0, &formatted_text)
+    end
   end
 end

data/lib/prawn_html/tag.rb

@@ -3,7 +3,7 @@
 module PrawnHtml
   class Tag
     CALLBACKS = {
-      'Highlight' => Callbacks::Highlight,
+      'Background' => Callbacks::Background,
       'StrikeThrough' => Callbacks::StrikeThrough
     }.freeze
     TAG_CLASSES = %w[A B Blockquote Body Br Code Del Div H Hr I Img Li Mark Ol P Pre Small Span Sub Sup U Ul].freeze
@@ -15,11 +15,11 @@ module PrawnHtml
     #
     # @param tag [Symbol] tag name
     # @param attributes [Hash] hash of element attributes
-    # @param element_styles [String] document styles tp apply to the element
-    def initialize(tag, attributes: {}, element_styles: '')
+    # @param options [Hash] options (container width/height/etc.)
+    def initialize(tag, attributes: {}, options: {})
       @tag = tag
+      @options = options
       @attrs = Attributes.new(attributes)
-      process_styles(element_styles, attributes['style'])
     end
 
     # Is a block tag?
@@ -38,6 +38,15 @@ module PrawnHtml
       block_styles
     end
 
+    # Process tag styles
+    #
+    # @param element_styles [String] extra styles to apply to the element
+    def process_styles(element_styles: nil)
+      attrs.merge_text_styles!(tag_styles, options: options) if respond_to?(:tag_styles)
+      attrs.merge_text_styles!(element_styles, options: options) if element_styles
+      attrs.merge_text_styles!(attrs.style, options: options)
+    end
+
     # Styles to apply on tag closing
     #
     # @return [Hash] hash of styles to apply
@@ -77,10 +86,6 @@ module PrawnHtml
 
     private
 
-    def process_styles(element_styles, inline_styles)
-      attrs.merge_text_styles!(tag_styles) if respond_to?(:tag_styles)
-      attrs.merge_text_styles!(element_styles)
-      attrs.merge_text_styles!(inline_styles)
-    end
+    attr_reader :options
   end
 end

data/lib/prawn_html/tags/a.rb

@@ -6,7 +6,13 @@ module PrawnHtml
       ELEMENTS = [:a].freeze
 
       def tag_styles
-        "href: #{attrs.href}" if attrs.href
+        return unless attrs.href
+
+        <<~STYLES
+          color: #00E;
+          href: #{attrs.href};
+          text-decoration: underline;
+        STYLES
       end
     end
   end

data/lib/prawn_html/tags/blockquote.rb

@@ -5,9 +5,9 @@ module PrawnHtml
     class Blockquote < Tag
       ELEMENTS = [:blockquote].freeze
 
-      MARGIN_BOTTOM = 10
-      MARGIN_LEFT = 25
-      MARGIN_TOP = 10
+      MARGIN_BOTTOM = 12.7
+      MARGIN_LEFT = 40.4
+      MARGIN_TOP = 12.7
 
       def block?
         true

data/lib/prawn_html/tags/body.rb

@@ -4,6 +4,10 @@ module PrawnHtml
   module Tags
     class Body < Tag
       ELEMENTS = [:body].freeze
+
+      def block?
+        true
+      end
     end
   end
 end

data/lib/prawn_html/tags/br.rb

@@ -5,14 +5,14 @@ module PrawnHtml
     class Br < Tag
       ELEMENTS = [:br].freeze
 
-      BR_SPACING = Utils.convert_size('12')
+      BR_SPACING = Utils.convert_size('17')
 
       def block?
         true
       end
 
       def custom_render(pdf, context)
-        return if context.last_text_node
+        return if context.last_text_node || context.previous_tag != :br
 
         pdf.advance_cursor(BR_SPACING)
       end

data/lib/prawn_html/tags/del.rb

@@ -6,7 +6,7 @@ module PrawnHtml
       ELEMENTS = [:del, :s].freeze
 
       def tag_styles
-        'callback: StrikeThrough'
+        'text-decoration: line-through'
       end
     end
   end

data/lib/prawn_html/tags/h.rb

@@ -6,30 +6,30 @@ module PrawnHtml
       ELEMENTS = [:h1, :h2, :h3, :h4, :h5, :h6].freeze
 
       MARGINS_TOP = {
-        h1: 25.5,
+        h1: 25,
         h2: 20.5,
-        h3: 19,
-        h4: 20,
+        h3: 18,
+        h4: 21.2,
         h5: 21.2,
-        h6: 23.5
+        h6: 22.8
       }.freeze
 
       MARGINS_BOTTOM = {
-        h1: 18.2,
-        h2: 17.5,
-        h3: 17.5,
-        h4: 22,
-        h5: 22,
-        h6: 26.5
+        h1: 15.8,
+        h2: 15.8,
+        h3: 15.8,
+        h4: 20,
+        h5: 21.4,
+        h6: 24.8
       }.freeze
 
       SIZES = {
-        h1: 31,
-        h2: 23.5,
-        h3: 18.2,
-        h4: 16,
+        h1: 31.5,
+        h2: 24,
+        h3: 18.7,
+        h4: 15.7,
         h5: 13,
-        h6: 10.5
+        h6: 10.8
       }.freeze
 
       def block?

data/lib/prawn_html/tags/img.rb

@@ -12,16 +12,17 @@ module PrawnHtml
       def custom_render(pdf, context)
         parsed_styles = Attributes.parse_styles(attrs.style)
         block_styles = context.block_styles
-        evaluated_styles = evaluate_styles(pdf, block_styles.merge(parsed_styles))
+        evaluated_styles = adjust_styles(pdf, block_styles.merge(parsed_styles))
         pdf.image(@attrs.src, evaluated_styles)
       end
 
       private
 
-      def evaluate_styles(pdf, img_styles)
+      def adjust_styles(pdf, img_styles)
         {}.tap do |result|
-          result[:width] = Utils.convert_size(img_styles['width'], pdf.bounds.width) if img_styles.include?('width')
-          result[:height] = Utils.convert_size(img_styles['height'], pdf.bounds.height) if img_styles.include?('height')
+          w, h = img_styles['width'], img_styles['height']
+          result[:width] = Utils.convert_size(w, options: pdf.page_width) if w
+          result[:height] = Utils.convert_size(h, options: pdf.page_height) if h
           result[:position] = img_styles[:align] if %i[left center right].include?(img_styles[:align])
         end
       end

data/lib/prawn_html/tags/li.rb

@@ -5,6 +5,9 @@ module PrawnHtml
     class Li < Tag
       ELEMENTS = [:li].freeze
 
+      INDENT_OL = -12
+      INDENT_UL = -6
+
       def block?
         true
       end
@@ -13,9 +16,21 @@ module PrawnHtml
         @counter ? "#{@counter}. " : "#{@symbol} "
       end
 
+      def block_styles
+        super.tap do |bs|
+          bs[:indent_paragraphs] = @indent
+        end
+      end
+
       def on_context_add(_context)
-        @counter = (parent.counter += 1) if parent.is_a? Ol
-        @symbol = parent.styles[:list_style_type] || '&bullet;' if parent.is_a? Ul
+        case parent.class.to_s
+        when 'PrawnHtml::Tags::Ol'
+          @indent = INDENT_OL
+          @counter = (parent.counter += 1)
+        when 'PrawnHtml::Tags::Ul'
+          @indent = INDENT_UL
+          @symbol = parent.styles[:list_style_type] || '&bullet;'
+        end
       end
     end
   end

data/lib/prawn_html/tags/mark.rb

@@ -6,7 +6,7 @@ module PrawnHtml
       ELEMENTS = [:mark].freeze
 
       def tag_styles
-        'callback: Highlight'
+        'background: #ff0'
       end
     end
   end

data/lib/prawn_html/tags/ol.rb

@@ -5,21 +5,38 @@ module PrawnHtml
     class Ol < Tag
       ELEMENTS = [:ol].freeze
 
-      MARGIN_LEFT = 25
+      MARGIN_TOP = 15
+      MARGIN_LEFT = 40
+      MARGIN_BOTTOM = 15
 
       attr_accessor :counter
 
-      def initialize(tag, attributes: {}, element_styles: '')
+      def initialize(tag, attributes: {}, options: {})
         super
         @counter = 0
+        @first_level = false
       end
 
       def block?
         true
       end
 
+      def on_context_add(context)
+        return if context.map(&:tag).count { |el| el == :ol } > 1
+
+        @first_level = true
+      end
+
       def tag_styles
-        "margin-left: #{MARGIN_LEFT}px"
+        if @first_level
+          <<~STYLES
+            margin-top: #{MARGIN_TOP}px;
+            margin-left: #{MARGIN_LEFT}px;
+            margin-bottom: #{MARGIN_BOTTOM}px;
+          STYLES
+        else
+          "margin-left: #{MARGIN_LEFT}px"
+        end
       end
     end
   end

data/lib/prawn_html/tags/p.rb

@@ -5,8 +5,8 @@ module PrawnHtml
     class P < Tag
       ELEMENTS = [:p].freeze
 
-      MARGIN_BOTTOM = 6
-      MARGIN_TOP = 6
+      MARGIN_BOTTOM = 12.5
+      MARGIN_TOP = 12.5
 
       def block?
         true

data/lib/prawn_html/tags/ul.rb

@@ -5,14 +5,35 @@ module PrawnHtml
     class Ul < Tag
       ELEMENTS = [:ul].freeze
 
-      MARGIN_LEFT = 25
+      MARGIN_TOP = 15
+      MARGIN_LEFT = 40
+      MARGIN_BOTTOM = 15
+
+      def initialize(tag, attributes: {}, options: {})
+        super
+        @first_level = false
+      end
 
       def block?
         true
       end
 
+      def on_context_add(context)
+        return if context.map(&:tag).count { |el| el == :ul } > 1
+
+        @first_level = true
+      end
+
       def tag_styles
-        "margin-left: #{MARGIN_LEFT}px"
+        if @first_level
+          <<~STYLES
+            margin-top: #{MARGIN_TOP}px;
+            margin-left: #{MARGIN_LEFT}px;
+            margin-bottom: #{MARGIN_BOTTOM}px;
+          STYLES
+        else
+          "margin-left: #{MARGIN_LEFT}px"
+        end
       end
     end
   end

data/lib/prawn_html/utils.rb

@@ -10,6 +10,24 @@ module PrawnHtml
       'underline' => :underline
     }.freeze
 
+    # Setup a background callback
+    #
+    # @param value [String] HTML string color
+    #
+    # @return [Array] callback name and argument value
+    def callback_background(value, options: nil)
+      ['Background', convert_color(value, options: options)]
+    end
+
+    # Setup a strike through callback
+    #
+    # @param value [String] unused
+    #
+    # @return [Array] callback name and argument value
+    def callback_strike_through(value, options: nil)
+      ['StrikeThrough', nil]
+    end
+
     # Converts a color string
     #
     # Supported formats:
@@ -21,7 +39,7 @@ module PrawnHtml
     # @param value [String] HTML string color
     #
     # @return [String] adjusted string color or nil if value is invalid
-    def convert_color(value)
+    def convert_color(value, options: nil)
       val = value.to_s.strip.downcase
       return Regexp.last_match[1] if val.match /\A#([a-f0-9]{6})\Z/ # rubocop:disable Performance/RedundantMatch
 
@@ -42,7 +60,7 @@ module PrawnHtml
     # @param value [String] string decimal
     #
     # @return [Float] converted and rounded float number
-    def convert_float(value)
+    def convert_float(value, options: nil)
       val = value&.gsub(/[^0-9.]/, '') || ''
       val.to_f.round(4)
     end
@@ -50,14 +68,14 @@ module PrawnHtml
     # Converts a size string
     #
     # @param value [String] size string
-    # @param container_size [Numeric] container size
+    # @param options [Numeric] container size
     #
     # @return [Float] converted and rounded size
-    def convert_size(value, container_size = nil)
+    def convert_size(value, options: nil)
       val = value&.gsub(/[^0-9.]/, '') || ''
       val =
-        if container_size && value.include?('%')
-          val.to_f * container_size * 0.01
+        if options && value&.include?('%')
+          val.to_f * options * 0.01
         else
           val.to_f * PrawnHtml::PX
         end
@@ -69,7 +87,7 @@ module PrawnHtml
     # @param value [String] string
     #
     # @return [Symbol] symbol
-    def convert_symbol(value)
+    def convert_symbol(value, options: nil)
       value.to_sym if value && !value.match?(/\A\s*\Z/)
     end
 
@@ -78,7 +96,7 @@ module PrawnHtml
     # @param value
     #
     # @return value
-    def copy_value(value)
+    def copy_value(value, options: nil)
       value
     end
 
@@ -97,13 +115,13 @@ module PrawnHtml
     # @param value [String] string
     #
     # @return [String] string without quotes at the beginning/ending
-    def unquote(value)
+    def unquote(value, options: nil)
       (value&.strip || +'').tap do |val|
         val.gsub!(/\A['"]|["']\Z/, '')
       end
     end
 
-    module_function :convert_color, :convert_float, :convert_size, :convert_symbol, :copy_value, :normalize_style,
-                    :unquote
+    module_function :callback_background, :callback_strike_through, :convert_color, :convert_float, :convert_size,
+                    :convert_symbol, :copy_value, :normalize_style, :unquote
   end
 end

data/lib/prawn_html/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module PrawnHtml # :nodoc:
-  VERSION = '0.4.2'
+  VERSION = '0.5.0'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: prawn-html
 version: !ruby/object:Gem::Version
-  version: 0.4.2
+  version: 0.5.0
 platform: ruby
 authors:
 - Mattia Roccoberton
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-09-03 00:00:00.000000000 Z
+date: 2021-09-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: oga
@@ -48,7 +48,7 @@ files:
 - README.md
 - lib/prawn-html.rb
 - lib/prawn_html/attributes.rb
-- lib/prawn_html/callbacks/highlight.rb
+- lib/prawn_html/callbacks/background.rb
 - lib/prawn_html/callbacks/strike_through.rb
 - lib/prawn_html/context.rb
 - lib/prawn_html/document_renderer.rb