summaryrefslogtreecommitdiff
blob: ffc94e9c97ab3f7c48d448580bd7f1cba83cdef0 (plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
From bdd195792b7752e4c5a0ff74cb5b01a6accee517 Mon Sep 17 00:00:00 2001
From: Jan Paul Batrina <jpmbatrina01@gmail.com>
Date: Thu, 25 Mar 2021 19:31:39 +0800
Subject: [PATCH] Use QPalette::highlight for the scrollbar minimap slider

Previously, QPalette::link() was used which is the color for hyperlinks.
Most themes set this to a shade of blue, so the slider will almost
always be blue even when, for example, the main color
of a theme is green.

The selection highlight color was chosen because it is also the color
used for indicating the active tab and is, generally, often attuned
to a theme's main color.

BUG: 434690
---
 src/view/kateviewhelpers.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/view/kateviewhelpers.cpp b/src/view/kateviewhelpers.cpp
index 365e07e0..48e94609 100644
--- a/src/view/kateviewhelpers.cpp
+++ b/src/view/kateviewhelpers.cpp
@@ -750,7 +750,7 @@ void KateScrollBar::miniMapPaintEvent(QPaintEvent *e)
     // calculate colors
     const QColor backgroundColor = m_view->defaultStyleAttribute(KTextEditor::dsNormal)->background().color();
     const QColor foregroundColor = m_view->defaultStyleAttribute(KTextEditor::dsNormal)->foreground().color();
-    const QColor highlightColor = palette().link().color();
+    const QColor highlightColor = palette().highlight().color();
 
     const int backgroundLightness = backgroundColor.lightness();
     const int foregroundLightness = foregroundColor.lightness();
-- 
GitLab