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


Looks good, though do note that the spec you link to is a Gnome spec and Gnome 
doesn't really care that much. I've proposed sometime ago to update the Galago 
spec with it but "we host libnotify at Gnome servers and so we get to control 
the spec" was moreless the answer. So I don't care about that Gnome spec.

Nevertheless, the patch is still correct even wrt the Galago spec.

- Martin Klapetek


On Oct. 24, 2015, 2:40 p.m., David Edmundson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/125770/
> -----------------------------------------------------------
> 
> (Updated Oct. 24, 2015, 2:40 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: plasma-workspace
> 
> 
> Description
> -------
> 
> Expired notifications should emit a different reason in
> NotificationClosed than manually closed notifications
> 
> See 
> https://people.gnome.org/~mccann/docs/notification-spec/notification-spec-1.1.html#signal-notification-closed
> 
> BUG: 354293
> 
> 
> Diffs
> -----
> 
>   applets/notifications/package/contents/ui/NotificationPopup.qml 
> 5fd06bb4ee437e673ef9cc09907764b40cfa5a74 
>   applets/notifications/package/contents/ui/Notifications.qml 
> 13be7d49a14ebbe5399a7f0bf333862f545671a2 
>   dataengines/notifications/notificationaction.cpp 
> 5302f7808fd43c924a3f7b1fd7cfc0acba45c500 
>   dataengines/notifications/notifications.operations 
> 046776ec1af981d7fb23db9ddb9bd9d34ec55300 
>   dataengines/notifications/notificationsengine.h 
> 52590950ad1c85f427502b598e0b21d8a65e587e 
>   dataengines/notifications/notificationsengine.cpp 
> 42e56d8ab27b3a012381cbd0ce39c6322d09e152 
> 
> Diff: https://git.reviewboard.kde.org/r/125770/diff/
> 
> 
> Testing
> -------
> 
> Ran dbus-monitor & notify-send
> 
> let one notification expire
> clicked close on another
> 
> 
> Thanks,
> 
> David Edmundson
> 
>

_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Reply via email to