prettyPrint and outline surround inline-tags with white space (where they shouldn't)

[EdyHinzen]

Using this code with jsoup 1.11.3:

String html = "<div><span>1:15</span>&ndash;<span>2:15</span>&nbsp;p.m.</div>";
org.jsoup.nodes.Document document = Jsoup.parse(html);
document.outputSettings().prettyPrint(false).outline(false);
System.out.println("Example 1:");
System.out.println(document.body().html());
document.outputSettings().prettyPrint(true).outline(false);
System.out.println("Example 2:");
System.out.println(document.body().html());
document.outputSettings().prettyPrint(true).outline(true);
System.out.println("Example 3:");
System.out.println(document.body().html());

Results in

Example 1:
<div><span>1:15</span>–<span>2:15</span>&nbsp;p.m.</div>
Example 2:
<div>
 <span>1:15</span>–
 <span>2:15</span>&nbsp;p.m.
</div>
Example 3:
<div>
 <span>1:15</span>
 –
 <span>2:15</span>
 &nbsp;p.m.
</div>

Example 1: Please watch that there are no white spaces around the en-dash. That's correct.
Example 2: Additional white space after the en-dash.
Example 3: Additional white space before and after the en-dash.

Examples 2 and 3 result in incorrect typography and I hereby consider that as a bug.

It is likely that this problem is similar to issue #851

[kovacstamasx]

Hi Edy,

I've opened a PR to fix your issue.
It would print inline tags in the same line when using pretty print without the outline option.

Though I'm not sure that your Example 3 is a bug.
When you use outline(true) all tags and nodes should be handled as blocks.
And line break is needed when you pretty print blocks.

[jhy]

Thanks! I have merged the PR (and tweaked with 279fbe1) to fix this.