D20629: Support CriticalNotification type and place it in a CriticalNotificationLayer

davidedmundson added a comment.


  I'm going to act as translator as you two are going in circles.
  
    A client enters fullscreen mode;
    Notification A is shown
    Notification A expires
    client leaves fullscreen
  
  If we want to show notification A, we would have to track full screen. 
  But AFAIK we don't want that.
  
  I assume Kai meant this case:
  
    A client enters fullscreen mode;
    Notification A is shown and it hasn't expired;
    client leaves fullscreen
  
  If we want to now show notifcation A we would now need to track fullscreen changes and show/hide as appropriate, whereas we don't with this approach.

REPOSITORY
  R108 KWin

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

To: broulik, #kwin
Cc: davidedmundson, zzag, kwin, jraleigh, GB_2, mkulinski, ragreen, jackyalcine, Pitel, iodelay, bwowk, ZrenBot, ngraham, alexeymin, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart