From 30530b6120c9441ba8a6d85d78650a6ad5a3acf4 Mon Sep 17 00:00:00 2001
From: Rasmus Karlsson <rasmus.karlsson@pajlada.com>
Date: Tue, 2 May 2023 20:52:00 +0200
Subject: [PATCH] Revert "Dim disabled items in context menus (#4423)"

This reverts commit 642718474c19479208482e08ad61dbf0a42089d2.
---
 CHANGELOG.md                           |  1 -
 resources/qss/settings.qss             | 11 -----------
 src/singletons/Theme.cpp               |  8 --------
 src/singletons/Theme.hpp               |  2 --
 src/widgets/BaseWindow.cpp             |  5 -----
 src/widgets/BaseWindow.hpp             | 17 ++++++++---------
 src/widgets/dialogs/SettingsDialog.cpp |  7 +++----
 7 files changed, 11 insertions(+), 40 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index bb0b11303a1..875ef0bc7b3 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -33,7 +33,6 @@
 - Bugfix: Fixed emote & badge tooltips not showing up when thumbnails were hidden. (#4509)
 - Bugfix: Fixed links with invalid IPv4 addresses being parsed. (#4576)
 - Bugfix: Fixed the macOS icon changing to the wrong icon when the application is open. (#4577)
-- Bugfix: Fixed disabled items in context-menus having a weird text-effect or the default text color. (#4423)
 - Dev: Disabling precompiled headers on Windows is now tested in CI. (#4472)
 - Dev: Themes are now stored as JSON files in `resources/themes`. (#4471, #4533)
 - Dev: Ignore unhandled BTTV user-events. (#4438)
diff --git a/resources/qss/settings.qss b/resources/qss/settings.qss
index a191b146d25..6d5114423b0 100644
--- a/resources/qss/settings.qss
+++ b/resources/qss/settings.qss
@@ -66,14 +66,3 @@ chatterino--NavigationLabel {
     font-size: 15px;
     color: #A6DDF4;
 }
-
-QMenu {
-    background: #242424;
-    border: #555555;
-    color: #ffffff;
-    selection-background-color: #555555;
-}
-
-QMenu::item:disabled {
-    color: #8c7f7f;
-}
diff --git a/src/singletons/Theme.cpp b/src/singletons/Theme.cpp
index 1eec47d1646..1b4f40cd2c4 100644
--- a/src/singletons/Theme.cpp
+++ b/src/singletons/Theme.cpp
@@ -212,14 +212,6 @@ void Theme::parseFrom(const QJsonObject &root)
         (this->isLightTheme() ? "#68B1FF"
                               : this->tabs.selected.backgrounds.regular.name());
 
-    this->window.contextMenuStyleSheet =
-        QStringLiteral("QMenu { background: %1; border: %2; color: %3; "
-                       "selection-background-color: %2; } "
-                       "QMenu::item:disabled { color: #8c7f7f; }")
-            .arg(splits.input.background.name(QColor::HexArgb),
-                 tabs.selected.backgrounds.regular.name(QColor::HexArgb),
-                 tabs.selected.text.name(QColor::HexArgb));
-
     // Usercard buttons
     if (this->isLightTheme())
     {
diff --git a/src/singletons/Theme.hpp b/src/singletons/Theme.hpp
index e6345f65bde..034bb799e54 100644
--- a/src/singletons/Theme.hpp
+++ b/src/singletons/Theme.hpp
@@ -39,8 +39,6 @@ class Theme final : public Singleton
     struct {
         QColor background;
         QColor text;
-
-        QString contextMenuStyleSheet;
     } window;
 
     /// TABS
diff --git a/src/widgets/BaseWindow.cpp b/src/widgets/BaseWindow.cpp
index 3da6f9aab32..60365081e67 100644
--- a/src/widgets/BaseWindow.cpp
+++ b/src/widgets/BaseWindow.cpp
@@ -292,11 +292,6 @@ bool BaseWindow::supportsCustomWindowFrame()
 
 void BaseWindow::themeChangedEvent()
 {
-    if (!this->flags_.has(BaseWindow::DisableStyleSheet))
-    {
-        this->setStyleSheet(this->theme->window.contextMenuStyleSheet);
-    }
-
     if (this->hasCustomWindowFrame())
     {
         QPalette palette;
diff --git a/src/widgets/BaseWindow.hpp b/src/widgets/BaseWindow.hpp
index c231f26a902..106cc1a3e5d 100644
--- a/src/widgets/BaseWindow.hpp
+++ b/src/widgets/BaseWindow.hpp
@@ -24,17 +24,16 @@ class BaseWindow : public BaseWidget
     Q_OBJECT
 
 public:
-    enum Flags : uint32_t {
+    enum Flags {
         None = 0,
         EnableCustomFrame = 1,
-        Frameless = (1 << 1),
-        TopMost = (1 << 2),
-        DisableCustomScaling = (1 << 3),
-        FramelessDraggable = (1 << 4),
-        DontFocus = (1 << 5),
-        Dialog = (1 << 6),
-        DisableLayoutSave = (1 << 7),
-        DisableStyleSheet = (1 << 8),
+        Frameless = 2,
+        TopMost = 4,
+        DisableCustomScaling = 8,
+        FramelessDraggable = 16,
+        DontFocus = 32,
+        Dialog = 64,
+        DisableLayoutSave = 128,
     };
 
     enum ActionOnFocusLoss { Nothing, Delete, Close, Hide };
diff --git a/src/widgets/dialogs/SettingsDialog.cpp b/src/widgets/dialogs/SettingsDialog.cpp
index aabd8bc5944..5bd218860af 100644
--- a/src/widgets/dialogs/SettingsDialog.cpp
+++ b/src/widgets/dialogs/SettingsDialog.cpp
@@ -29,10 +29,9 @@
 namespace chatterino {
 
 SettingsDialog::SettingsDialog(QWidget *parent)
-    : BaseWindow(
-          {BaseWindow::Flags::DisableCustomScaling, BaseWindow::Flags::Dialog,
-           BaseWindow::DisableLayoutSave, BaseWindow::DisableStyleSheet},
-          parent)
+    : BaseWindow({BaseWindow::Flags::DisableCustomScaling,
+                  BaseWindow::Flags::Dialog, BaseWindow::DisableLayoutSave},
+                 parent)
 {
     this->setObjectName("SettingsDialog");
     this->setWindowTitle("Chatterino Settings");