From edaeb3ccde38809daa031c9a2a806c759551a47e Mon Sep 17 00:00:00 2001
From: Nicolas Werner <nicolas.werner@hotmail.de>
Date: Sun, 28 Mar 2021 20:41:08 +0200
Subject: [PATCH] Fix emojis being split by rainbows

---
 src/Utils.cpp | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/src/Utils.cpp b/src/Utils.cpp
index 32d435f0..50ebbeb5 100644
--- a/src/Utils.cpp
+++ b/src/Utils.cpp
@@ -517,18 +517,20 @@ utils::markdownToHtml(const QString &text, bool rainbowify)
                         QTextBoundaryFinder tbf(QTextBoundaryFinder::BoundaryType::Grapheme,
                                                 nodeText);
                         while ((boundaryEnd = tbf.toNextBoundary()) != -1) {
+                                charIdx++;
                                 // Split text to get current char
                                 auto curChar =
                                   nodeText.midRef(boundaryStart, boundaryEnd - boundaryStart);
                                 boundaryStart = boundaryEnd;
                                 // Don't rainbowify whitespaces
-                                if (curChar.trimmed().isEmpty()) {
-                                        buf.append(curChar.toString());
+                                if (curChar.trimmed().isEmpty() ||
+                                    codepointIsEmoji(curChar.toUcs4().first())) {
+                                        buf.append(curChar);
                                         continue;
                                 }
 
                                 // get correct color for char index
-                                auto color = QColor::fromHsvF(1.0 / textLen * charIdx, 1.0, 1.0);
+                                auto color = QColor::fromHsvF((charIdx - 1.0) / textLen, 1.0, 1.0);
                                 // format color for HTML
                                 auto colorString = color.name(QColor::NameFormat::HexRgb);
                                 // create HTML element for current char
@@ -537,8 +539,6 @@ utils::markdownToHtml(const QString &text, bool rainbowify)
                                                         .arg(curChar);
                                 // append colored HTML element to buffer
                                 buf.append(curCharColored);
-
-                                charIdx++;
                         }
 
                         // create HTML_INLINE node to prevent HTML from being escaped