From 0d9d328ee7dbfeebd370d5be6a2be60f485b6075 Mon Sep 17 00:00:00 2001
From: RickyRister <ricky.rister.wang@gmail.com>
Date: Sat, 10 Jan 2026 04:01:47 -0800
Subject: [PATCH] move clearLayout into populate

---
 .../widgets/cards/additional_info/color_identity_widget.cpp   | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/cockatrice/src/interface/widgets/cards/additional_info/color_identity_widget.cpp b/cockatrice/src/interface/widgets/cards/additional_info/color_identity_widget.cpp
index 27b53c0a9..4847b1d46 100644
--- a/cockatrice/src/interface/widgets/cards/additional_info/color_identity_widget.cpp
+++ b/cockatrice/src/interface/widgets/cards/additional_info/color_identity_widget.cpp
@@ -31,6 +31,9 @@ void ColorIdentityWidget::populateManaSymbolWidgets()
     QString fullColorIdentity = "WUBRG";
     QStringList symbols = parseColorIdentity(colorIdentity); // Parse mana cost string
 
+    // clear old layout
+    QtUtils::clearLayoutRec(layout);
+
     // populate mana symbols
     if (SettingsCache::instance().getVisualDeckStorageDrawUnusedColorIdentities()) {
         for (const QString symbol : fullColorIdentity) {
@@ -47,7 +50,6 @@ void ColorIdentityWidget::populateManaSymbolWidgets()
 
 void ColorIdentityWidget::toggleUnusedVisibility()
 {
-    QtUtils::clearLayoutRec(layout);
     populateManaSymbolWidgets();
 }