Transform trailing whitespace to recommended way

[jwerre]

Created test to deal with issue #417

The following pug:

p Aliquam pulvinar enim eu enim finibus euismod. 
	| a(href="https://example.com") Link
	|  Suspendisse suscipit nunc non mauris sodales elementum.

should reformat to:

p Aliquam pulvinar enim eu enim finibus euismod.
	|
	| a(href="https://example.com") Link
	|
	| Suspendisse suscipit nunc non mauris sodales elementum.

[Shinigami92]

@serferdinand2 and @jwerre Please check the test files and tell me if I can add any more complex example
I already found a special case for nested tags, so everything can help and safes me from releasing an additional release after the bug fix

[jwerre]

@Shinigami92 I took a look at your formatted.pug file and it looks like the formatting is incorrect. The anchor tag should not have a pipe in front of it. For example:

Instead of

p Aliquam pulvinar enim eu enim finibus euismod.
  |
  | a(href="https://example.com") Link
  |
  | Suspendisse suscipit nunc non mauris sodales elementum.

div
  p Aliquam pulvinar enim eu enim finibus euismod.
  p Aliquam pulvinar enim eu enim finibus euismod.
    |
    | a(href="https://example.com") Link
    |
    | Suspendisse suscipit nunc non mauris sodales elementum.

It should be:

p Aliquam pulvinar enim eu enim finibus euismod.
  |
  a(href="https://example.com") Link
  |
  | Suspendisse suscipit nunc non mauris sodales elementum.

div
  p Aliquam pulvinar enim eu enim finibus euismod.
  p Aliquam pulvinar enim eu enim finibus euismod.
    |
    a(href="https://example.com") Link
    |
    | Suspendisse suscipit nunc non mauris sodales elementum.

I think that was mistake I made and you just copy and pasted it.

[Shinigami92]

@jwerre good catch 👍
Luckily it didn't changed anything as if it already reaches the a-tag, the fix is already processed

[jwerre]

@Shinigami92, I Think there may be a few more outliers.

1. You may not want spaces around other inline tags
2. You do not want spaces where the lines ends in a period

p 
  | You put the em
  em pha
  | sis on the wrong syl
  em lable
  | .

As you currently have it I think the result will end up like this:

p 
  | You put the em
  |
  em pha
  |
  | sis on the wrong syl
  |
  em lable
  | 
  | .

OUTPUT: You put the em pha sis on the wrong syl lable .
The above has spaces around the em tags and before the period

EXPECTED: You put the emphasis on the wrong syllable.

[Shinigami92]

Will add that tomorrow as a test (if you not already did as you have access to your own PR 😉)
But I don't think it will fail, as the text lines are not ending with a trailing whitespace
Only if empty whitespaces are at trailing ends, the formatter will add these additional pipes

[jwerre]

Only if empty whitespaces are at trailing ends, the formatter will add these additional pipes

Ahhh... ok. I'll take your word for it.