Implement stripping

[mmun]

The parser now respects the handlebars whitespace control flags. Whitespace is removed from the AST at parse time. Any unnecessary text nodes are removed. This works nicely with single element programs. Instead of writing

{{#each}}<li>foo</li>{{/each}}

you can write

{{#each~}}
  <li>foo</li>
{{~/each}}

and still generate an optimized fragment

function build(dom) {
  var el0 = dom.createElement('li');
  dom.appendText(el0, 'foo');
  return el0;
}

[stefanpenner]

sometimes whitespace is important?

[mmun]

@stefanpenner It only strips whitespace if you have added the ~ flags on the ends of the mustaches. If you haven't seen it before it's documented at the bottom of http://handlebarsjs.com/block_helpers.html.

[mmun]

Oops, the PR text was a bit confusing. I clarified it.

[stefanpenner]

@mmun aaaa

[oligriffiths]

Hey, I see this works for {{#each~}}, however it doesn't seem to work for an if statement:

works as expected:

{{#each titleParts as |part|~}}
    {{part.value}}
{{~/each}}

spaces appear:

{{#each titleParts as |part|~}}
    {{#if part.highlight}}
        <span class="highlight">
            {{part.value}}
        </span>
    {{else}}
        {{part.value}}
    {{/if}}
{{~/each}}

Any advice?

Thanks

[Cheralyn]

This is an automatically generated message.

coferc@yahoo-inc.com is no longer with Yahoo! Inc.

Your message will not be forwarded.

If you have a sales inquiry, please email yahoosales@yahoo-inc.com and someone will follow up with you shortly.

If you require assistance with a legal matter, please send a message to legal-notices@yahoo-inc.com

Thank you!

[oligriffiths]

It appears that

{{#each titleParts as |part|~}}
    {{~#if part.highlight~}}
        <span class="highlight">{{part.value}}</span>
    {{~else~}}
        <span class="no-highlight">{{part.value}}</span>
    {{/if}}
{{~/each}}

does the trick