Code review comment for lp:~philip.scott/granite/notifications-demo

Revision history for this message
Adam Bieńkowski (donadigo) wrote :

It looks & works great, but following recent changes we should have "X-GNOME-UsesNotifications=true" in the .desktop file to fully support the app settings. Also I'm not sure if we want to showcase non-granite feature in a granite-demo app, notice how all widgets & functions there are granite specific.

review: Needs Fixing (code / testing)

« Back to merge proposal