<div dir="ltr">Hi Dmitry - thanks a lot for taking over the tablet code. I built and tested your branch and it seems to work nicely for me. Krita seems to alright on my multiple monitor setup.<div><br><div>I rebased your branch on master to check for conflicts, sorry I didn't know you had started work on this already. There were some small conflicts having to do with the EventEater in KisInputManager, but the wintab code doesn't make any use of that, so the conflicts only need to resolve in such a way as to ensure the xcb version works.<div><br></div><div>The X tablet handling was so smooth that I noticed the wintab version was performing worse, and as a result I went back to fix a bug in the tracking code!</div></div></div><div><br></div><div><br></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Dec 12, 2015 at 3:49 PM, Boudewijn Rempt <span dir="ltr"><<a href="mailto:boud@valdyas.org" target="_blank">boud@valdyas.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Cool work! Ill build the branch before I have to go back to "work" -- but it might be a good idea to put hte diff on phab, too, for easier reviewing.<div class="HOEnZb"><div class="h5"><br>
<br>
On Sat, 12 Dec 2015, Dmitry Kazakov wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi, all!<br>
<br>
I have just pushed my branch 'krita-tablet-x11-kazakov', where I implemented the first 'forked' version of the tablet support code<br>
for Qt5. Please review it, and, if possible, test :)<br>
<br>
There are a lot of things not working yet:<br>
<br>
1) Keyboard modifiers do not work (I didn't fork that part yet).<br>
2) Popup palette doesn't have a cursor for some reason.<br>
3) Canvas decorations' event filters are not tested (I just don't know if they work or not)<br>
<br>
But the main problems are solved:<br>
<br>
1) There are no cursor lockups due to mixed mouse events.<br>
2) The strokes are very smooth (at least on Wacom devices)<br>
<br>
What I tested already:<br>
<br>
1) I tested Wacom Intuos 5, Wacom Graphire 2, Genius G-Pen 560 devices and they seem to work fine.<br>
2) The devices are tested on Qt 5.4 only. The code might break on other versions of Qt, who knows...<br>
<br>
What needs to be tested:<br>
<br>
1) Other versions of Qt<br>
2) Other tablet devices<br>
3) Configurations with multiple displays<br>
<br>
For Michael Abrahams:<br>
<br>
Michael, could you please review my branch and check if there might be any conflicts with the refactoring you do atm? I based my<br>
code on your initial implementation, but I had to revert a few commits, basically the ones which did that weird filtering for<br>
unordered events :)<br>
<br>
--<br>
Dmitry Kazakov<br>
<br>
<br>
</blockquote>
<br></div></div><span class="HOEnZb"><font color="#888888">
-- <br>
Boudewijn Rempt | <a href="http://www.krita.org" rel="noreferrer" target="_blank">http://www.krita.org</a>, <a href="http://www.valdyas.org" rel="noreferrer" target="_blank">http://www.valdyas.org</a><br>
_______________________________________________<br>
Krita mailing list<br>
<a href="mailto:kimageshop@kde.org" target="_blank">kimageshop@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/kimageshop" rel="noreferrer" target="_blank">https://mail.kde.org/mailman/listinfo/kimageshop</a><br>
</font></span></blockquote></div><br></div>