Clean up bogus `BR` elements about scribe HOT 6 OPEN

I think this example is related, the following lines:

Test line 1

Test line 2

Become:

<p>Test line 1</p><p><br></p><p>Test line 2</p>

from scribe.

@hassankhan Can you elaborate? I don’t see this happening when I paste (from a plain text editor).

from scribe.

I typed it in (don't know if that makes a difference?) on the demo page and that was the generated markup.

Shouldn't the correct markup be:

<p>Test line 1</p><br><p>Test line 2</p>

from scribe.

@hassankhan The sample text you included has two new line characters in it — by this I’m assuming you’re sending the ENTER key twice. If this is the case, your output HTML is correct. We create a new paragraph (P element) each time the ENTER key is sent. The correct HTML in this case:

<p>Test line 1</p><p><br></p><p>Test line 2</p>

Alternatively you can send the SHIFT+ENTER keys which will create a BR element instead of a P.

<p>Test line 1<br>Test line 2</p>

Does this help?

from scribe.

Ahh right, thanks, that was exactly what I was looking for. Sorry to clog up the issue, I read the title and thought it was related.

from scribe.

Is there any progress update on this issue? We are finding the bogus <br> very annoying in our project.

if (! this.allowsBlockElements()) {
    // Set bogus BR element for Firefox — see explanation in BR mode files.
    content = content + '<br>';
}

from scribe.