<table><tr><td style="">romangg accepted this revision.<br />romangg added a comment.<br />This revision is now accepted and ready to land.
</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/D20186">View Revision</a></tr></table><br /><div><div><p>I tested it now. I think it's working robustly and should still go in the upcoming release. We have the beta phase to further test then.</p>

<p>For the next release some pointers where still work could be done:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Touchpads do never feature acceleration profiles I believe. Just never show the all the time disabled controls. Same holds for scroll button. The respective backend interfaces can then be cleaned out as well.</li>
<li class="remarkup-list-item">There is some weirdness going on if no touchpad is connected and one opens the touchpad KCM. Normally all controls should be greyed out. But at least on Wayland they are not but instead some of them are not shown.</li>
</ul>

<p>But for now let's celebrate finally shipping a fitting libinput KCM on X as well! Great job Atul! Thank you for your hard work and for being patient in the review process.</p></div></div><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D20186#inline-118924">View Inline</a><span style="color: #4b4d51; font-weight: bold;">libinputcommon.h:2</span></div>
<div style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; white-space: pre-wrap; clear: both; padding: 4px 0; margin: 0;"><div style="padding: 0 8px; margin: 0 4px; "><span style="color: #74777d">/*</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(251, 175, 175, .7);"><span style="color: #74777d"> * Copyright 201<span class="bright">7 Roman Gilg <subdiff</span>@gmail.com></span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);"><span style="color: #74777d"> * Copyright 201<span class="bright">9 Atul Bisht <atulbisht26</span>@gmail.com></span>
</div><div style="padding: 0 8px; margin: 0 4px; "><span style="color: #74777d"> *</span>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Most of the code in this file was written by me and is now copy-pasted. Please keep in my copyright and add your own above/below.</p></div></div><br /><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D20186#inline-118923">View Inline</a><span style="color: #4b4d51; font-weight: bold;">libinputcommon.h:54</span></div>
<div style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; white-space: pre-wrap; clear: both; padding: 4px 0; margin: 0;"><div style="padding: 0 8px; margin: 0 4px; ">    <span style="color: #74777d">// general</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(251, 175, 175, .7);">    <span class="n">Q_PROPERTY</span><span class="p">(</span><span class="n">QString</span> <span class="n">name</span> <span class="n">READ</span> <span class="n">name</span> <span class="n">CONSTANT</span><span class="p">)</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(251, 175, 175, .7);">    <span class="n">Q_PROPERTY</span><span class="p">(</span><span style="color: #aa4000">bool</span> <span class="n">supportsDisableEvents</span> <span class="n">READ</span> <span class="n">supportsDisableEvents</span> <span class="n">CONSTANT</span><span class="p">)</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">    <span class="n">Q_PROPERTY</span><span class="p">(</span><span class="n">QString</span> <span class="n">name</span> <span class="n">READ</span> <span class="n">name</span> <span class="n">CONSTANT</span><span class="p">)<span class="bright"></span></span><span class="bright"></span><span style="color: #74777d"><span class="bright">//</span></span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">    <span class="n">Q_PROPERTY</span><span class="p">(</span><span style="color: #aa4000">bool</span> <span class="n">supportsDisableEvents</span> <span class="n">READ</span> <span class="n">supportsDisableEvents</span> <span class="n">CONSTANT</span><span class="p">)<span class="bright"></span></span><span class="bright"></span><span style="color: #74777d"><span class="bright">//</span></span>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">rm these comment markers</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R119 Plasma Desktop</div></div></div><br /><div><strong>BRANCH</strong><div><div>arcpatch-D20186</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D20186">https://phabricator.kde.org/D20186</a></div></div><br /><div><strong>To: </strong>atulbi, ngraham, romangg, davidedmundson, Plasma<br /><strong>Cc: </strong>GB_2, jriddell, knambiar, plasma-devel, jraleigh, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>