Bug 453958

Summary:	Scrollbars go over listview content
Product:	[Frameworks and Libraries] frameworks-qqc2-desktop-style	Reporter:	Tobias Fella <fella>
Component:	general	Assignee:	Marco Martin <notmart>
Status:	RESOLVED FIXED
Severity:	normal	CC:	ahiemstra, kde, kinofhek, nate, noahadvs
Priority:	NOR	Keywords:	regression
Version First Reported In:	unspecified
Target Milestone:	---
Platform:	Other
OS:	Linux
Latest Commit:	https://invent.kde.org/frameworks/qqc2-desktop-style/commit/5df82e90e171dbec604581f32fee581acb369d2b	Version Fixed/Implemented In:
Sentry Crash Report:
Attachments:	Test case screenshot

Description Tobias Fella 2022-05-17 22:03:02 UTC

Created attachment 148936 [details]
Test case

Since https://invent.kde.org/frameworks/qqc2-desktop-style/-/merge_requests/143 the size of a scrollview is no longer adjusted for the width of the scrollbar, which causes the content to go behind it.

STEPS TO REPRODUCE
1. Save test case
2. run using QT_QUICK_CONTROLS_STYLE=org.kde.desktop qmlscene-qt5 --apptype widgets scrollbars.qml

OBSERVED RESULT
content behind scrollbars

EXPECTED RESULT
content not behind scrollbars

SOFTWARE/OS VERSIONS
KDE Plasma Version: 5.24
KDE Frameworks Version: git master
Qt Version: 5.15

ADDITIONAL INFORMATION

Comment 1 Tobias Fella 2022-05-17 22:04:31 UTC

Created attachment 148937 [details]
screenshot

Comment 2 Felipe Kinoshita 2022-05-18 03:45:25 UTC

Got same issue here

Comment 3 Nate Graham 2022-05-18 15:27:51 UTC

Can reproduce. I was just about to report this myself, so you beat me!

Comment 4 Marco Martin 2022-05-20 09:59:57 UTC

Git commit 5df82e90e171dbec604581f32fee581acb369d2b by Marco Martin.
Committed on 20/05/2022 at 09:58.
Pushed by mart into branch 'master'.

Fix overlapping scrollbar

margin condition logic was exactly inverted

M  +2    -2    org.kde.desktop/ScrollView.qml

https://invent.kde.org/frameworks/qqc2-desktop-style/commit/5df82e90e171dbec604581f32fee581acb369d2b