asciidoctor-pdf 2.0.0 → 2.0.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 027a0c91049b52843552b50087dc100ecaf0143bf2d342c2d3e5a8d37bd8fcda
-  data.tar.gz: 1723e9c0ab2eab7f2f25dd31e6085e31103724c27eff2e3b9a1fe818d64161af
+  metadata.gz: b41dc215a6b942da9191850d15cf3e68224f1b633147b2df34a5e1e88756f797
+  data.tar.gz: 929dd071b2dc29b6d6f4500efb87e1ffa251c9413ae1e8076a14285bca51a600
 SHA512:
-  metadata.gz: d563c7b4def055b2732563910f0abc318d09e566423592a008535730480363a52f36f0d3bd27ed431490d3cf80bc855c8e98dc271556b8b27ea63ebd1a7b284d
-  data.tar.gz: 82834fe8af002bc265e09880d8ef11e25e6e9943c1bf6fb4359f655364a15b49e8b71134adbe0fb3db567759944915faabe1cf5720a558751c3fcb53266c1c3f
+  metadata.gz: 17d8c003b47e99e0e7156027783ff1fa1e8298767b82de209f40a523e40d6d1f35f7263c1b673bb686082a474a219c9fda26e3f2894852f51374dd554844aae2
+  data.tar.gz: 9711e5816aa9c9893ddaf139c717fb125d285b5788d42ebac4473660148f66126b1eccc1f389d781e4fc13ccb02010f9a909fa69580b7239765b907a160a4a3b

data/CHANGELOG.adoc

@@ -5,6 +5,45 @@
 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.3 (2022-05-25) - @mojavelinux
+
+Improvements::
+
+* compute the optimize settings in init_pdf and store as Hash instead of after writing the PDF file
+
+Bug Fixes::
+
+* adjust TrimBox to fit inside of BleedBox when using optimizer and compliance is PDF/X (#2203)
+* set height of resized image to available height to avoid float precision error when scaling down image to fit page (#2205)
+* prevent content on title page from overrunning the bounds of a single page and warn; restriction applies to `ink_title_page`
+
+=== Details
+
+{url-repo}/releases/tag/v2.0.3[git tag] | {url-repo}/compare/v2.0.2\...v2.0.3[full diff]
+
+== 2.0.2 (2022-05-22) - @mojavelinux
+
+Bug Fixes::
+
+* use specified column widths to avoid bugs in column width calculation when using colspans (#1368)
+* advance table to next page if rowspan in first row does not fit in space remaining on current page (#403)
+
+=== Details
+
+{url-repo}/releases/tag/v2.0.2[git tag] | {url-repo}/compare/v2.0.1\...v2.0.2[full diff]
+
+== 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 height and the bottom gutter (#2193)
+* short-circuit formatted text routine and log error if fragments in first line cannot fit on an empty page
+* break and wrap long contiguous text in source block when linenums is 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.3, 2022-05-25
 // 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.3
 // URLs:
 :url-gem: https://rubygems.org/gems/asciidoctor-pdf
 :url-project: https://github.com/asciidoctor/asciidoctor-pdf
@@ -97,14 +97,7 @@ Pass the name of the gem to the `gem install` command as follows:
 Installing Asciidoctor PDF will install a number of other gems mentioned in these docs, including asciidoctor, prawn, prawn-svg, prawn-table, prawn-icon, and ttfunk.
 For the most part, the versions of these dependencies are locked to the version of Asciidoctor PDF.
 
-If you're using Ruby 3.1 or better, you must also install the matrix gem until Prawn 2.5.0 or better is released.
-
- $ gem install matrix
-
-The matrix gem used to be bundled in the Ruby distribution, but was split out starting in Ruby 3.1.
-This requirement will be lifted once Prawn declares it as a runtime dependency.
-
-For further installation information, see {url-project-docs}/install/[the installation documentation].
+For further installation information about installing Asciidoctor PDF, see {url-project-docs}/install/[the installation documentation].
 For troubleshooting help, see {url-project-docs}/install/#installation-troubleshooting[Installation troubleshooting].
 
 === Install a prerelease or development version

data/lib/asciidoctor/pdf/converter.rb

@@ -152,7 +152,7 @@ module Asciidoctor
           # TODO: delegate to convert_method_missing
           log :warn, %(missing convert handler for #{name} node in #{@backend} backend)
         end
-        # NOTE: inline nodes generate pseudo-HTML strings; the remainder write directly to PDF object
+        # NOTE: inline node handlers generate HTML-like strings; all other handlers write directly to the PDF object
         ::Asciidoctor::Inline === node ? result : self
       end
 
@@ -174,7 +174,9 @@ module Asciidoctor
         if (has_title_page = (title_page_on = doc.doctype == 'book' || (doc.attr? 'title-page')) && (start_title_page doc))
           # NOTE: the base font must be set before any content is written to the main or scratch document
           font @theme.base_font_family, size: @root_font_size, style: @theme.base_font_style
-          ink_title_page doc
+          if perform_on_single_page { ink_title_page doc }
+            log :warn, 'the title page contents has been truncated to prevent it from overrunning the bounds of a single page'
+          end
           start_new_page
         else
           @page_margin_by_side[:cover] = @page_margin_by_side[:recto] if @media == 'prepress' && page_number == 0
@@ -321,6 +323,7 @@ module Asciidoctor
         stamp_foreground_image doc, has_front_cover
         ink_cover_page doc, :back
         add_dest_for_top doc
+        state.pages.each {|it| fit_trim_box it } if (@optimize&.[] :compliance)&.start_with? 'PDF/X'
         nil
       end
 
@@ -411,8 +414,14 @@ module Asciidoctor
         # NOTE: we have to init Pdfmark class here while we have reference to the doc
         @pdfmark = (doc.attr? 'pdfmark') ? (Pdfmark.new doc) : nil
         # NOTE: defer instantiating optimizer until we know min pdf version
-        if (@optimize = doc.attr 'optimize')
-          @optimize = nil unless (defined? ::Asciidoctor::PDF::Optimizer) || !(Helpers.require_library OptimizerRequirePath, 'rghost', :warn).nil? # rubocop:disable Style/SoleNestedConditional
+        if (optimize = doc.attr 'optimize') &&
+            ((defined? ::Asciidoctor::PDF::Optimizer) || !(Helpers.require_library OptimizerRequirePath, 'rghost', :warn).nil?)
+          @optimize = (optimize.include? ',') ?
+            ([:quality, :compliance].zip (optimize.split ',', 2)).to_h :
+            ((optimize.include? '/') ? { compliance: optimize } : { quality: optimize })
+          fit_trim_box if @optimize[:compliance]&.start_with? 'PDF/X'
+        else
+          @optimize = nil
         end
         allocate_scratch_prototype
         self
@@ -717,7 +726,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 +823,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 +1090,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')
@@ -1110,17 +1119,17 @@ module Asciidoctor
               end
               if (srclang = node.attr 'language')
                 if srclang.include? '?'
-                  if (lexer = ::Rouge::Lexer.find_fancy srclang) && lexer.tag == 'php' && !(node.option? 'mixed') && !((lexer_opts = lexer.options).key? 'start_inline')
+                  if (lexer = ::Rouge::Lexer.find_fancy srclang)&.tag == 'php' && !(node.option? 'mixed') && !((lexer_opts = lexer.options).key? 'start_inline')
                     lexer = lexer.class.new lexer_opts.merge 'start_inline' => true
                   end
-                elsif (lexer = ::Rouge::Lexer.find srclang)
-                  lexer = lexer.new start_inline: true if lexer.tag == 'php' && !(node.option? 'mixed')
+                elsif (lexer = ::Rouge::Lexer.find srclang)&.tag == 'php' && !(node.option? 'mixed')
+                  lexer = lexer.new start_inline: true
                 end
               end
               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
@@ -1205,10 +1214,10 @@ module Asciidoctor
           b_left_width = nil
           b_width = nil if (b_width = @theme[%(#{category}_border_width)]) == 0
         end
-        if (attribution = (node.attr? 'attribution') && (node.attr 'attribution'))
+        if (attribution = node.attr 'attribution')
           # NOTE: temporary workaround to allow bare & to be used without having to wrap value in single quotes
           attribution = escape_amp attribution if attribution.include? '&'
-          if (citetitle = node.attr 'citetitle') && (citetitle.include? '&')
+          if (citetitle = node.attr 'citetitle')&.include? '&'
             citetitle = escape_amp citetitle
           end
         end
@@ -1396,7 +1405,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
@@ -1760,7 +1769,7 @@ module Asciidoctor
                   advance_page
                   available_h = cursor - caption_h
                 end
-                rendered_w, rendered_h = image_info.calc_image_dimensions height: available_h if rendered_h > available_h
+                rendered_w = (image_info.calc_image_dimensions height: (rendered_h = available_h))[0] if rendered_h > available_h
               end
               add_dest_for_block node if node.id
               # NOTE: workaround to fix Prawn not adding fill and stroke commands on page that only has an image;
@@ -2167,6 +2176,7 @@ module Asciidoctor
 
         left_padding = right_padding = nil
         table table_data, table_settings do
+          @column_widths = column_widths unless column_widths.empty?
           # NOTE: call width to capture resolved table width
           table_width = width
           @pdf.ink_table_caption node, alignment, table_width, caption_max_width if node.title? && caption_end == :top
@@ -3261,64 +3271,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
@@ -3601,7 +3617,6 @@ module Asciidoctor
           end
         end
 
-        # TODO: prevent content from spilling to next page
         theme_font :title_page do
           if (title_top = @theme.title_page_title_top)
             @y = resolve_top title_top
@@ -3866,18 +3881,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
 
@@ -4397,13 +4411,8 @@ module Asciidoctor
           pdf_doc.render_file target
           # QUESTION: restore attributes first?
           @pdfmark&.generate_file target
-          if (quality = @optimize)
-            if quality.include? ','
-              quality, compliance = quality.split ',', 2
-            elsif quality.include? '/'
-              quality, compliance = nil, quality
-            end
-            (Optimizer.new quality, pdf_doc.min_version, compliance).optimize_file target
+          if (optimize = @optimize)
+            (Optimizer.new optimize[:quality], pdf_doc.min_version, optimize[:compliance]).optimize_file target
           end
           to_file = true
         end
@@ -4578,14 +4587,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
@@ -4628,6 +4639,15 @@ module Asciidoctor
         (max_height = bounds.height) < preferred_size ? max_height : preferred_size
       end
 
+      def fit_trim_box page_ = page
+        page_.dictionary.data[:TrimBox].tap do |trim_box|
+          trim_box[0] += 1e-4
+          trim_box[1] += 1e-4
+          trim_box[2] -= 1e-4
+          trim_box[3] -= 1e-4
+        end
+      end
+
       def font_path font_file, fonts_dir
         # resolve relative to built-in font dir unless path is absolute
         ::File.absolute_path font_file, fonts_dir
@@ -4930,14 +4950,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/ext/prawn-table.rb

@@ -1,6 +1,17 @@
 # frozen_string_literal: true
 
 require 'prawn/table'
+
+Prawn::Table.prepend (Module.new do
+  def initial_row_on_initial_page
+    return 0 if fits_on_page? @pdf.bounds.height
+    height_required = (row (0..number_of_header_rows)).height_with_span
+    return -1 if fits_on_page? height_required, true
+    @pdf.bounds.move_past_bottom
+    0
+  end
+end)
+
 require_relative 'prawn-table/cell'
 require_relative 'prawn-table/cell/asciidoc'
 require_relative 'prawn-table/cell/text'

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.3'
   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.3
 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-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: asciidoctor