<table><tr><td style="">davidedmundson requested changes to this revision.<br />davidedmundson added a comment.<br />This revision now requires changes to proceed.
</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/D26859">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>We are using the same value as the second argument of addTablet() (se below). If we change here the serialId, we'd have to change it there too and we'd have the same behaviour.</p></blockquote>

<p>You might be right about serialId having read the docs, but I'm still seeing waaaaaay too many:</p>

<p>zwp_tablet_seat_v2.tool_added()</p>

<p>one for every move.</p>

<p>I have kwin on master + this patch, and master of kwayland.</p>

<p>I'll try and debug and find out.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R108 KWin</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D26859">https://phabricator.kde.org/D26859</a></div></div><br /><div><strong>To: </strong>apol, KWin, Plasma, zzag, davidedmundson<br /><strong>Cc: </strong>davidedmundson, zzag, kwin, Orage, cacarry, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, GB_2, mkulinski, ragreen, jackyalcine, iodelay, crozbo, bwowk, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, hardening, romangg, jensreuterberg, abetts, sebas, apol, ahiemstra, mart<br /></div>