make usage message logic in sendAnnouncementColor more explicit

Chatterino · Mar 14, 2024 · 058560f · 058560f
1 parent 17d770c
commit 058560f
Showing 1 changed file with 16 additions and 6 deletions.
diff --git a/src/controllers/commands/builtin/twitch/Announce.cpp b/src/controllers/commands/builtin/twitch/Announce.cpp
@@ -39,11 +39,20 @@ QString sendAnnouncementColor(const CommandContext &ctx,
 
     if (ctx.words.size() < 2)
     {
-        ctx.channel->addMessage(makeSystemMessage(
-            QString("Usage: /announce%1 <message> - Call attention to your "
-                    "message with a %1%2highlight.")
-                .arg(colorStr)
-                .arg(color == HelixAnnouncementColor::Primary ? "" : " ")));
+        QString usageMsg;
+        if (color == HelixAnnouncementColor::Primary)
+        {
+            usageMsg = "Usage: /announce <message> - Call attention to your "
+                       "message with a highlight.";
+        }
+        else
+        {
+            usageMsg =
+                QString("Usage: /announce%1 <message> - Call attention to your "
+                        "message with a %1 highlight.")
+                    .arg(colorStr);
+        }
+        ctx.channel->addMessage(makeSystemMessage(usageMsg));
         return "";
     }
 
@@ -92,7 +101,8 @@ QString sendAnnouncementColor(const CommandContext &ctx,
         });
     return "";
 }
-} // namespace
+
+}  // namespace
 
 namespace chatterino::commands {