From 6ecd2c7f314d0e5fe7c8ce18ab520e3a38e8d717 Mon Sep 17 00:00:00 2001
From: SabrePenguin <147069705+SabrePenguin@users.noreply.github.com>
Date: Fri, 26 Apr 2024 16:51:23 -0400
Subject: [PATCH] Update launcher/Markdown.cpp

Co-authored-by: Alexandru Ionut Tripon <alexandru.tripon97@gmail.com>
Signed-off-by: SabrePenguin <147069705+SabrePenguin@users.noreply.github.com>
---
 launcher/Markdown.cpp | 23 ++++++++++++++---------
 1 file changed, 14 insertions(+), 9 deletions(-)

diff --git a/launcher/Markdown.cpp b/launcher/Markdown.cpp
index 31e026dc7..f9081057c 100644
--- a/launcher/Markdown.cpp
+++ b/launcher/Markdown.cpp
@@ -26,18 +26,23 @@ QString markdownToHTML(const QString& markdown)
     QString htmlStr(buffer);
 
     free(buffer);
-    
-    // Insert a breakpoint between a </ul> and <img> tag as this can cause visual bugs
-    int first_pos = htmlStr.indexOf("</ul>");
-    int img_pos;
-    while (first_pos != -1) {
-        img_pos = htmlStr.indexOf("<img", first_pos);
 
-        if (img_pos - (first_pos + 5) < 3 && img_pos - (first_pos + 5) > -1)  // 5 is the size of the </ul> tag
-            htmlStr.insert(img_pos, "<br>");
+    int pos = htmlStr.indexOf("</ul>");
+    int imgPos;
+    while (pos != -1) {
+        pos = pos + 5;  // 5 is the size of the </ul> tag
+        imgPos = htmlStr.indexOf("<img", pos);
+        if (imgPos == -1)
+            break;  // no image after the tag
 
-        first_pos = htmlStr.indexOf("</ul>", first_pos + 5);
+        auto textBetween = htmlStr.mid(pos, imgPos - pos).trimmed();  // trim all white spaces
+
+        if (textBetween.isEmpty())
+            htmlStr.insert(pos, "<br>");
+
+        pos = htmlStr.indexOf("</ul>", pos);
     }
 
+
     return htmlStr;
 }
\ No newline at end of file