This revision was automatically updated to reflect the committed changes.
Closed by commit R858:9dfb38179ac5: Remove scrollbar related workarounds from
list delegates (authored by ahiemstra).
REPOSITORY
R858 Qt Quick Controls 2: Desktop Style
CHANGES SINCE LAST UPDATE
https://phabricator.kde
ahiemstra added a comment.
That logic is in ScrollView already. Since the delegate no longer needs
special padding, this changes the padding to the same value for left or right.
We could change the padding depending on where the scrollbar is, but I don't
think that would be an improvement.
ngraham added a comment.
Will we still need the `LayoutMirroring.enabled ?` logic to account for when
using an RTL language where the scrollbar is on the left side?
REPOSITORY
R858 Qt Quick Controls 2: Desktop Style
REVISION DETAIL
https://phabricator.kde.org/D26686
To: ahiemstra, #plas
ahiemstra created this revision.
ahiemstra added a reviewer: Plasma.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
ahiemstra requested review of this revision.
REVISION SUMMARY
Now that we no longer overlap scrollbars, we can get rid of all the
workarounds
needed to
