Review Request 126517: [ShellManager] Make use of StatusNotifier DrKonqi

David Edmundson david at davidedmundson.co.uk
Sun Dec 27 18:19:15 UTC 2015


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

Ship it!


Ship It!

- David Edmundson


On Dec. 25, 2015, 4:24 p.m., Kai Uwe Broulik wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/126517/
> -----------------------------------------------------------
> 
> (Updated Dec. 25, 2015, 4:24 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: plasma-workspace
> 
> 
> Description
> -------
> 
> This enables the StatusNotifier option introduced in Review 126516, see Review 126515 for more information.
> 
> When Plasma reached the recursion counter it disables the passive option since there will most likely be no System Tray to show this error and then something's really wrong in the first place.
> 
> 
> Diffs
> -----
> 
>   shell/shellmanager.cpp 0a6ac5e 
> 
> Diff: https://git.reviewboard.kde.org/r/126517/diff/
> 
> 
> Testing
> -------
> 
> Crashed plasma two times, it restarted and I got two SNIs, crashed it another time, got black screen with DrKonqi dialog.
> 
> 
> Thanks,
> 
> Kai Uwe Broulik
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20151227/023b1533/attachment.html>


More information about the Plasma-devel mailing list