D9391: [effects] Add 'Fullscreen' effect

[Martin Flöser]

graesslin added a comment.
Restricted Application edited projects, added Plasma; removed KWin.


  >> Due to that I personally would prefer if we don't add this for 5.12 or only for Wayland windows.
  > 
  > Is it okay to support only Wayland?
  
  Personally I would prefer to have the code generic. But we also see in the maximize effect that there are differences between X11 and Wayland which don't allow to have a generic effect (see bug 384462). So from a maintenance and forward compatibility point of view it makes sense to restrict to Wayland if we are aware that X11 is tricky.

REPOSITORY
  R108 KWin

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

To: zzag, #kwin, #plasma, graesslin
Cc: abetts, graesslin, broulik, plasma-devel, kwin, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180101/3b2c70e0/attachment.html>