Differential D27280

ScrollView: Use scrollbar height as bottom padding, not width
ClosedPublic
Actions

Authored by ahiemstra on Feb 10 2020, 10:48 AM.

Details

Reviewers
davidedmundson
Group Reviewers
Plasma
Commits
R858:2b4feba0896e: ScrollView: Use scrollbar height as bottom padding, not width
R858:6995b4ae81a2: 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

Diff Detail

Repository
R858 Qt Quick Controls 2: Desktop Style
Branch
horizontal_height
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 22288
Build 22306: arc lint + arc unit
ahiemstra created this revision.Feb 10 2020, 10:48 AM
Restricted Application added a project: Plasma. · View Herald TranscriptFeb 10 2020, 10:48 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
ahiemstra requested review of this revision.Feb 10 2020, 10:48 AM
davidedmundson accepted this revision.Feb 10 2020, 10:49 AM
This revision is now accepted and ready to land.Feb 10 2020, 10:49 AM
Closed by commit R858:6995b4ae81a2: ScrollView: Use scrollbar height as bottom padding, not width (authored by ahiemstra). · Explain WhyFeb 10 2020, 10:51 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Morg.kde.desktop/ScrollView.qml (4 lines)

Diff 75346

View Options

org.kde.desktop/ScrollView.qml

Loading...