mart accepted this revision.
mart added a reviewer: mart.
mart added inline comments.
This revision is now accepted and ready to land.

INLINE COMMENTS

> hein wrote in xwindowtasksmodel.h:211
> No, because this is X11-specific code that is never run on Wayland and not 
> built if X11 is not available at build time.
> 
> Now, we don't have a solution yet for dragging Wayland windows by reference. 
> Whether we can find a solution that works on both or need different 
> codepaths: Time will tell. For now this is just something we can do on X11 
> but not on Wayland, like many others - and it's not a new thing, this is old 
> code just moved here.
> 
> Note that once the Pager is ported to libtaskmanager both sides will go 
> through libtaskmanager and the TM appley already uses a generic data role 
> (AbstractTasksModel::MimeData) when starting a drag, so we'll have some 
> freedom to design a solution.
> 
> For now it's just keeping something already working working while allowing us 
> to drop liblegacytaskmanager and make forward progress :)

ok, thx

REPOSITORY
  rPLASMAWORKSPACE Plasma Workspace

BRANCH
  master

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

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: hein, #plasma, mart
Cc: mart, plasma-devel, jensreuterberg, abetts, sebas
_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Reply via email to