davidedmundson accepted this revision.
davidedmundson added inline comments.
This revision is now accepted and ready to land.
Restricted Application edited projects, added Plasma; removed KWin.

INLINE COMMENTS

> idle_inhibition.cpp:42
> +
> +void IdleInhibition::registerShellClient(ShellClient *client)
> +{

We could do with a line in wayland-protocol docs to say it has to be a toplevel 
surface.
Right now we're making an assumption. Albeit a fairly safe one.

REPOSITORY
  R108 KWin

BRANCH
  idle-inhibit

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

To: graesslin, #kwin, #plasma, davidedmundson
Cc: davidedmundson, plasma-devel, kwin, ZrenBot, progwolff, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

Reply via email to