Markdownify Not Closing A Paragraph

[MarkRosemaker]

What version of Hugo are you using (hugo version)?

$ hugo version
hugo v0.120.4-f11bca5fec2ebb3a02727fb2a5cfb08da96fd9df+extended darwin/arm64 BuildDate=2023-11-08T11:18:07Z VendorInfo=brew

Does this issue reproduce with the latest release?

Yes

Description

When I use markdownify on a title and paragraph, the resulting HTML leaves the paragraph unclosed.

So using .Page.RenderString or markdownify on "# My Title\nparagraph" results in:

<h1 id=\"my-title\">My Title</h1>\n<p>paragraph

Minimal example: https://github.com/MarkRosemaker/hugo-bug-markdownify

Draft PR with failing test: #11699

[jmooring]

Workaround... the RenderString method on Page takes an options map, so this:

{{ $s := `
## Title

Text
`}}
{{ $.Page.RenderString (dict "display" "block") $s }}

Is rendered to:

<h2 id="title">Title</h2>
<p>Text</p>

The markdownify function is effectively aliased to .Site.Home.RenderString, but without the ability to specify inline vs. block. Another reason to use RenderString instead of markdownify is #9692.

This is the bit that does the p removal:
https://github.com/gohugoio/hugo/blob/master/helpers/content.go#L253-L267

[jmooring]

Yeah, we should fix this. This works fine:

{{ $s := `
## Title

My list:

- item one
- item two
`}}
{{ .RenderString $s }}

So this should too:

{{ $s := `
## Title

Text
`}}
{{ .RenderString $s }}

---

It also breaks with markdown attributes.

{{ $s := `
foo
{#bar}
`}}
{{ .RenderString $s }}

Rendered to:

<p id="bar">foo

Since the user specified an attribute, they must want a paragraph. So it should be:

<p id="bar">foo</p>

[github-actions]

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.