subdiff created this revision. subdiff added a reviewer: KWin. subdiff added a subscriber: plasma-devel. subdiff set the repository for this revision to R108 KWin. subdiff added a project: KWin. Restricted Application added a subscriber: kwin.
REVISION SUMMARY Unsetting a scroll method should only work, if the scroll method is currently active. For example when scroll-two-finger is active (i.e. scroll-edge, scroll-on-button-down is not active): - unsetting scroll-two-finger should deactivate it and activate no-scroll - but unsetting for example scroll-edge should **not** deactivate scroll-two-finger and **not** activate no-scroll Reworked setting scroll method auto test in order to test it. Regarding acceleration profiles: Unsetting one, should always switch to the other possible one as long as both are supported. In this case LIBINPUT_CONFIG_ACCEL_PROFILE_NONE is not a valid option in contrast to the definition of the scroll method enums. TEST PLAN Manually and auto tests. REPOSITORY R108 KWin REVISION DETAIL https://phabricator.kde.org/D3590 AFFECTED FILES autotests/libinput/device_test.cpp libinput/device.cpp libinput/device.h EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: subdiff, #kwin Cc: kwin, plasma-devel, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas
