Add optional way to strip whitespace from templates

[sorvell]

It's typically a very minor performance improvement if whitespace is stripped from templates, but it's cumbersome to write html this way. We could expose a strip-whitespace option to templates that would be processed at registration time and would strip all whitespace from templates.

[ebidel]

Could this just be the default? You can't look inside a template in the
tools and generated dom will be formatted.

On Wed, Sep 30, 2015, 4:37 PM Steve Orvell notifications@github.com wrote:

It's typically a very minor performance improvement if whitespace is
stripped from templates, but it's cumbersome to write html this way. We
could expose a strip-whitespace option to templates that would be
processed at registration time and would strip all whitespace from
templates.

—
Reply to this email directly or view it on GitHub
#2511.

[samccone]

whitespace sometimes can be really important in templates, I wonder if automatically stripping by default might break what people expect and be quite difficult to debug.

[ebidel]

is DOM based, where whitespace doesn't matter.

On Wed, Sep 30, 2015, 4:48 PM Sam Saccone notifications@github.com wrote:

whitespace sometimes can be really important in templates, I wonder if
automatically stripping by default might break what people expect and be
quite difficult to debug.

—
Reply to this email directly or view it on GitHub
#2511 (comment).

[samccone]

oh perhaps I am not correctly understanding what

whitespace is stripped from templates

means then 😉

[ebidel]

@samccone it's a blink bug where style recalc takes a large amont of time on textnodes:
https://code.google.com/p/chromium/issues/detail?id=399816

[ebidel]

This is fixed from Steve's commit.

[idanen]

The addition of this feature caused a weird bug for me when using paper-toggle-button in an AngularJS app.
When used in a container that repeats using ng-repeat the toggle-button is drawn twice.
I made a plunkr demoing the issue: plunkr
When removing the strip-whitespace attribute, it goes back to normal.
I'm not sure whether to open a new issue or just re-opening this one...