allow role or theme to control text alignment of colist

Author: mojavelinux

CHANGELOG.adoc

@@ -38,6 +38,7 @@ Enhancements::
 * set `chapter-numeral` attribute in running content on pages in chapter if `sectnums` attribute is set (#1373)
 * set `part-numeral` attribute in running content on pages in part if `partnums` attribute is set (#1373)
 * add support for normal_italic font style (to reset font style to normal, then apply italic) (#1603)
+* honor text alignment roles (e.g., `text-left`) or `list-text-align` theme key on callout lists
 * honor text alignment roles (e.g., `text-center`) on block image (#1609)
 * honor role and inherited role on inline image (#1939, #1376)
 * disable running header and/or footer on toc pages if `noheader` and/or `nofooter` option is set on toc macro (#1378)

lib/asciidoctor/pdf/converter.rb

@@ -1112,9 +1112,15 @@ def convert_colist node
         @list_numerals << 1
         line_metrics = theme_font(:conum) { calc_line_metrics @base_line_height }
         last_item = node.items[-1]
+        item_spacing = @theme.callout_list_item_spacing || @theme.list_item_spacing
+        item_opts = { margin_bottom: item_spacing, normalize_line_height: true }
+        if (item_align = (resolve_alignment_from_role node.roles) || @theme.list_text_align&.to_sym)
+          item_opts[:align] = item_align
+        end
         node.items.each do |item|
           allocate_space_for_list_item line_metrics
-          convert_colist_item item, margin_bottom: (item == last_item ? 0 : @theme.list_item_spacing), normalize_line_height: true
+          item_opts[:margin_bottom] = 0 if item == last_item
+          convert_colist_item item, item_opts
         end
         @list_numerals.pop
         theme_margin :prose, :bottom, (next_enclosed_block node)

spec/list_spec.rb

@@ -454,7 +454,7 @@
     end
 
     it 'should allow text alignment to be set using theme', visual: true do
-      to_file = to_pdf_file <<~EOS, 'list-text-left-role.pdf', pdf_theme: { list_text_align: 'left' }
+      to_file = to_pdf_file <<~EOS, 'list-text-left-theme.pdf', pdf_theme: { list_text_align: 'left' }
       * #{lorem_ipsum '2-sentences-1-paragraph'}
       EOS
       (expect to_file).to visually_match 'list-text-left.pdf'
@@ -1983,6 +1983,27 @@
       item_text = pdf.find_unique_text 'description'
       (expect item_text[:page_number]).to be 2
     end
+
+    it 'should allow text alignment to be set using role', visual: true do
+      to_file = to_pdf_file <<~EOS, 'colist-text-align-left-role.pdf'
+      ----
+      data <1>
+      ----
+      [.text-left]
+      <1> #{lorem_ipsum '2-sentences-1-paragraph'}
+      EOS
+      (expect to_file).to visually_match 'colist-text-align-left.pdf'
+    end
+
+    it 'should allow text alignment to be set using theme', visual: true do
+      to_file = to_pdf_file <<~EOS, 'colist-text-align-left-theme.pdf', pdf_theme: { list_text_align: 'left' }
+      ----
+      data <1>
+      ----
+      <1> #{lorem_ipsum '2-sentences-1-paragraph'}
+      EOS
+      (expect to_file).to visually_match 'colist-text-align-left.pdf'
+    end
   end
 
   context 'Bibliography' do