----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/110476/#review33973 -----------------------------------------------------------
Anyone please? :-) - Dan Vrátil On May 16, 2013, 9:38 p.m., Dan Vrátil wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/110476/ > ----------------------------------------------------------- > > (Updated May 16, 2013, 9:38 p.m.) > > > Review request for kde-workspace. > > > Description > ------- > > Call KNotification::close() when user clicks the 'X' button in the > Notifications applet. > > This prevents leaking persistent notifications and allows applications to > depend on KNotification::closed() signal. > > > Diffs > ----- > > > plasma/generic/applets/notifications/contents/ui/NotificationDelegate/NotificationDelegate.qml > 64d9298 > plasma/generic/applets/notifications/contents/ui/Notifications.qml c5be0a3 > plasma/generic/dataengines/notifications/notificationaction.cpp 397073e > plasma/generic/dataengines/notifications/notificationsengine.cpp 25fccbb > > Diff: http://git.reviewboard.kde.org/r/110476/diff/ > > > Testing > ------- > > > Thanks, > > Dan Vrátil > >
