D28155: Make it possible to have a separate cursor for the tablet

davidedmundson requested changes to this revision.
davidedmundson added a comment.
This revision now requires changes to proceed.


  Idea seems sensible.

INLINE COMMENTS

> pointer_input.cpp:172
>  {
> +    KWin::Cursors::self()->setMouse(new KWin::Cursor(this));
>      QVERIFY(Test::setupWaylandConnection(Test::AdditionalWaylandInterface::Seat | Test::AdditionalWaylandInterface::Decoration));

why is this needed?

> cursor.cpp:49
> +
> +void Cursors::addCursor(Cursor* cursor)
> +{

What happens when a cursor is deleted? For the tablet case it's parented to the tool

REPOSITORY
  R108 KWin

REVISION DETAIL
  https://phabricator.kde.org/D28155

To: apol, #kwin, cblack, davidedmundson
Cc: davidedmundson, cblack, kwin, Orage, cacarry, LeGast00n, The-Feren-OS-Dev, 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