Differential D21049

[wizard] Enable 1-finger touch scrolling
ClosedPublic
Actions

Authored by nicolasfella on May 6 2019, 5:23 PM.

Details

Reviewers
drosca
bshah
Group Reviewers
Plasma
Summary

Set a QScroller on the ListView

BUG: 406544

Test Plan

Scrolling with touchpad: No change
Drag one finger on a touchscreen on the devicelist: List moves

Diff Detail

Repository
R97 Bluedevil
Branch
touchscroll
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 11605
Build 11623: arc lint + arc unit
nicolasfella created this revision.May 6 2019, 5:23 PM
Restricted Application added a project: Plasma. · View Herald TranscriptMay 6 2019, 5:23 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
nicolasfella requested review of this revision.May 6 2019, 5:23 PM
nicolasfella edited the summary of this revision. (Show Details)May 6 2019, 5:23 PM
apol added a subscriber: apol.May 6 2019, 5:45 PM

Would it maybe be possible to find a more generic way to tackle this? Maybe at Breeze or KStyle level, this is an issue on every QListView :(.

Also this will be needed on different kind of devices, would it make sense to consider porting this to QQC2?

ngraham added a subscriber: ngraham.May 6 2019, 7:07 PM
In D21049#461799, @apol wrote:

Would it maybe be possible to find a more generic way to tackle this? Maybe at Breeze or KStyle level, this is an issue on every QListView :(.

+1, that would be super awesome if we can find a way to provide touch scrolling to a bunch of QWidgets components all at once.

bshah accepted this revision as: bshah.Aug 30 2020, 4:46 PM
bshah added a subscriber: bshah.

I guess it would be nice to have this fixed somewhere else down the stack, but would be nice to have this merged anyway as at least interim solution

This revision is now accepted and ready to land.Aug 30 2020, 4:46 PM
nicolasfella closed this revision.Aug 30 2020, 6:44 PM

Revision Contents
Changeset List

PathPackages
Msrc/wizard/pages/discover.cpp (3 lines)

Diff 57672

View Options

src/wizard/pages/discover.cpp

Loading...