Whitespace / empty line below headlines and list items

[johnjohndoe]

Context

• Android
• Given the following markdown:

# Lorem ipsum
- First item
- Second item

Observed

• Below the headline a empty line is rendered which has the same height as the headline text.
• Below the list item a empty line is rendered which has the same height as the list item text.

Lorem ipsum

⦁ First item

⦁ Second item

Expected

• Below the headline no empty line is rendered.
• Below the list item no empty line is rendered.

Lorem ipsum
⦁ First item
⦁ Second item

Related

• Markdown doesn't render correctly on Android when a line ends with two spaces followed by a newline. #215
• Added support for processing setext headers #86
• Renderer ignores new-line characters #279

[mikepenz]

You can control the amount of padding you see with the padding argument on the Markdown composable:

padding = markdownPadding(
    listItemTop = 0.dp,
    listItemBottom = 0.dp
),

private const val MARKDOWN = """

Lorem ipsum

• First item
• Second item
  """

[johnjohndoe]

Thank you for the hint. I did not discover padding = markdownPadding() immediately.

I tried version 0.31.0-b02 in my project and set all padding to 0.dp:

padding = markdownPadding(
    block = 0.dp,
    list = 0.dp,
    listItemTop = 0.dp,
    listItemBottom = 0.dp,
    listIndent = 0.dp,
    codeBlock = PaddingValues(0.dp),
    blockQuote = PaddingValues(0.dp),
    blockQuoteText = PaddingValues(0.dp),
    blockQuoteBar = PaddingValues.Absolute(left = 0.dp, top = 0.dp, right = 0.dp, bottom = 0.dp),
)

The padding below the headline and the list items is still shown, though.

[mikepenz]

headlines have quite a bit of spacing due to the textstyle of that element. I don't think that has anything to do with the list 🤔

can you show a screenshot with the padding you mean below list items still?

[johnjohndoe]

Sure:

Same for unordered lists.

[mikepenz]

Are you sure this isn't something your theme provides? See the screenshots from current dev.
Can you attempt to reproduce it please with the sample app

[johnjohndoe]

I can confirm that in the sample app on develop (7a247ad) my example looks like this:

Once I set both listItemTop and listItemBottom to 0.dp I can reduce the spacing:

I will test develop with my app and investigate where the spacing comes from.

[mikepenz]

Are you using material 2 or material 3? it looks like line height or some text related styling is happening here 🤔

[johnjohndoe]

💡 Today, I had time to debug the issue. The padding depends on the data which is read from an XML file. Apparently, line endings in that file are encoded as \r\n (Windows line endings, Carriage Return \r + Line Feed \n) instead of \n (Linux/macOS line endings, Line Feed \n).

🧹 I am not sure if different line endings are something that you want to be sanitized by the library. I will sanitize the data at the parsing stage of my app. Please feel free to use the issue or close it as resolved. - Thank you for your help!

[mikepenz]

Sorry for the late follow up. Are we good to close this ticket?