Bug 453958 - Scrollbars go over listview content
Summary: Scrollbars go over listview content
Status: RESOLVED FIXED
Alias: None
Product: frameworks-qqc2-desktop-style
Classification: Frameworks and Libraries
Component: general (show other bugs)
Version: unspecified
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: Marco Martin
URL:
Keywords: regression
Depends on:
Blocks:
 
Reported: 2022-05-17 22:03 UTC by Tobias Fella
Modified: 2022-05-20 09:59 UTC (History)
5 users (show)

See Also:
Latest Commit: https://invent.kde.org/frameworks/qqc2-desktop-style/commit/5df82e90e171dbec604581f32fee581acb369d2b
Version Fixed In:
Sentry Crash Report:

Attachments
Test case (335 bytes, text/plain)
2022-05-17 22:03 UTC, Tobias Fella 		Details
screenshot (75.83 KB, image/png)
2022-05-17 22:04 UTC, Tobias Fella 		Details
View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
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