Space incorrectly removed

[paulyoung]

The space before "team" is incorrectly removed in the example below.

Other trailing spaces are preserved, so it may be an issue with leading spaces.

---

HTML input

<p>Peer-to-peer full-stack visual programming for your fingers.<br/>Funded by <a href="http://www.kickstarter.com/projects/noflo/noflo-development-environment">Kickstarter</a>, from the <a href="http://noflojs.org/">NoFlo</a> team.</p>

---

GOM IR

{
  "tag": "p",
  "children": [
    "Peer-to-peer full-stack visual programming for your fingers.",
    {
      "tag": "br"
    },
    "Funded by ",
    {
      "tag": "a",
      "attributes": {
        "href": "http://www.kickstarter.com/projects/noflo/noflo-development-environment"
      },
      "children": [
        "Kickstarter"
      ]
    },
    ", from the ",
    {
      "tag": "a",
      "attributes": {
        "href": "http://noflojs.org/"
      },
      "children": [
        "NoFlo"
      ]
    },
    "team."
  ]
}

---

HTML output

<p>Peer-to-peer full-stack visual programming for your fingers.<br/>Funded by <a>Kickstarter</a>, from the <a>NoFlo</a>team.</p>

[d4tocchini]

yeah, space handling is not straight forward... It's an easy fix but makes testing annoying. I'll give it a crack later this evening

[d4tocchini]

it seems even some of the minifiers don't have it figured out fully... see the work around mentioned here:

http://perfectionkills.com/experimenting-with-html-minifier/#collapse_whitespace

"to never fully remove white space characters, and instead always collapse them to one white space character. HTML 4.01 is actually specified to do just that, so there’s no harm in doing it upfront."

But, that's mentioned as a workaround that isn't even implemented... Not to mention it breaks if the style relies on white-space:pre; ugh...

Best option may be to keep all whitespace by default (which will make some tests very ugly), then add an API for removing / collapsing whitespace intelligently & only when wanted