D22033: Remove confirmation overlay from screenshotFullscreen

David Redondo noreply at phabricator.kde.org
Sun Jun 23 12:20:36 BST 2019


davidre created this revision.
davidre added a reviewer: KWin.
Herald added a project: KWin.
Herald added a subscriber: kwin.
davidre requested review of this revision.

REVISION SUMMARY
  It only appears for the method with the filedescriptor argument and not for the
  other argument variations. Also here the user doesn't need to select a Screen
  like in screenshotScreen as we always take a screenshot of all screens.

TEST PLAN
  Overlay doesn't appear when you call the Method

REPOSITORY
  R108 KWin

BRANCH
  screenshto (branched from master)

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

AFFECTED FILES
  effects/screenshot/screenshot.cpp
  effects/screenshot/screenshot.h

To: davidre, #kwin
Cc: kwin, LeGast00n, sbergeron, jraleigh, fbampaloukas, GB_2, mkulinski, ragreen, jackyalcine, Pitel, iodelay, bwowk, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kwin/attachments/20190623/e3d79cb4/attachment.html>


More information about the kwin mailing list