D25427: [Wayland] Allow to take full resolution screenshot when scaling is used

[David Edmundson]

davidedmundson added a comment.


  > For single screen, there is nothing to argue, I believe, the patch solves the issue : do not downscale.
  
  Ack. The changes to blitScreenshot are all good to go. +2 on that bit.
  
  > pixel precision is needed for use cases such as debugging, "real" pixel screenshot, screenshot apps such as spectacle
  
  The spectacle part is where I'm a bit lost. Could be that you have additional plans there.
  
  Spectacle (currently) shows one mega-window spanning across all screens showing one image for its crop/select area function. Because it's all one window all contents need to be at the same scale as that wl_surface only has 1 scale.
  
  I don't see how Spectacle could do anything useful with this unless it got the image, then mapped it back to the screen geometry and split it up again into individual pieces.
  At that point would it work better for spectacle to call DBus  screenshotScreen(0); screenshotScreen(1); and getting native images back directly pre-split?
  
  > If that sound reasonable I will add a screenshotFullscreenScaled or a boolean scaled argument to screenshotFullscreen, to the screenshotEffect.
  
  Remember I loathe the term "scaled", but in principle I would be ok with that if we can work out how spectacle would use this.

REPOSITORY
  R108 KWin

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

To: meven, davidedmundson, #kwin
Cc: nicolasfella, zzag, davidre, kwin, Orage, cacarry, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, mkulinski, ragreen, jackyalcine, iodelay, crozbo, bwowk, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, hardening, romangg, jensreuterberg, abetts, sebas, apol, ahiemstra, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kwin/attachments/20200415/f3e1f754/attachment-0001.html>