From 6bd0dca983d721e8c4f5b1afdda170e2d3912339 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Luk=C3=A1=C5=A1=20Tinkl?= Date: Wed, 18 Jan 2023 10:32:34 +0100 Subject: [PATCH] fix: Highlight color of jump-to-message has too little contrast Use the colors defined by design: ``` Light theme: Light Desktop / Blue 20% (4360DF 20%) Dark theme: Dark Desktop / Blue 20% (869EFF 20%) ``` Fixes #8271 --- ui/StatusQ/src/StatusQ/Components/StatusMessage.qml | 6 +++--- ui/StatusQ/src/StatusQ/Core/Theme/StatusDarkTheme.qml | 2 ++ ui/StatusQ/src/StatusQ/Core/Theme/StatusLightTheme.qml | 2 ++ ui/StatusQ/src/StatusQ/Core/Theme/ThemePalette.qml | 2 ++ 4 files changed, 9 insertions(+), 3 deletions(-) diff --git a/ui/StatusQ/src/StatusQ/Components/StatusMessage.qml b/ui/StatusQ/src/StatusQ/Components/StatusMessage.qml index a8406939e6..d0d6718644 100644 --- a/ui/StatusQ/src/StatusQ/Components/StatusMessage.qml +++ b/ui/StatusQ/src/StatusQ/Components/StatusMessage.qml @@ -136,8 +136,8 @@ Control { } width: 2 visible: root.isPinned || root.hasMention - color: root.isPinned ? Theme.palette.pinColor1 : root.hasMention ? Theme.palette.mentionColor1 - : "transparent" // not visible really + color: root.hasMention ? Theme.palette.mentionColor1 : root.isPinned ? Theme.palette.pinColor1 + : "transparent" // not visible really } } @@ -175,7 +175,7 @@ Control { anchors.fill: parent opacity: 0 visible: opacity > 0.001 - color: Theme.palette.baseColor2 + color: Theme.palette.messageHighlightColor } MouseArea { diff --git a/ui/StatusQ/src/StatusQ/Core/Theme/StatusDarkTheme.qml b/ui/StatusQ/src/StatusQ/Core/Theme/StatusDarkTheme.qml index e3f7483cf4..9582ffb411 100644 --- a/ui/StatusQ/src/StatusQ/Core/Theme/StatusDarkTheme.qml +++ b/ui/StatusQ/src/StatusQ/Core/Theme/StatusDarkTheme.qml @@ -65,6 +65,8 @@ ThemePalette { statusLoadingHighlight: getColor('white', 0.03) statusLoadingHighlight2: getColor('white', 0.07) + messageHighlightColor: getColor('blue4', 0.2) + userCustomizationColors: [ "#AAC6FF", "#887AF9", diff --git a/ui/StatusQ/src/StatusQ/Core/Theme/StatusLightTheme.qml b/ui/StatusQ/src/StatusQ/Core/Theme/StatusLightTheme.qml index 899a145593..04017b370d 100644 --- a/ui/StatusQ/src/StatusQ/Core/Theme/StatusLightTheme.qml +++ b/ui/StatusQ/src/StatusQ/Core/Theme/StatusLightTheme.qml @@ -63,6 +63,8 @@ ThemePalette { statusLoadingHighlight: getColor('lightPattensBlue', 0.5) statusLoadingHighlight2: indirectColor3 + messageHighlightColor: getColor('blue', 0.2) + userCustomizationColors: [ "#2946C4", "#887AF9", diff --git a/ui/StatusQ/src/StatusQ/Core/Theme/ThemePalette.qml b/ui/StatusQ/src/StatusQ/Core/Theme/ThemePalette.qml index d702f64f83..395b29b653 100644 --- a/ui/StatusQ/src/StatusQ/Core/Theme/ThemePalette.qml +++ b/ui/StatusQ/src/StatusQ/Core/Theme/ThemePalette.qml @@ -169,6 +169,8 @@ QtObject { property color statusLoadingHighlight property color statusLoadingHighlight2 + property color messageHighlightColor + property var userCustomizationColors: [] property var identiconRingColors: []