backport fix for #2521 do not hyphenate autolink when hyphenation is …

…enabled

CHANGELOG.adoc

@@ -7,6 +7,10 @@ For a detailed view of what has changed, refer to the {url-repo}/commits/main[co
 
 == Unreleased
 
+Improvements::
+
+* don't hyphenate autolink when hyphenation is enabled (#2521) (*@meonkeys*)
+
 Compliance::
 
 * lock version of rexml (pulled in by asciidoctor-diagram and prawn-svg) to fix incompatibility

lib/asciidoctor/pdf/text_transformer.rb

@@ -8,6 +8,7 @@ module TextTransformer
       TagFilterRx = /(<[^>]+>)|([^<]+)/
       ContiguousCharsRx = /\p{Graph}+/
       WordRx = /\p{Word}+/
+      BareClassRx = / class="bare[" ]/
       Hyphen = '-'
       SoftHyphen = ?\u00ad
       LowerAlphaChars = 'a-z'
@@ -31,7 +32,15 @@ def capitalize_words string
 
       def hyphenate_words_pcdata string, hyphenator
         if XMLMarkupRx.match? string
-          string.gsub(PCDATAFilterRx) { $2 ? (hyphenate_words $2, hyphenator) : $1 }
+          skipping = false
+          string.gsub PCDATAFilterRx do
+            if $2
+              skipping ? $2 : (hyphenate_words $2, hyphenator)
+            else
+              skipping = skipping ? $1 != '</a>' : ($1.start_with? '<a ') && (BareClassRx.match? $1)
+              $1
+            end
+          end
         else
           hyphenate_words string, hyphenator
         end

spec/hyphens_spec.rb

@@ -215,6 +215,18 @@
     (expect lines[1]).to eql 'graph'
   end
 
+  it 'should not hyphenate URL of autolink' do
+    pdf = to_pdf <<~'END', analyze: true
+    :hyphens:
+
+    https://pellentesqueullamcorperpellentesqueullamcorperconsecteturviverrascelerisque.example.com
+    END
+
+    lines = pdf.lines
+    (expect lines.size).to be > 1
+    (expect lines[0]).not_to end_with ?\u00ad
+  end
+
   it 'should show visible hyphen at locate where word is split across lines', visual: true do
     to_file = to_pdf_file <<~'EOS', 'hyphens-word-break.pdf'
     :hyphens: