Medium.js icon indicating copy to clipboard operation
Medium.js copied to clipboard

Break cleaned up by default

Open xfra35 opened this issue 11 years ago • 1 comments

Hi, I've noticed that <br> tags (ctrl+Enter) were removed when using the default settings. Looking into the code, I found that the reason for this was that inside the clean() function, the shouldDelete flag was not raised on breaks, unless "br" belongs to tags.innerLevel.

Is this the expected behaviour? Shouldn't we add br in the default tags.innerLevel then?

xfra35 avatar Dec 01 '14 06:12 xfra35

You are right. On Dec 1, 2014 1:59 AM, "Florent" [email protected] wrote:

Hi, I've noticed that
tags (ctrl+Enter) were removed when using the default settings. Looking into the code, I found that the reason for this was that inside the clean() function, the shouldDelete flag was not raised on breaks, unless "br" belongs to tags.innerLevel.

Is this the expected behaviour? Shouldn't we add br in the default tags.innerLevel then?

— Reply to this email directly or view it on GitHub https://github.com/jakiestfu/Medium.js/issues/114.

robertleeplummerjr avatar Dec 01 '14 13:12 robertleeplummerjr