Fix _replaceBrs

Previously, `nextElem` was not actually proceeding to the next element, and therefore aborting the paragraph at the first `<br>` (rather than the first `<br><br>` as the comment indicates).
pull/440/head
David A Roberts 6 years ago committed by Gijs
parent 3ff9a166fb
commit 8414158fa9

@ -457,7 +457,7 @@ Readability.prototype = {
while (next) {
// If we've hit another <br><br>, we're done adding children to this <p>.
if (next.tagName == "BR") {
var nextElem = this._nextElement(next);
var nextElem = this._nextElement(next.nextSibling);
if (nextElem && nextElem.tagName == "BR")
break;
}

@ -2,10 +2,12 @@
<div>
<p style="display: inline;" class="readability-styled"> Lorem ipsum</p>
<p style="display: inline;" class="readability-styled">dolor sit</p>
<p> amet, consectetur adipisicing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia deserunt mollit anim id est laborum. </p>
<p> amet, consectetur adipisicing elit, sed do eiusmod<br/> tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam,<br/> quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo</p>
<p> consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse<br/> cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non<br/> proident, sunt in culpa qui officia deserunt mollit anim id est laborum. </p>
</div>
<div>
<p style="display: inline;" class="readability-styled"> Tempor</p>
<p>incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia deserunt mollit anim id est laborum. </p>
<p>incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam,<br/> quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo<br/> consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse</p>
<p> cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non<br/> proident, sunt in culpa qui officia deserunt mollit anim id est laborum. </p>
</div>
</div>

@ -8,19 +8,19 @@
<article>
<h1>Lorem</h1>
<div>
Lorem ipsum<br/>dolor sit<br/> <br/><br/>amet, consectetur adipisicing elit, sed do eiusmod
tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam,
quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo
consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse
cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non
Lorem ipsum<br/>dolor sit<br/> <br/><br/>amet, consectetur adipisicing elit, sed do eiusmod<br/>
tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam,<br/>
quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo<br/> <br/>
consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse<br/>
cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non<br/>
proident, sunt in culpa qui officia deserunt mollit anim id est laborum.
</div>
<h2>Foo</h2>
<div>
Tempor<br/><br/>incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam,
quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo
consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse
cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non
Tempor<br/><br/>incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam,<br/>
quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo<br/>
consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse<br/><br/>
cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non<br/>
proident, sunt in culpa qui officia deserunt mollit anim id est laborum.
</div>
</article>

Loading…
Cancel
Save