ngraham added inline comments.

INLINE COMMENTS

> PassiveNotification.qml:18
> +notifications in the app.
> + * used to show messages of limited importance that make sense only when
> + * the user is using the application and wouldn't be suited as a global 

`messages of limited importance` -> maybe something a bit more explicit like 
`transient, ignorable status messages of limited importance`

> PassiveNotification.qml:21
> + * system-wide notification.
> + * This is not a full-fledged notification system. the applciation should 
> + * use this with care and only one notification should be visible at once 
> per app.

`the applciation` -> `The application`

REPOSITORY
  R169 Kirigami

REVISION DETAIL
  https://phabricator.kde.org/D28103

To: mart, #kirigami
Cc: ngraham, plasma-devel, fbampaloukas, GB_2, domson, dkardarakos, apol, 
ahiemstra, davidedmundson, mart

Reply via email to