From 232f50512d511235ea6ee5e916637cd0bfdf9d18 Mon Sep 17 00:00:00 2001
From: danferns <dannywritescode@gmail.com>
Date: Fri, 2 Aug 2024 13:55:11 +0530
Subject: [PATCH] Attempt to replicate KeyDelegate UI in WKey

---
 src/widget/wkey.cpp | 24 +++++++++++++++++++-----
 src/widget/wkey.h   |  8 ++++++--
 2 files changed, 25 insertions(+), 7 deletions(-)

diff --git a/src/widget/wkey.cpp b/src/widget/wkey.cpp
index 3186e227534..f20d1ab80fc 100644
--- a/src/widget/wkey.cpp
+++ b/src/widget/wkey.cpp
@@ -1,18 +1,25 @@
 #include "widget/wkey.h"
 
+#include <qboxlayout.h>
+#include <qevent.h>
+
 #include "library/library_prefs.h"
 #include "moc_wkey.cpp"
 #include "skin/legacy/skincontext.h"
 #include "track/keyutils.h"
+#include "util/color/color.h"
+#include "widget/wlabel.h"
+#include "widget/wwidgetgroup.h"
 
 WKey::WKey(const QString& group, QWidget* pParent)
-        : WLabel(pParent),
+        : WWidgetGroup(pParent),
           m_dOldValue(0),
           m_keyNotation(mixxx::library::prefs::kKeyNotationConfigKey, this),
           m_engineKeyDistance(group,
                   "visual_key_distance",
                   this,
-                  ControlFlag::AllowMissingOrInvalid) {
+                  ControlFlag::AllowMissingOrInvalid),
+          m_keyLabel() {
     setValue(m_dOldValue);
     m_keyNotation.connectValueChanged(this, &WKey::keyNotationChanged);
     m_engineKeyDistance.connectValueChanged(this, &WKey::setCents);
@@ -26,7 +33,8 @@ void WKey::onConnectedControlChanged(double dParameter, double dValue) {
 }
 
 void WKey::setup(const QDomNode& node, const SkinContext& context) {
-    WLabel::setup(node, context);
+    WWidgetGroup::setup(node, context);
+    m_keyLabel.setup(node, context);
     m_displayCents = context.selectBool(node, "DisplayCents", false);
     m_displayKey = context.selectBool(node, "DisplayKey", true);
 }
@@ -52,9 +60,15 @@ void WKey::setValue(double dValue) {
             }
             keyStr.append(QString(" %1%2c").arg(sign).arg(qAbs(cents_to_display)));
         }
-        setText(keyStr);
+        m_keyLabel.setText(keyStr);
+
+        QColor keyColor = KeyUtils::keyToColor(key);
+        QString colorStr = keyColor.name();
+        QString textColor = Color::chooseContrastColor(keyColor, 140).name();
+        m_keyLabel.setStyleSheet(QString("QLabel { background-color : %1; color : %2 }")
+                                         .arg(colorStr, textColor));
     } else {
-        setText("");
+        m_keyLabel.setText("");
     }
 }
 
diff --git a/src/widget/wkey.h b/src/widget/wkey.h
index 99215b1fa9e..d8eca99066e 100644
--- a/src/widget/wkey.h
+++ b/src/widget/wkey.h
@@ -1,9 +1,12 @@
 #pragma once
 
-#include "widget/wlabel.h"
+#include <qboxlayout.h>
+
 #include "control/controlproxy.h"
+#include "widget/wlabel.h"
+#include "widget/wwidgetgroup.h"
 
-class WKey : public WLabel  {
+class WKey : public WWidgetGroup {
     Q_OBJECT
   public:
     explicit WKey(const QString& group, QWidget* pParent = nullptr);
@@ -22,4 +25,5 @@ class WKey : public WLabel  {
     bool m_displayKey;
     ControlProxy m_keyNotation;
     ControlProxy m_engineKeyDistance;
+    WLabel m_keyLabel;
 };