Trim line breaks

[andreyvolokitin]

It makes sense to remove line breaks from start and end of the included tree (especially considering that line break at the end is often inserted automatically by code editor)

[
  '\n',
  {tag: 'a'},
  '\n'
]

or

[
  '\n some text \n'
]

[Scrum]

there should be no ambiguity in parsing

[andreyvolokitin]

What do you mean?

[andreyvolokitin]

Currently, I'm trying to do this in posthtml-extend and it looks pretty easy. I will make a PR there and then we'll see. The main point is that the module/include should not dictate vertical whitespace outside its own boundaries, because it is simply a module, and such formatting should be done within the main file which includes this module, i.e. this code:

<h2>Head</h2>
<include src="module.html"></include>
<h2>Head</h2>

should end up like this:

<h2>Head</h2>
module contents
<h2>Head</h2>

and not like this:

<h2>Head</h2>

module contents

<h2>Head</h2>

[Scrum]

if it's part of posthtml-include then you are right

[andreyvolokitin]

Check this out: posthtml/posthtml-extend#14

[stevenvachon]

There's a potential smell to this request. See normalize-html-whitespace.

[andreyvolokitin]

There is mainly vertical whitespace / linebreaks problem (specifically with posthtml-include, which is different to posthtml-extend). The leading/trailing line breaks for the first/last text node should be removed somehow (and not replaced by a single space as normalize-html-whitespace suggests). Because the contents of include.html as such:

...shouldn't dictate its own vertical formatting. Also trailing line break is often added automatically/unintentionally