----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126517/#review90102 -----------------------------------------------------------
On first thought this is awesome. I like the video. It looks good. but now I'm going to point something out that's going to ruin your Christmas If Plasma crashes, you're going to try and show an SNI *in plasma* ... which isn't running. That's fine for a single crash, not for ones that block startup. I'm sure we can workaround this somehow - there's a timer in KCrash internals where we can know if something is a crash within the first 20 seconds of loading. - 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 > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
