asciidoctor-pdf 2.0.0 → 2.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 027a0c91049b52843552b50087dc100ecaf0143bf2d342c2d3e5a8d37bd8fcda
-  data.tar.gz: 1723e9c0ab2eab7f2f25dd31e6085e31103724c27eff2e3b9a1fe818d64161af
+  metadata.gz: 0d912c0aa04b380a6b2a69420f01a3c4795bb897dd3b1fbc22316ec38f333905
+  data.tar.gz: 6a717bfc06aa32799bc4bfb24c0e67b621709eac500efb1e78c1cf9f42d06afc
 SHA512:
-  metadata.gz: d563c7b4def055b2732563910f0abc318d09e566423592a008535730480363a52f36f0d3bd27ed431490d3cf80bc855c8e98dc271556b8b27ea63ebd1a7b284d
-  data.tar.gz: 82834fe8af002bc265e09880d8ef11e25e6e9943c1bf6fb4359f655364a15b49e8b71134adbe0fb3db567759944915faabe1cf5720a558751c3fcb53266c1c3f
+  metadata.gz: 69b76de214031c839abed9c1de5a0df55925ff638e4f9eab327e8970b66e78b499a1e9966bf7df436767a71728f6e6396608ad821feebf7a995de0585ca00861
+  data.tar.gz: 5f86394d598bdd7aa2c574912f95213cefa0620f375a34c35ff15d00369c0ca36b2dcec836ac95c460f9222f5ca24fa2b7573dd85448a5b5677cba5c1d2dd164

data/CHANGELOG.adoc

@@ -5,6 +5,18 @@
 This document provides a high-level view of the changes to the {project-name} by release.
 For a detailed view of what has changed, refer to the {url-repo}/commits/main[commit history] on GitHub.
 
+== 2.0.1 (2022-05-21) - @mojavelinux
+
+Bug Fixes::
+
+* scale inline image to fit within available height of page, accounting for the top padding of the line and the bottom gutter (#2193)
+* short-circuit formatted_text routine and log error if fragments in first line cannot fit on a new page
+* break and wrap long contiguous text in source block when linenums are enabled (#2198)
+
+=== Details
+
+{url-repo}/releases/tag/v2.0.1[git tag] | {url-repo}/compare/v2.0.0\...v2.0.1[full diff]
+
 == 2.0.0 (2022-05-18) - @mojavelinux
 
 Improvements::

data/README.adoc

@@ -1,6 +1,6 @@
 = Asciidoctor PDF: A native PDF converter for AsciiDoc
 Dan Allen <https://github.com/mojavelinux[@mojavelinux]>; Sarah White <https://github.com/graphitefriction[@graphitefriction]>
-v2.0.0, 2022-05-18
+v2.0.1, 2022-05-21
 // Settings:
 :experimental:
 :idprefix:
@@ -24,7 +24,7 @@ endif::[]
 :project-handle: asciidoctor-pdf
 // Variables:
 :release-line: 2.0.x
-:release-version: 2.0.0
+:release-version: 2.0.1
 // URLs:
 :url-gem: https://rubygems.org/gems/asciidoctor-pdf
 :url-project: https://github.com/asciidoctor/asciidoctor-pdf

data/lib/asciidoctor/pdf/converter.rb

@@ -717,7 +717,7 @@ module Asciidoctor
             when 'page'
               pagenums = term.dests.uniq {|dest| dest[:page] }.map {|dest| %(<a anchor="#{dest[:anchor]}">#{dest[:page]}</a>) }
             when 'range'
-              first_anchor_per_page = term.dests.each_with_object({}) {|dest, accum| accum[dest[:page]] ||= dest[:anchor] }
+              first_anchor_per_page = {}.tap {|accum| term.dests.each {|dest| accum[dest[:page]] ||= dest[:anchor] } }
               pagenums = (consolidate_ranges first_anchor_per_page.keys).map do |range|
                 anchor = first_anchor_per_page[(range.include? '-') ? (range.partition '-')[0] : range]
                 %(<a anchor="#{anchor}">#{range}</a>)
@@ -814,7 +814,7 @@ module Asciidoctor
         if (text_align = resolve_text_align_from_role (roles = node.roles), query_theme: true, remove_predefined: true)
           prose_opts[:align] = text_align
         end
-        role_keys = roles.map {|role| %(role_#{role}).to_sym } unless roles.empty?
+        role_keys = roles.map {|role| %(role_#{role}) } unless roles.empty?
         if (text_indent = @theme.prose_text_indent) > 0 ||
             ((text_indent = @theme.prose_text_indent_inner) > 0 && node.previous_sibling&.context == :paragraph)
           prose_opts[:indent_paragraphs] = text_indent
@@ -1081,7 +1081,7 @@ module Asciidoctor
                   highlight_lines = nil
                 else
                   pg_highlight_bg_color = pg_block_styles[:highlight_background_color]
-                  highlight_lines = highlight_lines.map {|linenum| [linenum, pg_highlight_bg_color] }.to_h
+                  highlight_lines = {}.tap {|accum| highlight_lines.each {|linenum| accum[linenum] = pg_highlight_bg_color } }
                 end
               end
               if (node.option? 'linenums') || (node.attr? 'linenums')
@@ -1120,7 +1120,7 @@ module Asciidoctor
               lexer ||= ::Rouge::Lexers::PlainText
               source_string, conum_mapping = extract_conums source_string if callouts_enabled
               if (node.attr? 'highlight') && !(hl_lines = (node.resolve_lines_to_highlight source_string, (node.attr 'highlight'))).empty?
-                formatter_opts[:highlight_lines] = hl_lines.map {|linenum| [linenum, true] }.to_h
+                formatter_opts[:highlight_lines] = {}.tap {|accum| hl_lines.each {|linenum| accum[linenum] = true } }
               end
               fragments = formatter.format (lexer.lex source_string), formatter_opts rescue [text: source_string]
               source_chunks = conum_mapping ? (restore_conums fragments, conum_mapping) : fragments
@@ -1396,7 +1396,7 @@ module Asciidoctor
           end
           theme_margin :prose, :bottom, (next_enclosed_block actual_node) #unless actual_node.nested?
         when 'qanda'
-          @list_numerals << '1'
+          @list_numerals << 1
           convert_list node
           @list_numerals.pop
         else
@@ -3261,64 +3261,70 @@ module Asciidoctor
             end
           end
 
-          colspec_dict = PageSides.each_with_object({}) do |side, acc|
-            side_trim_content_width = trim_content_width[side]
-            if (custom_colspecs = @theme[%(#{periphery}_#{side}_columns)] || @theme[%(#{periphery}_columns)])
-              case (colspecs = (custom_colspecs.to_s.tr ',', ' ').split).size
-              when 0, 1
-                colspecs = { left: '0', center: colspecs[0] || '100', right: '0' }
-              when 2
-                colspecs = { left: colspecs[0], center: '0', right: colspecs[1] }
-              else # 3
-                colspecs = { left: colspecs[0], center: colspecs[1], right: colspecs[2] }
-              end
-              tot_width = 0
-              side_colspecs = colspecs.map do |col, spec|
-                if (alignment_char = spec.chr).to_i.to_s == alignment_char
-                  alignment = :left
-                  rel_width = spec.to_f
-                else
-                  alignment = AlignmentTable[alignment_char]
-                  rel_width = (spec.slice 1, spec.length).to_f
+          colspec_dict = {}.tap do |acc|
+            PageSides.each do |side|
+              side_trim_content_width = trim_content_width[side]
+              if (custom_colspecs = @theme[%(#{periphery}_#{side}_columns)] || @theme[%(#{periphery}_columns)])
+                case (colspecs = (custom_colspecs.to_s.tr ',', ' ').split).size
+                when 0, 1
+                  colspecs = { left: '0', center: colspecs[0] || '100', right: '0' }
+                when 2
+                  colspecs = { left: colspecs[0], center: '0', right: colspecs[1] }
+                else # 3
+                  colspecs = { left: colspecs[0], center: colspecs[1], right: colspecs[2] }
                 end
-                tot_width += rel_width
-                [col, align: alignment, width: rel_width, x: 0]
-              end.to_h
-              # QUESTION: should we allow the columns to overlap (capping width at 100%)?
-              side_colspecs.each {|_, colspec| colspec[:width] = (colspec[:width] / tot_width) * side_trim_content_width }
-              side_colspecs[:right][:x] = (side_colspecs[:center][:x] = side_colspecs[:left][:width]) + side_colspecs[:center][:width]
-              acc[side] = side_colspecs
-            else
-              acc[side] = {
-                left: { align: :left, width: side_trim_content_width, x: 0 },
-                center: { align: :center, width: side_trim_content_width, x: 0 },
-                right: { align: :right, width: side_trim_content_width, x: 0 },
-              }
+                tot_width = 0
+                side_colspecs = {}.tap do |accum|
+                  colspecs.each do |col, spec|
+                    if (alignment_char = spec.chr).to_i.to_s == alignment_char
+                      alignment = :left
+                      rel_width = spec.to_f
+                    else
+                      alignment = AlignmentTable[alignment_char]
+                      rel_width = (spec.slice 1, spec.length).to_f
+                    end
+                    tot_width += rel_width
+                    accum[col] = { align: alignment, width: rel_width, x: 0 }
+                  end
+                end
+                # QUESTION: should we allow the columns to overlap (capping width at 100%)?
+                side_colspecs.each {|_, colspec| colspec[:width] = (colspec[:width] / tot_width) * side_trim_content_width }
+                side_colspecs[:right][:x] = (side_colspecs[:center][:x] = side_colspecs[:left][:width]) + side_colspecs[:center][:width]
+                acc[side] = side_colspecs
+              else
+                acc[side] = {
+                  left: { align: :left, width: side_trim_content_width, x: 0 },
+                  center: { align: :center, width: side_trim_content_width, x: 0 },
+                  right: { align: :right, width: side_trim_content_width, x: 0 },
+                }
+              end
             end
           end
 
-          content_dict = PageSides.each_with_object({}) do |side, acc|
-            side_content = {}
-            ColumnPositions.each do |position|
-              next if (val = @theme[%(#{periphery}_#{side}_#{position}_content)]).nil_or_empty?
-              val = val.to_s unless ::String === val
-              if (val.include? ':') && val =~ ImageAttributeValueRx
-                attrlist = $2
-                image_attrs = (AttributeList.new attrlist).parse %w(alt width)
-                image_path, image_format = ::Asciidoctor::Image.target_and_format $1, image_attrs
-                if (image_path = resolve_image_path doc, image_path, image_format, @themesdir) && (::File.readable? image_path)
-                  image_opts = resolve_image_options image_path, image_format, image_attrs, container_size: [colspec_dict[side][position][:width], trim_content_height[side]]
-                  side_content[position] = [image_path, image_opts, image_attrs['link']]
+          content_dict = {}.tap do |acc|
+            PageSides.each do |side|
+              side_content = {}
+              ColumnPositions.each do |position|
+                next if (val = @theme[%(#{periphery}_#{side}_#{position}_content)]).nil_or_empty?
+                val = val.to_s unless ::String === val
+                if (val.include? ':') && val =~ ImageAttributeValueRx
+                  attrlist = $2
+                  image_attrs = (AttributeList.new attrlist).parse %w(alt width)
+                  image_path, image_format = ::Asciidoctor::Image.target_and_format $1, image_attrs
+                  if (image_path = resolve_image_path doc, image_path, image_format, @themesdir) && (::File.readable? image_path)
+                    image_opts = resolve_image_options image_path, image_format, image_attrs, container_size: [colspec_dict[side][position][:width], trim_content_height[side]]
+                    side_content[position] = [image_path, image_opts, image_attrs['link']]
+                  else
+                    # NOTE: allows inline image handler to report invalid reference and replace with alt text
+                    side_content[position] = %(image:#{image_path}[#{attrlist}])
+                  end
                 else
-                  # NOTE: allows inline image handler to report invalid reference and replace with alt text
-                  side_content[position] = %(image:#{image_path}[#{attrlist}])
+                  side_content[position] = val
                 end
-              else
-                side_content[position] = val
               end
-            end
 
-            acc[side] = side_content
+              acc[side] = side_content
+            end
           end
 
           if (trim_bg_color = trim_styles[:bg_color]) || trim_bg_image || trim_border_width > 0
@@ -3866,18 +3872,17 @@ module Asciidoctor
 
       def register_fonts font_catalog, fonts_dir
         return unless font_catalog
-        dirs = (fonts_dir.split ValueSeparatorRx, -1).map do |dir|
-          dir == 'GEM_FONTS_DIR' || dir.empty? ? ThemeLoader::FontsDir : dir
-        end
+        dirs = (fonts_dir.split ValueSeparatorRx, -1).map {|dir| dir == 'GEM_FONTS_DIR' || dir.empty? ? ThemeLoader::FontsDir : dir }
         font_catalog.each do |key, styles|
-          styles = styles.each_with_object({}) do |(style, path), accum|
-            found = dirs.any? do |dir|
-              resolved_font_path = font_path path, dir
-              accum[style.to_sym] = resolved_font_path if ::File.readable? resolved_font_path
+          register_font key => ({}.tap do |accum|
+            styles.each do |style, path|
+              found = dirs.any? do |dir|
+                resolved_font_path = font_path path, dir
+                accum[style.to_sym] = resolved_font_path if ::File.readable? resolved_font_path
+              end
+              raise ::Errno::ENOENT, ((File.absolute_path? path) ? %(#{path} not found) : %(#{path} not found in #{fonts_dir.gsub ValueSeparatorRx, ' or '})) unless found
             end
-            raise ::Errno::ENOENT, ((File.absolute_path? path) ? %(#{path} not found) : %(#{path} not found in #{fonts_dir.gsub ValueSeparatorRx, ' or '})) unless found
-          end
-          register_font key => styles
+          end)
         end
       end
 
@@ -4578,14 +4583,16 @@ module Asciidoctor
       def consolidate_ranges nums
         if nums.size > 1
           prev = nil
-          nums.each_with_object [] do |num, accum|
+          accum = []
+          nums.each do |num|
             if prev && (prev.to_i + 1) == num.to_i
               accum[-1][1] = num
             else
               accum << [num]
             end
             prev = num
-          end.map {|range| range.join '-' }
+          end
+          accum.map {|range| range.join '-' }
         else
           nums
         end
@@ -4930,14 +4937,16 @@ module Asciidoctor
           result = yield
         else
           email = nil
-          original_attrs = AuthorAttributeNames.each_with_object({}) do |(prop_name, attr_name), accum|
-            accum[attr_name] = doc.attr attr_name
-            if (val = author[prop_name])
-              doc.set_attr attr_name, val
-              # NOTE: email attribute could be a url
-              email = val if prop_name == :email
-            else
-              doc.remove_attr attr_name
+          original_attrs = {}.tap do |accum|
+            AuthorAttributeNames.each do |prop_name, attr_name|
+              accum[attr_name] = doc.attr attr_name
+              if (val = author[prop_name])
+                doc.set_attr attr_name, val
+                # NOTE: email attribute could be a url
+                email = val if prop_name == :email
+              else
+                doc.remove_attr attr_name
+              end
             end
           end
           doc.set_attr 'url', ((email.include? '@') ? %(mailto:#{email}) : email) if email

data/lib/asciidoctor/pdf/ext/prawn/extensions.rb

@@ -4,7 +4,7 @@ Prawn::Font::AFM.instance_variable_set :@hide_m17n_warning, true
 
 require 'prawn/icon'
 
-Prawn::Icon::Compatibility.send :prepend, (::Module.new { def warning *_args; end })
+Prawn::Icon::Compatibility.prepend (::Module.new { def warning *_args; end })
 
 module Asciidoctor
   module Prawn
@@ -431,15 +431,19 @@ module Asciidoctor
 
       # NOTE: override built-in fill_formatted_text_box to insert leading before second line when :first_line is true
       def fill_formatted_text_box text, options
-        if (initial_gap = options[:initial_gap]) && (first_text = text[0]) && first_text[:from_page] != page_number
+        if (initial_gap = options[:initial_gap]) && !text.empty? && text[0][:from_page] != page_number
           self.y -= initial_gap
         end
         merge_text_box_positioning_options options
         box = ::Prawn::Text::Formatted::Box.new text, options
-        remaining_text = box.render
+        remaining_fragments = box.render
         @no_text_printed = box.nothing_printed?
         @all_text_printed = box.everything_printed?
-        remaining_text[0][:from_page] = page_number unless remaining_text.empty?
+        unless remaining_fragments.empty? || (remaining_fragments[0][:from_page] ||= page_number) == page_number
+          log :error, %(cannot fit formatted text on page: #{remaining_fragments.map {|it| it[:image_path] || it[:text] }.join})
+          page.tare_content_stream
+          remaining_fragments = {}
+        end
 
         if @final_gap || (options[:first_line] && !(@no_text_printed || @all_text_printed))
           self.y -= box.height + box.line_gap + box.leading
@@ -447,7 +451,7 @@ module Asciidoctor
           self.y -= box.height
         end
 
-        remaining_text
+        remaining_fragments
       end
 
       # NOTE: override built-in draw_indented_formatted_line to set first_line flag
@@ -483,7 +487,13 @@ module Asciidoctor
         else
           remaining_fragments = box.render dry_run: true
         end
-        remaining_fragments.empty? ? (remaining_fragments = nil) : (remaining_fragments[0][:from_page] = page_number)
+        if remaining_fragments.empty?
+          remaining_fragments = nil
+        elsif (remaining_fragments[0][:from_page] ||= page_number) != page_number
+          log :error, %(cannot fit formatted text on page: #{remaining_fragments.map {|it| it[:image_path] || it[:text] }.join})
+          page.tare_content_stream
+          remaining_fragments = nil
+        end
         if first_line_text_transform
           # NOTE: applying text transform here could alter the wrapping, so isolate first line and shrink it to fit
           first_line_text = (box.instance_variable_get :@printed_lines)[0]

data/lib/asciidoctor/pdf/ext/prawn/formatted_text/protect_bottom_gutter.rb

@@ -2,12 +2,15 @@
 
 module Prawn::Text::Formatted::ProtectBottomGutter
   def enough_height_for_this_line?
-    return super unless @arranger.finished?
-    begin
-      @height -= @bottom_gutter
+    if @arranger.finished? && @arranger.fragments.none? {|it| it.format_state[:full_height] }
+      begin
+        @height -= @bottom_gutter
+        super
+      ensure
+        @height += @bottom_gutter
+      end
+    else
       super
-    ensure
-      @height += @bottom_gutter
     end
   end
 end

data/lib/asciidoctor/pdf/formatted_text/inline_image_arranger.rb

@@ -38,7 +38,7 @@ module Asciidoctor::PDF::FormattedText
       return if (raw_image_fragments = fragments.select {|f| (f.key? :image_path) && !(f.key? :image_obj) }).empty?
       scratch = doc.scratch?
       available_w = available_width
-      available_h = doc.page.empty? ? doc.cursor : doc.bounds.height
+      available_h = doc.bounds.height
       last_fragment = {}
       raw_image_fragments.each do |fragment|
         if fragment[:object_id] == last_fragment[:object_id]
@@ -96,6 +96,10 @@ module Asciidoctor::PDF::FormattedText
           if (f_height = image_h) > (line_font = doc.font).height * 1.5
             # align with descender (equivalent to vertical-align: bottom in CSS)
             fragment[:ascender] = f_height - (fragment[:descender] = line_font.descender)
+            if f_height == available_h
+              fragment[:ascender] -= (doc.calc_line_metrics (doc.instance_variable_get :@base_line_height), line_font, doc.font_size).padding_top
+              fragment[:full_height] = true
+            end
             doc.font_size (fragment[:size] = f_height * (doc.font_size / line_font.height))
             # align with baseline (roughly equivalent to vertical-align: baseline in CSS)
             #fragment[:ascender] = f_height

data/lib/asciidoctor/pdf/formatted_text/source_wrap.rb

@@ -11,15 +11,15 @@ module Asciidoctor
         def wrap array
           return super unless array[0][:linenum] # sanity check
           initialize_wrap array
+          @line_wrap.extend SourceLineWrap
           highlight_line = stop = nil
           unconsumed = @arranger.unconsumed
           until stop
             if (first_fragment = unconsumed[0])[:linenum]
               linenum_text = first_fragment[:text]
-              linenum_spacer ||= { text: (NoBreakSpace.encode linenum_text.encoding) + (' ' * (linenum_text.length - 1)) }
+              linenum_spacer ||= { text: (NoBreakSpace.encode linenum_text.encoding) + (' ' * (linenum_text.length - 1)), linenum: :spacer }
               highlight_line = (second_fragment = unconsumed[1])[:highlight] ? second_fragment.dup : nil
-            else
-              # NOTE: a wrapped line
+            else # wrapped line
               first_fragment[:text] = first_fragment[:text].lstrip
               @arranger.unconsumed.unshift highlight_line if highlight_line
               @arranger.unconsumed.unshift linenum_spacer.dup
@@ -43,6 +43,12 @@ module Asciidoctor
           @arranger.unconsumed
         end
       end
+
+      module SourceLineWrap
+        def update_line_status_based_on_last_output
+          @arranger.current_format_state[:linenum] ? nil : super
+        end
+      end
     end
   end
 end

data/lib/asciidoctor/pdf/formatted_text/transform.rb

@@ -95,22 +95,24 @@ module Asciidoctor
                 styles: (to_styles theme.menu_font_style),
               }.compact,
             }
-            revise_roles = [].to_set
-            theme.each_pair.each_with_object @theme_settings do |(key, val), accum|
-              next unless (key = key.to_s).start_with? 'role_'
-              role, key = (key.slice 5, key.length).split '_', 2
-              if (prop = ThemeKeyToFragmentProperty[key])
-                (accum[role] ||= {})[prop] = val
-              #elsif key == 'font_kerning'
-              #  unless (resolved_val = val == 'none' ? false : (val == 'normal' ? true : nil)).nil?
-              #    (accum[role] ||= {})[:kerning] = resolved_val
-              #  end
-              elsif key == 'font_style' || key == 'text_decoration'
-                revise_roles << role
+            @theme_settings.tap do |accum|
+              revise_roles = [].to_set
+              theme.each_pair do |key, val|
+                next unless (key = key.to_s).start_with? 'role_'
+                role, key = (key.slice 5, key.length).split '_', 2
+                if (prop = ThemeKeyToFragmentProperty[key])
+                  (accum[role] ||= {})[prop] = val
+                #elsif key == 'font_kerning'
+                #  unless (resolved_val = val == 'none' ? false : (val == 'normal' ? true : nil)).nil?
+                #    (accum[role] ||= {})[:kerning] = resolved_val
+                #  end
+                elsif key == 'font_style' || key == 'text_decoration'
+                  revise_roles << role
+                end
+              end
+              revise_roles.each do |role|
+                (accum[role] ||= {})[:styles] = to_styles theme[%(role_#{role}_font_style)], theme[%(role_#{role}_text_decoration)]
               end
-            end
-            revise_roles.each_with_object @theme_settings do |role, accum|
-              (accum[role] ||= {})[:styles] = to_styles theme[%(role_#{role}_font_style)], theme[%(role_#{role}_text_decoration)]
             end
             @theme_settings['line-through'] = { styles: [:strikethrough].to_set } unless @theme_settings.key? 'line-through'
             @theme_settings['underline'] = { styles: [:underline].to_set } unless @theme_settings.key? 'underline'
@@ -276,8 +278,10 @@ module Asciidoctor
               when '#' # hex string (e.g., #FF0000)
                 fragment[:color] = value.length == 7 ? (value.slice 1, 6) : (value.slice 1, 3).each_char.map {|c| c * 2 }.join if HexColorRx.match? value
               when '[' # CMYK array (e.g., [50, 100, 0, 0])
-                fragment[:color] = ((((value.slice 1, value.length).chomp ']').split ', ', 4).each_with_object ::Array.new 4, 0).with_index do |(it, accum), idx|
-                  accum[idx] = (ival = it.to_i) == (fval = it.to_f) ? ival : fval
+                fragment[:color] = [0, 0, 0, 0].tap do |accum|
+                  (((value.slice 1, value.length).chomp ']').split ', ', 4).each_with_index do |it, idx|
+                    accum[idx] = (ival = it.to_i) == (fval = it.to_f) ? ival : fval
+                  end
                 end
               else # assume a 6-character hex color (internal only)
                 fragment[:color] = value

data/lib/asciidoctor/pdf/theme_loader.rb

@@ -15,9 +15,7 @@ module Asciidoctor
       BaseThemePath = ::File.join ThemesDir, 'base-theme.yml'
       BundledThemeNames = (::Dir.children ThemesDir).map {|it| it.slice 0, it.length - 10 }
       DeprecatedCategoryKeys = { 'blockquote' => 'quote', 'key' => 'kbd', 'literal' => 'codespan', 'outline_list' => 'list' }
-      DeprecatedKeys = %w(base heading heading_h1 heading_h2 heading_h3 heading_h4 heading_h5 heading_h6 title_page abstract abstract_title admonition_label sidebar_title toc_title).each_with_object({ 'table_caption_side' => 'table_caption_end' }) do |prefix, accum|
-        accum[%(#{prefix}_align)] = %(#{prefix}_text_align)
-      end
+      DeprecatedKeys = { 'table_caption_side' => 'table_caption_end' }.tap {|accum| %w(base heading heading_h1 heading_h2 heading_h3 heading_h4 heading_h5 heading_h6 title_page abstract abstract_title admonition_label sidebar_title toc_title).each {|prefix| accum[%(#{prefix}_align)] = %(#{prefix}_text_align) } }
       PaddingBottomHackKeys = %w(example_padding quote_padding sidebar_padding verse_padding)
 
       VariableRx = /\$([a-z0-9_-]+)/
@@ -159,10 +157,12 @@ module Asciidoctor
         elsif key == 'font_fallbacks'
           data[key] = ::Array === val ? val.map {|name| expand_vars name.to_s, data } : []
         elsif key.start_with? 'admonition_icon_'
-          data[key] = val.map do |(key2, val2)|
-            key2 = key2.tr '-', '_' if key2.include? '-'
-            [key2.to_sym, (key2.end_with? '_color') ? (to_color evaluate val2, data) : (evaluate val2, data)]
-          end.to_h if val
+          data[key] = {}.tap do |accum|
+            val.each do |key2, val2|
+              key2 = key2.tr '-', '_' if key2.include? '-'
+              accum[key2.to_sym] = (key2.end_with? '_color') ? (to_color evaluate val2, data) : (evaluate val2, data)
+            end
+          end if val
         elsif ::Hash === val
           if (rekey = DeprecatedCategoryKeys[key])
             logger.warn %(the #{key.tr '_', '-'} theme category is deprecated; use the #{rekey.tr '_', '-'} category instead)

data/lib/asciidoctor/pdf/version.rb

@@ -2,6 +2,6 @@
 
 module Asciidoctor
   module PDF
-    VERSION = '2.0.0'
+    VERSION = '2.0.1'
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: asciidoctor-pdf
 version: !ruby/object:Gem::Version
-  version: 2.0.0
+  version: 2.0.1
 platform: ruby
 authors:
 - Dan Allen
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2022-05-18 00:00:00.000000000 Z
+date: 2022-05-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: asciidoctor