<table><tr><td style="">subdiff added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D8168" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>Wow, very nice! I'm still not sure if on Wayland we should combine Touchpad and Mouse KCMs to one "Pointers" KCM how it was proposed years ago, because their properties in libinput are very similar. But since you already did the work for adding multiple backend to the Mouse KCM let's go with two separate ones now!</p>

<p>Your change is very big though. Could we split it up in a patch series? Basically by your bullets point in the description. Then we would have the following patches in order:</p>

<ol class="remarkup-list">
<li class="remarkup-list-item">UI in kcmmouse.ui.</li>
<li class="remarkup-list-item">Backend refactoring</li>
<li class="remarkup-list-item">Libinput mouse acceleration and revert scroll (on X)</li>
<li class="remarkup-list-item">Libinput macros</li>
<li class="remarkup-list-item">No revert scrolling for touchpads</li>
</ol>

<p>Maybe (2) needs to switch position with (3) and/or (4) with (5).</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R119 Plasma Desktop</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D8168" rel="noreferrer">https://phabricator.kde.org/D8168</a></div></div><br /><div><strong>To: </strong>xuetianweng, davidedmundson, mart<br /><strong>Cc: </strong>subdiff, ngraham, plasma-devel, leezu, ZrenBot, alexeymin, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, eliasp, sebas, apol, mart, hein<br /></div>