From 7e7c6a979c472e86252259c8ea328624fb4d56f1 Mon Sep 17 00:00:00 2001
From: luisangelsm <luisangelsm@gmail.com>
Date: Sat, 28 Feb 2026 18:20:13 +0100
Subject: [PATCH] Add TODO to document bug in recolorSvgXML

---
 common/themes/icon_utils.cpp | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/common/themes/icon_utils.cpp b/common/themes/icon_utils.cpp
index f0ca5f9d..680a78a8 100644
--- a/common/themes/icon_utils.cpp
+++ b/common/themes/icon_utils.cpp
@@ -66,6 +66,9 @@ QString recolorSvgXML(QString &svg,
                       const QString &placeHolder,
                       const QColor &color)
 {
+    // TODO: svg colors can work without ;, we need to update all the svg files to inlude ;
+    // Include the trailing ';' so e.g. "#ff0;" never accidentally matches
+    // inside a previously-substituted value like "#ff0000;".
     return svg.replace(placeHolder, color.name(QColor::HexRgb), Qt::CaseInsensitive);
 }