From c7625d14df4e6acfa3679cb227bdef64c98c701c Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Tor=20Arne=20Vestb=C3=B8?= Date: Mon, 2 Jun 2025 23:34:05 +0200 Subject: [PATCH] Decouple QApplication::isEffectEnabled from QColormap Change-Id: I26468cc528d2c516492db11ca4109e6be63c1b5a Reviewed-by: Volker Hilsheimer Reviewed-by: Paul Olav Tvete --- src/widgets/kernel/qapplication.cpp | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/widgets/kernel/qapplication.cpp b/src/widgets/kernel/qapplication.cpp index 0b6e0cd8f70..8a041a4ac2c 100644 --- a/src/widgets/kernel/qapplication.cpp +++ b/src/widgets/kernel/qapplication.cpp @@ -3640,7 +3640,9 @@ void QApplication::setEffectEnabled(Qt::UIEffect effect, bool enable) bool QApplication::isEffectEnabled(Qt::UIEffect effect) { CHECK_QAPP_INSTANCE(false) - return QColormap::instance().depth() >= 16 + const auto primaryScreenDepth = QGuiApplication::primaryScreen() ? + QGuiApplication::primaryScreen()->depth() : 24; + return primaryScreenDepth >= 16 && (QApplicationPrivate::enabledAnimations & QPlatformTheme::GeneralUiEffect) && (QApplicationPrivate::enabledAnimations & uiEffectToFlag(effect)); }