D22209: [effects/screenshot] Disable screenshot to file on Wayland

[Nathaniel Graham]

ngraham added subscribers: davidre, ngraham.
ngraham requested changes to this revision.
ngraham added reviewers: Spectacle, davidre.
ngraham added a comment.
This revision now requires changes to proceed.


  I understand the security implications here, and I don't approve of the change.
  
  In general, security that burdens and annoys the user is at best useless and at worst harmful, because the user will eventually be annoyed into finding a way to bypass it, or stop using the software. The whole "click to acknowledge that you want to take a screenshot" UX on Wayland is a major gripe of mine, and it make Spectacle less useful and elegant than it is on X11. There are even use cases where it would actively destroy the user experience, such as taking a full-screen screenshot to a file while playing an first-person shooter game. In this case, when you hit the screenshot key, you could get killed while the screen is showing the screenshot, and you could fire your gun by accident when you click to acknowledge it, potentially killing a teammate or giving away your position.
  
  I would prefer that we step back and re-examine how we provide authorization to take screenshots on Wayland, because there clearly are real security implications at play here. Instead of demanding authorization via click before every screenshot, perhaps we should allow apps to request blanket screenshot permissions once, not unlike how it's done on modern mobile phones. Then you would authorize Spectacle once, and never have to think about it again.
  
  This would in fact improve security because with fewer permissions requests, the user is more likely to actually read and evaluate each one individually. By contrast, when the user sees them all the time, they're likely to mindlessly click through to get back to what they're doing, like they do on Windows with its UAC spam.
  
  I'm adding #spectacle <https://phabricator.kde.org/tag/spectacle/> and @davidre so we can broaden the conversation and come up with something that works for everyone, results in a pleasant user experience, and retains security.

REPOSITORY
  R108 KWin

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

To: graesslin, #kwin, ngraham, #spectacle, davidre
Cc: ngraham, davidre, davidedmundson, kwin, LeGast00n, sbergeron, jraleigh, fbampaloukas, GB_2, mkulinski, ragreen, jackyalcine, Pitel, iodelay, crozbo, bwowk, ZrenBot, 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/20190705/a28ae92b/attachment-0001.html>