[Differential] [Commented On] D3476: Support screenshot current screen and fullscreen on KWin/Wayland

[graesslin (Martin Gräßlin)]

graesslin added inline comments.

INLINE COMMENTS

> bgupta wrote in KWinWaylandImageGrabber.h:48
> Can we re-use the ImageGrabber::CaptureMode enum here?

I would say no as the CaptureMode contains several choices which don't fit here. E.g. we cannot capture the ActiveWindow or a RectangularRegion through the DBus interface.

REPOSITORY
  rSPECTACLE Spectacle

BRANCH
  kwin-screenshot-screen

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

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

To: graesslin, #plasma_on_wayland, bgupta
Cc: plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20161124/9c04b779/attachment.html>