Don't trim whitespace if previous element was inline

Fixes #2325

Author: jhy

CHANGES.md

@@ -16,6 +16,7 @@
 ### Bug Fixes
 * The contents of a  `script` in a `svg` foreign context should be parsed as script data, not text. [#2320](https://github.com/jhy/jsoup/issues/2320)
 * `Tag#isFormSubmittable()` was updating the Tag's options. [#2323](https://github.com/jhy/jsoup/issues/2323)
+* The HTML pretty-printer would incorrectly trim whitespace when text followed an inline element in a block element. [#2325](https://github.com/jhy/jsoup/issues/2325)
 
 ## 1.20.1 (2025-04-29)
 

src/main/java/org/jsoup/nodes/Printer.java

@@ -119,8 +119,13 @@ int textTrim(TextNode node, int options) {
             Node prev = node.previousSibling();
             Node next = node.nextSibling();
 
-            if (prev == null || !(prev instanceof TextNode) && shouldIndent(prev))
-                options |= Entities.TrimLeading;
+            // if previous is not an inline element
+            if (!(prev instanceof Element && !isBlockEl(prev))) {
+                // if there is no previous sib; or not a text node and should be indented
+                if (prev == null || !(prev instanceof TextNode) && shouldIndent(prev))
+                    options |= Entities.TrimLeading;
+            }
+
             if (next == null || !(next instanceof TextNode) && shouldIndent(next))
                 options |= Entities.TrimTrailing; // don't trim if there is a TextNode sequence
 

src/test/java/org/jsoup/nodes/PrinterTest.java

@@ -71,4 +71,11 @@ public class PrinterTest {
         assertEquals("<div>\n <div></div>\n Hello <!-- -_- -->\n  there\n</div>", body.html());
     }
 
+    @Test void spaceAfterSpanInBlock() {
+        Document doc = Jsoup.parse("<div> <span>Span</span> \n Text  <span>Follow</span></div> <p> <span>Span</span>  Text <span>Follow</span> </p>");
+        Element body = doc.body();
+        assertEquals("Span Text Follow Span Text Follow", body.text());
+        assertEquals("<div>\n <span>Span</span> Text <span>Follow</span>\n</div>\n<p><span>Span</span> Text <span>Follow</span></p>", body.html());
+    }
+
 }

src/test/java/org/jsoup/parser/XmlTreeBuilderTest.java

@@ -495,7 +495,7 @@ private static void assertXmlNamespace(Element el) {
         doc.outputSettings().prettyPrint(true);
         assertEquals("<package>\n" +
             " <metadata xmlns:dc=\"http://purl.org/dc/elements/1.1/\">\n" +
-            "  <dc:identifier id=\"pub-id\">id</dc:identifier><dc:title>title</dc:title> <dc:language>ja</dc:language> <dc:description>desc</dc:description>\n" +
+            "  <dc:identifier id=\"pub-id\">id</dc:identifier> <dc:title>title</dc:title> <dc:language>ja</dc:language> <dc:description>desc</dc:description>\n" +
             " </metadata>\n" +
             "</package>", doc.html());