Re: Review Request 127162: RFC: Improve frozen window experience

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/127162/
-----------------------------------------------------------

(Updated Sept. 6, 2016, 8:56 nachm.)


Status
------

This change has been discarded.


Review request for kwin and KDE Usability.


Repository: kwin


Description
-------

This is a little experiment I made which raises a bunch of technical and usability concerns. It basically consists of two parts:

* Introduce a visual effect for a window that is not responding, something that has been floating around as an idea for years and is widely used in other DEs
* Make the KWin killer notification prettier

A video can be found here: https://www.youtube.com/watch?v=h3UnH1yx624

When Plasma is not running or you use it in an environment whose notification system doesn't support action buttons (lxqt?) it falls back to the old message box we have right now.

My questions/concerns are:

* Not too happy with the name, also not very consistent "frozen app" vs "ping timed out" vs ...
* Now that I think about it I'm not too sure about using a Plasma notification is the right approach. I mostly did this as the modal message box dims the window and makes the effect hard to see :D Most importantly, though, it does not accept keyboard input, so the old usecase of "press X, wait, hit return" no longer works. Not that we should encourage people to kill apps, though. Another approach could be a floating popup connected to the window that says "hey this app is frozen [terminate]". I recall Windows Longhorn (Vista Alpha) showed a concept where a baloon tooltip emerged from the close button, I quite liked it, but I can't find a screenshot right now
* Is it possible to detect hung applications if not explicitly clicking the X button? so when I click the window it can already apply the effect after, say 3 seconds, and then after another 3s or another click show the notification? I guess... Wayland?


Diffs
-----

  abstract_client.h fff6feb 
  abstract_client.cpp 8820e6d 
  client.h 697b641 
  client.cpp cb8dbf1 
  effects.h dfe6f12 
  effects.cpp 8ecf1d7 
  effects/CMakeLists.txt 0e5ed97 
  effects/frozenapp/CMakeLists.txt PRE-CREATION 
  effects/frozenapp/package/CMakeLists.txt PRE-CREATION 
  effects/frozenapp/package/contents/code/frozenapp.js PRE-CREATION 
  effects/frozenapp/package/metadata.desktop PRE-CREATION 
  killer/CMakeLists.txt 9e867ec 
  killer/killer.cpp 3b4e21c 
  kwin.notifyrc f5a62c3 
  libkwineffects/kwineffects.h d3e3612 

Diff: https://git.reviewboard.kde.org/r/127162/diff/


Testing
-------


Thanks,

Kai Uwe Broulik