----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/105893/#review16967 -----------------------------------------------------------
5 minutes fixed in code.. Two possible suggestions. 1. Could you perhaps use the screensaver which probably means that this auto deletion is always done except when the screensaver is running? 2. Use KIdleTime thus always auto delete notifications except when the user is idle. http://api.kde.org/4.8-api/kdelibs-apidocs/kutils/html/classKIdleTime.html Just a few suggestions :) - Mark Gaiser On Aug. 6, 2012, 11:01 a.m., Martin Koller wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/105893/ > ----------------------------------------------------------- > > (Updated Aug. 6, 2012, 11:01 a.m.) > > > Review request for Plasma and Marco Martin. > > > Description > ------- > > as discussed in private mail, the auto-deletion timer shall only be started > from one single place, which is > the notificationmanager, which starts the timer only if there is user > activity on the desktop so that a notification > which arrives while the user is not at his desk does not get deleted unseen. > Also, the auto-deletion timeout is reduced to 5 minutes, as I think 20 > minutes is much too long (Notifications should only > provide temporary information for the user for things happening NOW. After > some minutes the information is irrelevant). > For information which is of critical importance, the application can use the > urgency=2 which avoids the auto-deletion completely. > > > Diffs > ----- > > plasma/generic/applets/notifications/core/notification.cpp 1e1350e > > Diff: http://git.reviewboard.kde.org/r/105893/diff/ > > > Testing > ------- > > > Thanks, > > Martin Koller > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
