D6852: Reset last_active_client when a ShellClient is removed

2017-07-23 Thread Martin Flöser
This revision was automatically updated to reflect the committed changes. Closed by commit R108:bd158a632100: Reset last_active_client when a ShellClient is removed (authored by graesslin). REPOSITORY R108 KWin CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D6852?vs=17060&id=17061 RE

D6852: Reset last_active_client when a ShellClient is removed

2017-07-23 Thread Martin Flöser
graesslin added a comment. In https://phabricator.kde.org/D6852#127904, @davidedmundson wrote: > there's 3 other differences between removeClient that I think should also be here: yep I know, guess what I will work on next ;-) should_get_focus is btw. currently not yet a prob

D6852: Reset last_active_client when a ShellClient is removed

2017-07-23 Thread David Edmundson
davidedmundson accepted this revision. davidedmundson added a comment. This revision is now accepted and ready to land. there's 3 other differences between removeClient that I think should also be here: --- should_get_focus.removeAll(c); (this looks like it could lead to a sim

D6852: Reset last_active_client when a ShellClient is removed

2017-07-23 Thread Martin Flöser
graesslin created this revision. Restricted Application added a project: KWin. Restricted Application added subscribers: kwin, plasma-devel. REVISION SUMMARY The last_active_client is set when an AbstractClient gets activated. For the X11 case the last_active_client was getting reset to nullpt