From a4a2ded09fde70562af210c6b04a052491389fb2 Mon Sep 17 00:00:00 2001
From: M66B <M66B@users.noreply.github.com>
Date: Sat, 30 Mar 2019 17:48:04 +0100
Subject: [PATCH] Reduced white space

---
 .../java/eu/faircode/email/HtmlHelper.java    | 61 +++++++++----------
 1 file changed, 30 insertions(+), 31 deletions(-)

diff --git a/app/src/main/java/eu/faircode/email/HtmlHelper.java b/app/src/main/java/eu/faircode/email/HtmlHelper.java
index a621b9afca..75293777f9 100644
--- a/app/src/main/java/eu/faircode/email/HtmlHelper.java
+++ b/app/src/main/java/eu/faircode/email/HtmlHelper.java
@@ -120,37 +120,6 @@ public class HtmlHelper {
             for (Element quote : document.select("blockquote"))
                 quote.html("&#8230;");
 
-        // Tables
-        for (Element col : document.select("th,td")) {
-            // prevent line breaks
-            //col.select("br").tagName("span").html("&nbsp;");
-            //col.select("div").tagName("span");
-
-            // separate columns by a space
-            if (col.nextElementSibling() != null)
-                col.append("&nbsp;");
-
-            if ("th".equals(col.tagName()))
-                col.tagName("strong");
-            else
-                col.tagName("span");
-        }
-
-        for (Element row : document.select("tr"))
-            row.tagName("span").appendElement("br");
-
-        document.select("caption").tagName("p");
-        document.select("table").tagName("div");
-
-        // Lists
-        for (Element li : document.select("li")) {
-            li.tagName("span");
-            li.prependText("* ");
-            li.appendElement("br"); // line break after list item
-        }
-        document.select("ol").tagName("div");
-        document.select("ul").tagName("div");
-
         // Short quotes
         for (Element q : document.select("q")) {
             q.prependText("\"");
@@ -260,6 +229,36 @@ public class HtmlHelper {
             }
         }
 
+        // Tables
+        for (Element col : document.select("th,td")) {
+            // separate columns by a space
+            if (col.nextElementSibling() == null) {
+                if (col.selectFirst("div") == null)
+                    col.appendElement("br");
+            } else
+                col.append("&nbsp;");
+
+            if ("th".equals(col.tagName()))
+                col.tagName("strong");
+            else
+                col.tagName("span");
+        }
+
+        for (Element row : document.select("tr"))
+            row.tagName("span");
+
+        document.select("caption").tagName("p");
+        document.select("table").tagName("div");
+
+        // Lists
+        for (Element li : document.select("li")) {
+            li.tagName("span");
+            li.prependText("* ");
+            li.appendElement("br"); // line break after list item
+        }
+        document.select("ol").tagName("div");
+        document.select("ul").tagName("div");
+
         // Autolink
         NodeTraversor.traverse(new NodeVisitor() {
             @Override