HtmlCustomContainerRenderer does not honour EnableHtmlForBlock == false #222

New Issue

claunia · 2026-01-29T14:30:45Z

claunia commented

2026-01-29 14:30:45 +00:00

Originally created by @ghost on GitHub (Sep 10, 2018).

We're building our pipeline with the UseAdvancedExtensions method as part of it and using this pipeline for rendering the same piece of text as both HTML and plaintext. For the plaintext we are setting up a new HtmlRenderer as:

 var renderer = new HtmlRenderer(writer)
                {
                    EnableHtmlForBlock = false,
                    EnableHtmlForInline = false
                };

However any content that 'HtmlCustomContainerRenderer' would pick up (i.e. between nested sets of 3 or more colons) is still rendered out as HTML. Having dug into the source I can see that 'HtmlCustomContainerRenderer' does not honour the 'EnableHtmlForBlock' bool that is set on the renderer, which every other extension we use does.

Any chance of this being honoured in a future release?

Originally created by @ghost on GitHub (Sep 10, 2018). We're building our pipeline with the `UseAdvancedExtensions` method as part of it and using this pipeline for rendering the same piece of text as both HTML and plaintext. For the plaintext we are setting up a new HtmlRenderer as: ``` var renderer = new HtmlRenderer(writer) { EnableHtmlForBlock = false, EnableHtmlForInline = false }; ``` However any content that 'HtmlCustomContainerRenderer' would pick up (i.e. between nested sets of 3 or more colons) is still rendered out as HTML. Having dug into the source I can see that 'HtmlCustomContainerRenderer' does not honour the 'EnableHtmlForBlock' bool that is set on the renderer, which every other extension we use does. Any chance of this being honoured in a future release?

claunia added the enhancement PR Welcome! labels 2026-01-29 14:30:45 +00:00

claunia closed this issue

2026-01-29 14:30:46 +00:00

claunia commented

2026-01-29 14:30:47 +00:00

@xoofx commented on GitHub (Sep 10, 2018):

Don't remember the details about this, just a send a PR and I will have a look at it.

@xoofx commented on GitHub (Sep 10, 2018): Don't remember the details about this, just a send a PR and I will have a look at it.

claunia referenced this issue

2026-01-29 14:46:42 +00:00

[PR #222] [MERGED] Add tests for `ConfigureNewLineExtension`'s `ConfigureNewLine` #876

claunia referenced this issue