From 4762fcb256e2ed69886fa53e82b0055025e14030 Mon Sep 17 00:00:00 2001 From: Claudio Cambra Date: Wed, 23 Aug 2023 09:11:45 +0800 Subject: [PATCH] Move warning yellow palette generation for warn labels to anonymous namespace function Signed-off-by: Claudio Cambra --- src/gui/folderwizard.cpp | 23 +++++++++++++---------- 1 file changed, 13 insertions(+), 10 deletions(-) diff --git a/src/gui/folderwizard.cpp b/src/gui/folderwizard.cpp index d9332aca0c70c..84fa76d0489e5 100644 --- a/src/gui/folderwizard.cpp +++ b/src/gui/folderwizard.cpp @@ -45,6 +45,15 @@ namespace { constexpr QColor darkWarnYellow(63, 63, 0); constexpr QColor lightWarnYellow(255, 255, 192); + +QPalette yellowWarnWidgetPalette(const QPalette &existingPalette) +{ + const auto warnYellow = OCC::Theme::instance()->darkMode() ? darkWarnYellow : lightWarnYellow; + auto modifiedPalette = existingPalette; + modifiedPalette.setColor(QPalette::Window, warnYellow); + modifiedPalette.setColor(QPalette::Base, warnYellow); + return modifiedPalette; +} } namespace OCC { @@ -168,11 +177,8 @@ void FolderWizardLocalPath::changeEvent(QEvent *e) void FolderWizardLocalPath::changeStyle() { - const auto warnYellow = Theme::instance()->darkMode() ? darkWarnYellow : lightWarnYellow; - auto modifiedPalette = _ui.warnLabel->palette(); - modifiedPalette.setColor(QPalette::Window, warnYellow); - modifiedPalette.setColor(QPalette::Base, warnYellow); - _ui.warnLabel->setPalette(modifiedPalette); + const auto yellowWarnPalette = yellowWarnWidgetPalette(_ui.warnLabel->palette()); + _ui.warnLabel->setPalette(yellowWarnPalette); } // ================================================================================= @@ -550,11 +556,8 @@ void FolderWizardRemotePath::changeEvent(QEvent *e) void FolderWizardRemotePath::changeStyle() { - const auto warnYellow = Theme::instance()->darkMode() ? darkWarnYellow : lightWarnYellow; - auto modifiedPalette = _ui.warnLabel->palette(); - modifiedPalette.setColor(QPalette::Window, warnYellow); - modifiedPalette.setColor(QPalette::Base, warnYellow); - _ui.warnLabel->setPalette(modifiedPalette); + const auto yellowWarnPalette = yellowWarnWidgetPalette(_ui.warnLabel->palette()); + _ui.warnLabel->setPalette(yellowWarnPalette); } // ====================================================================================