Bug 416348

Summary: Scrollbar on the right side is off-center
Product: [Plasma] Breeze Reporter: Matej Mrenica <matejm98mthw>
Component: QStyleAssignee: Unassigned bugs mailing-list <unassigned-bugs>
Status: RESOLVED FIXED    
Severity: minor CC: kde, matejm98mthw, nate, noahadvs, notmart
Priority: NOR Keywords: regression
Version: 5.17.90   
Target Milestone: ---   
Platform: Arch Linux   
OS: Linux   
Latest Commit: Version Fixed In: 5.18.0
Sentry Crash Report:
Attachments: Screenshot

Description Matej Mrenica 2020-01-16 19:23:07 UTC
It is moved to the right side more.


SOFTWARE/OS VERSIONS
KDE Plasma Version: 5.18 Beta
KDE Frameworks Version: 5.66
Qt Version: 5.14
Comment 1 Kai Uwe Broulik 2020-01-16 19:25:34 UTC
Screenshot, please.
Comment 2 Matej Mrenica 2020-01-16 19:39:32 UTC
Created attachment 125178 [details]
Screenshot
Comment 3 Nate Graham 2020-01-17 20:32:03 UTC
Minor bug in Breeze with the new scrollbar style.
Comment 4 Noah Davis 2020-01-17 22:40:30 UTC
This should have been fixed already.
Comment 5 Nate Graham 2020-01-20 15:49:36 UTC
I still see a one-pixel mis-alignment. It seems to be fixed with https://phabricator.kde.org/D26783 though.
Comment 6 Marco Martin 2020-01-21 08:56:20 UTC
Git commit 5a447b59fc96e81ea8a74ff338aca5c35ec6714a by Marco Martin.
Committed on 21/01/2020 at 08:56.
Pushed by mart into branch 'master'.

Center only during drawing, not the hit rects

Summary:
center only the drawing of the components but never modify the
subcontrolrects (and hit targets)
FIXED-IN: 5.18.0

Test Plan:
now the handle activates immediately without one single pixel
margin

Reviewers: #vdg, hpereiradacosta, ndavis

Reviewed By: #vdg, ndavis

Subscribers: ngraham, plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D26783

M  +42   -10   kstyle/breezestyle.cpp

https://commits.kde.org/breeze/5a447b59fc96e81ea8a74ff338aca5c35ec6714a
Comment 7 Marco Martin 2020-01-21 08:56:55 UTC
Git commit 650f2cef0752297526fa4ccfa65530fd48aeb607 by Marco Martin.
Committed on 21/01/2020 at 08:56.
Pushed by mart into branch 'Plasma/5.18'.

Center only during drawing, not the hit rects

Summary:
center only the drawing of the components but never modify the
subcontrolrects (and hit targets)
FIXED-IN: 5.18.0

Test Plan:
now the handle activates immediately without one single pixel
margin

Reviewers: #vdg, hpereiradacosta, ndavis

Reviewed By: #vdg, ndavis

Subscribers: ngraham, plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D26783

M  +42   -10   kstyle/breezestyle.cpp

https://commits.kde.org/breeze/650f2cef0752297526fa4ccfa65530fd48aeb607