<br /> preceded by a newline is lost when parsing an HTML fragment

[rgrove]

When a <br /> element is preceded by a newline in an HTML fragment, Nokogiri seems to remove it when the fragment is parsed. Here's an irb session demonstrating the issue (using Nokogiri 1.4.0 with libxml2 2.7.6):

>> require 'rubygems'
=> false
>> require 'nokogiri'
=> true
>> html = "First line\nSecond line<br />Broken line"
=> "First line\nSecond line<br />Broken line"
>> fragment = Nokogiri::HTML::DocumentFragment.parse(html)
=> #<Nokogiri::HTML::DocumentFragment:0x80c94f5c name="#document-fragment" children=[#<Nokogiri::XML::Text:0x80c94c64 "First line\nSecond lineBroken line">]>
>> fragment.to_xhtml
=> "First line\nSecond lineBroken line"
>> fragment.to_html
=> "First line\nSecond lineBroken line"

If I remove the newline, the fragment is parsed just fine:

>> html = "First line<br />Broken line"
=> "First line<br />Broken line"
>> fragment = Nokogiri::HTML::DocumentFragment.parse(html)
=> #<Nokogiri::HTML::DocumentFragment:0x80c8c118 name="#document-fragment" children=[#<Nokogiri::XML::Text:0x80c8be20 "First line">, #<Nokogiri::XML::Element:0x80c8bd80 name="br">, #<Nokogiri::XML::Text:0x80c8bc7c "Broken line">]>
>> fragment.to_xhtml
=> "First line<br />Broken line"

[wbharding]

This also applies to other HTML -- I have observed it with anchors, i.e., "One line\nTwo line\n\n<a href="http://brokenlink.com"&gt;This won't be a link after parsing</a>"

However, if I wrap the text block in a <div> and </div>, it works. I'm thinking that the newline must somehow interfere with Nokogiri's ability to discern the insides as HTML.

[flavorjones]

OK, will investigate.

[flavorjones]

fixing leading text node with newline in fragment parsing. closed by b659302.

[wbharding]

Great turnaround time, thanks a lot! Between this and the fix to the document.root.namespace exception you recently did, I'd love to see a gem bump soon so I can strip out my collection of kluge-fixes. :)

[flavorjones]

Should be bumped this weekend. Cross your fingers.