Preserve inline styling inside tidy link labels

Author: st0012

lib/rdoc/markup/to_html.rb

@@ -158,19 +158,16 @@ def handle_regexp_RDOCLINK(target)
   def handle_regexp_TIDYLINK(target)
     text = target.text
 
-    return text unless
-      text =~ /^\{(.*)\}\[(.*?)\]$/ or text =~ /^(\S+)\[(.*?)\]$/
-
-    label = $1
-    url   = CGI.escapeHTML($2)
+    if tidy_link_capturing?
+      return finish_tidy_link(text)
+    end
 
-    if /^rdoc-image:/ =~ label
-      label = handle_RDOCLINK(label)
-    else
-      label = CGI.escapeHTML(label)
+    if text.start_with?('{') && !text.include?('}')
+      start_tidy_link text
+      return ''
     end
 
-    gen_url url, label
+    convert_complete_tidy_link(text)
   end
 
   # :section: Visitor
@@ -359,7 +356,14 @@ def accept_table(header, body, aligns)
   # CGI-escapes +text+
 
   def convert_string(text)
-    CGI.escapeHTML text
+    html = CGI.escapeHTML text
+
+    if tidy_link_capturing?
+      append_to_tidy_label html
+      ''
+    else
+      html
+    end
   end
 
   ##
@@ -458,4 +462,104 @@ def to_html(item)
     super convert_flow @am.flow item
   end
 
+  private
+
+  def append_to_tidy_label(fragment)
+    return unless @tidy_link_buffer
+
+    @tidy_link_buffer << fragment
+  end
+
+  def convert_complete_tidy_link(text)
+    return text unless
+      text =~ /^\{(.*)\}\[(.*?)\]$/ or text =~ /^(\S+)\[(.*?)\]$/
+
+    label = $1
+    url   = CGI.escapeHTML($2)
+
+    label_html = if /^rdoc-image:/ =~ label
+                   handle_RDOCLINK(label)
+                 else
+                   render_tidy_link_label(label)
+                 end
+
+    gen_url url, label_html
+  end
+
+  def emit_tidy_link_fragment(res, fragment)
+    if tidy_link_capturing?
+      append_to_tidy_label fragment
+    else
+      res << fragment
+    end
+  end
+
+  def finish_tidy_link(text)
+    label_tail, url, trailing = extract_tidy_link_parts(text)
+
+    append_to_tidy_label CGI.escapeHTML(label_tail) unless label_tail.empty?
+
+    return '' unless url
+
+    label_html = @tidy_link_buffer
+
+    @tidy_link_buffer = nil
+
+    link = gen_url(url, label_html)
+
+    return link if trailing.empty?
+
+    link + CGI.escapeHTML(trailing)
+  end
+
+  def extract_tidy_link_parts(text)
+    if text =~ /^(.*?)\}\[(.*?)\](.*)$/
+      [$1, CGI.escapeHTML($2), $3]
+    else
+      [text, nil, '']
+    end
+  end
+
+  def on_tags(res, item)
+    each_attr_tag(item.turn_on) do |tag|
+      emit_tidy_link_fragment(res, annotate(tag.on))
+      @in_tt += 1 if tt? tag
+    end
+  end
+
+  def off_tags(res, item)
+    each_attr_tag(item.turn_off, true) do |tag|
+      emit_tidy_link_fragment(res, annotate(tag.off))
+      @in_tt -= 1 if tt? tag
+    end
+  end
+
+  def start_tidy_link(text)
+    @tidy_link_buffer = String.new
+    append_to_tidy_label CGI.escapeHTML(text.delete_prefix('{'))
+  end
+
+  def tidy_link_capturing?
+    !!@tidy_link_buffer
+  end
+
+  def render_tidy_link_label(label)
+    RDoc::Markup::LinkLabelToHtml.render(label, @options, @from_path)
+  end
+end
+
+##
+# Formatter dedicated to rendering tidy link labels without mutating the
+# calling formatter's state.
+
+class RDoc::Markup::LinkLabelToHtml < RDoc::Markup::ToHtml
+  def self.render(label, options, from_path)
+    new(options, from_path).to_html(label)
+  end
+
+  def initialize(options, from_path = nil)
+    super(options)
+
+    self.from_path = from_path if from_path
+  end
 end

test/rdoc/rdoc_markdown_test.rb

@@ -1263,6 +1263,23 @@ def test_gfm_table_with_backslashes_in_code_spans
     assert_equal expected, doc
   end
 
+  def test_markdown_link_with_styled_label
+    markdown = <<~MD
+    [Link to Foo](https://example.com)
+
+    [Link to `Foo`](https://example.com)
+
+    [Link to **Foo**](https://example.com)
+    MD
+
+    doc = parse markdown
+    html = @to_html.convert doc
+
+    assert_includes html, '<a href="https://example.com">Link to Foo</a>'
+    assert_includes html, '<a href="https://example.com">Link to <code>Foo</code></a>'
+    assert_includes html, '<a href="https://example.com">Link to <strong>Foo</strong></a>'
+  end
+
   def parse(text)
     @parser.parse text
   end

test/rdoc/rdoc_markup_to_html_test.rb

@@ -734,6 +734,22 @@ def test_convert_TIDYLINK_multiple
     assert_equal expected, result
   end
 
+  def test_convert_TIDYLINK_with_code_label
+    result = @to.convert '{Link to +Foo+}[https://example.com]'
+
+    expected = "\n<p><a href=\"https://example.com\">Link to <code>Foo</code></a></p>\n"
+
+    assert_equal expected, result
+  end
+
+  def test_convert_TIDYLINK_with_bold_label
+    result = @to.convert '{Link to *Foo*}[https://example.com]'
+
+    expected = "\n<p><a href=\"https://example.com\">Link to <strong>Foo</strong></a></p>\n"
+
+    assert_equal expected, result
+  end
+
   def test_convert_TIDYLINK_image
     result =
       @to.convert '{rdoc-image:path/to/image.jpg}[http://example.com]'