----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126515/ -----------------------------------------------------------
(Updated Dez. 29, 2015, 9:19 nachm.) Review request for KDE Frameworks, Plasma, KDE Usability, and Martin Gräßlin. Changes ------- * Add context menu actions to SNI: Report Bug (to make it a bit more explicit in addition to just clicking it); Restart Application (in case you missed the notification you can still restart the app as long as the SNI is there) * Add contexts to i18n * Re-order code to be a bit clearer * Use custom notification event; the Catastrophe one draws a lot of attention to it with its sound Repository: plasma-workspace Description ------- This adds a new "--passive" option to DrKonqi where it will only show a StatusNotifierItem rather than bringing up the crash dialog right away. This can be useful for auto-restarting shell services (like plasmashell, krunner, kded) to improve the perceived quality of the product. On Windows RT, for example, the guidelines even explicitly say "rather just dump the user on the home screen than telling him something went wrong, so he can just quickly start the app again instead of being annoyed by an error message". On iOS you also just get dropped on the home screen. Windows desktop shows a "Searching for a solution" dialog which was *the* major annoyance when something crashed, rather than the actual crash. Video here: https://www.youtube.com/watch?v=t0ZLs-juYKc Diffs (updated) ----- drkonqi/CMakeLists.txt eaeaad4 drkonqi/main.cpp 7cbaae7 drkonqi/statusnotifier.h PRE-CREATION drkonqi/statusnotifier.cpp PRE-CREATION Diff: https://git.reviewboard.kde.org/r/126515/diff/ Testing ------- I crashed plasmashell, it restarted so fast that you didn't even have a black screen inbetween, just the panel restarting. Afterwards I got a SNI which opened DrKonqi when tapped. The SNI disappears after 1 minute because if you didn't bother to look after it by then, you probably forgot what you did to cause the crash anyway :) Thanks, Kai Uwe Broulik
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
