synzen / MonitoRSS

MonitoRSS RSS bot (formerly known as Discord.RSS) with customizable feeds. https://monitorss.xyz
https://monitorss.xyz
MIT License
1.05k stars 237 forks source link

Excess newline due to formatting #343

Closed m417z closed 9 months ago

m417z commented 9 months ago

Describe the bug

An excess newline is added in some cases.

For example, feed content:

<p><strong>Before</strong>:</p>
<p><img src="https://raw.githubusercontent.com/...png" alt="Before" /></p>
<p><strong>After</strong>:</p>
<p><img src="https://raw.githubusercontent.com/...png" alt="After" /></p>

Result:

image

To Reproduce

Steps to reproduce the behavior:

  1. Have a feed with content similar to the above
  2. Observe that an unnecessary newline is added
synzen commented 9 months ago

Thanks for the report. Just to be abundantly clear on expectations, what is the exact output that you're looking for in this case?

m417z commented 9 months ago

I expect this: <p><strong>Before</strong>:</p>

To become:

Before:

And not:

Before
:

synzen commented 9 months ago

Thanks for clarifying. I believe this should now be fixed in https://github.com/synzen/MonitoRSS/commit/ffa503c5eb879607ea338b4217a273b9727e2241