From 403e959d3f098ee14ec40d692496e3b506fbaa46 Mon Sep 17 00:00:00 2001
From: Nav <nav@navnav.me>
Date: Sat, 23 Sep 2023 01:17:06 +0100
Subject: [PATCH] Style tweaks on old list views

---
 .../MemoryRegionManager/RegionItem.cpp                      | 5 +++--
 .../Stylesheets/MemoryRegionManagerWindow.qss               | 6 ++++++
 .../TargetRegisterInspector/RegisterHistoryWidget/Item.cpp  | 5 +++--
 .../Stylesheets/TargetRegisterInspectorWindow.qss           | 5 +++++
 4 files changed, 17 insertions(+), 4 deletions(-)

diff --git a/src/Insight/UserInterfaces/InsightWindow/Widgets/TargetMemoryInspectionPane/MemoryRegionManager/RegionItem.cpp b/src/Insight/UserInterfaces/InsightWindow/Widgets/TargetMemoryInspectionPane/MemoryRegionManager/RegionItem.cpp
index 76bb5bf2..b3054766 100644
--- a/src/Insight/UserInterfaces/InsightWindow/Widgets/TargetMemoryInspectionPane/MemoryRegionManager/RegionItem.cpp
+++ b/src/Insight/UserInterfaces/InsightWindow/Widgets/TargetMemoryInspectionPane/MemoryRegionManager/RegionItem.cpp
@@ -67,8 +67,9 @@ namespace Widgets
 
     void RegionItem::setSelected(bool selected) {
         this->setProperty("selected", selected);
-        this->style()->unpolish(this);
-        this->style()->polish(this);
+
+        // TODO: This is a horrible hack. It will be binned when I rewrite this widget to use ListView
+        this->setStyleSheet(this->styleSheet());
 
         if (selected) {
             emit this->selected(this);
diff --git a/src/Insight/UserInterfaces/InsightWindow/Widgets/TargetMemoryInspectionPane/MemoryRegionManager/Stylesheets/MemoryRegionManagerWindow.qss b/src/Insight/UserInterfaces/InsightWindow/Widgets/TargetMemoryInspectionPane/MemoryRegionManager/Stylesheets/MemoryRegionManagerWindow.qss
index 83f550a5..e59c26bd 100644
--- a/src/Insight/UserInterfaces/InsightWindow/Widgets/TargetMemoryInspectionPane/MemoryRegionManager/Stylesheets/MemoryRegionManagerWindow.qss
+++ b/src/Insight/UserInterfaces/InsightWindow/Widgets/TargetMemoryInspectionPane/MemoryRegionManager/Stylesheets/MemoryRegionManagerWindow.qss
@@ -85,6 +85,12 @@
     color: #8a8a8d;
 }
 
+#region-selector #region-item[selected=true] #address-label,
+#region-selector #region-item[selected=true] #type-label,
+#region-selector #region-item[selected=true] #time-label {
+    color: #afb1b3;
+}
+
 #region-selector #region-item #type-label,
 #region-selector #region-item #time-label {
     font-size: 13px;
diff --git a/src/Insight/UserInterfaces/InsightWindow/Widgets/TargetRegisterInspector/RegisterHistoryWidget/Item.cpp b/src/Insight/UserInterfaces/InsightWindow/Widgets/TargetRegisterInspector/RegisterHistoryWidget/Item.cpp
index 749f558d..8d921c42 100644
--- a/src/Insight/UserInterfaces/InsightWindow/Widgets/TargetRegisterInspector/RegisterHistoryWidget/Item.cpp
+++ b/src/Insight/UserInterfaces/InsightWindow/Widgets/TargetRegisterInspector/RegisterHistoryWidget/Item.cpp
@@ -18,8 +18,9 @@ namespace Widgets
 
     void Item::setSelected(bool selected) {
         this->setProperty("selected", selected);
-        this->style()->unpolish(this);
-        this->style()->polish(this);
+
+        // TODO: This is a horrible hack. It will be binned when I rewrite this widget to use ListView
+        this->setStyleSheet(this->styleSheet());
 
         if (selected) {
             emit this->selected(this);
diff --git a/src/Insight/UserInterfaces/InsightWindow/Widgets/TargetRegisterInspector/Stylesheets/TargetRegisterInspectorWindow.qss b/src/Insight/UserInterfaces/InsightWindow/Widgets/TargetRegisterInspector/Stylesheets/TargetRegisterInspectorWindow.qss
index 547f4ce1..7a8e5c43 100644
--- a/src/Insight/UserInterfaces/InsightWindow/Widgets/TargetRegisterInspector/Stylesheets/TargetRegisterInspectorWindow.qss
+++ b/src/Insight/UserInterfaces/InsightWindow/Widgets/TargetRegisterInspector/Stylesheets/TargetRegisterInspectorWindow.qss
@@ -102,6 +102,11 @@
     color: #8a8a8d;
 }
 
+#target-register-history-widget #register-history-item[selected=true] #value-label,
+#target-register-history-widget #register-history-item[selected=true] #description-label {
+    color: #afb1b3;
+}
+
 #target-register-history-widget #separator-label {
     /*font-style: italic;*/
     color: #8a8a8d;