Labeled list indentation in 6.8+ darkfish is inconsistent with previous versions

[flavorjones]

Labeled list indentation seems to have changed, and I'm not sure that it was an intentional change.

Previously, the list syntax [title] always had a visual line break before the value, while title:: stayed on the same line. Now, the value always starts on the same line as the title.

From the RDoc::MarkupReference docs, here's how it renders in older darkfish:

And here's how it renders with the style changes in 6.8:

As a result, markup like this:

# [Parameters]
# - +input+ (String) RELAX NG schema definition
# - +options:+ (Nokogiri::XML::ParseOptions)
#   Defaults to ParseOptions::DEFAULT_SCHEMA

which used to render well:

now renders oddly:

I'll take a look in the next few days, but wanted to capture what I'm seeing before I forget.

[st0012]

Seems to be related to #1199 too

[flavorjones]

@st0012 Yes, I hadn't seen that one, but they are related.

Here's a more complex example of the strange styling with labeled lists:

      # [Required Parameters]
      # - +input+ (String | IO): The \XML document to parse.
      #
      # [Optional Parameters]
      # - +url:+ (String) The base URL of the document.
      # - +encoding:+ (String) The name of the encoding of the document.
      # - +options:+ (Integer | ParseOptions) Options to control the parser behavior.
      #   Defaults to +ParseOptions::STRICT+.
      #
      # [Yields]
      # If present, the block will be passed a Nokogiri::XML::ParseOptions object to modify before
      # the fragment is parsed. See Nokogiri::XML::ParseOptions for more information.

[flavorjones]

#1209 is a great fix for this issue, I +1ed it there.

[st0012]

Closing as I've merged #1209