summaryrefslogtreecommitdiffstats
path: root/kde/patch/qqc2-desktop-style/qqc2-desktop-style_scrollbar.patch
blob: 9f577e22ae48d216270c1e40089dba94420e4f95 (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
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
From 6995b4ae81a2f3ca3c24d2fa6d1560bfe0898737 Mon Sep 17 00:00:00 2001
From: Arjen Hiemstra <ahiemstra@heimr.nl>
Date: Mon, 10 Feb 2020 11:47:00 +0100
Subject: ScrollView: Use scrollbar height as bottom padding, not width

Summary:
Apparently I accidentally used the bottom scrollbar width instead of
height. Oops.

Test Plan: ScrollView with horizontal scrollbar no longer hides content

Reviewers: #plasma, davidedmundson

Reviewed By: #plasma, davidedmundson

Subscribers: plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D27280
---
 org.kde.desktop/ScrollView.qml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/org.kde.desktop/ScrollView.qml b/org.kde.desktop/ScrollView.qml
index 4bea5c7..38bc8ed 100644
--- a/org.kde.desktop/ScrollView.qml
+++ b/org.kde.desktop/ScrollView.qml
@@ -45,7 +45,7 @@ T.ScrollView {
     rightPadding: (internal.backgroundVisible && background.hasOwnProperty("rightPadding") ? background.rightPadding : 0)
                     + (!LayoutMirroring.enabled ? internal.verticalScrollBarWidth : 0)
     bottomPadding: (internal.backgroundVisible && background.hasOwnProperty("bottomPadding") ? background.bottomPadding : 0)
-                    + internal.horizontalScrollBarWidth
+                    + internal.horizontalScrollBarHeight
 
     //create a background only after Component.onCompleted, see on the component creation below for explanation
     Component.onCompleted: {
@@ -97,7 +97,7 @@ T.ScrollView {
 
             readonly property bool backgroundVisible: controlRoot.background && controlRoot.background.visible
             readonly property real verticalScrollBarWidth: controlRoot.ScrollBar.vertical.visible && !Kirigami.Settings.tabletMode ? controlRoot.ScrollBar.vertical.width : 0
-            readonly property real horizontalScrollBarWidth: controlRoot.ScrollBar.horizontal.visible && !Kirigami.Settings.tabletMode ? controlRoot.ScrollBar.horizontal.width : 0
+            readonly property real horizontalScrollBarHeight: controlRoot.ScrollBar.horizontal.visible && !Kirigami.Settings.tabletMode ? controlRoot.ScrollBar.horizontal.height : 0
         }
     ]
     ScrollBar.vertical: ScrollBar {
-- 
cgit v1.1