ngraham added inline comments.

INLINE COMMENTS

> PassiveNotification.qml:47
>          if (timeout == "short") {
> -            timer.interval = 1000;
>          } else if (timeout == "long") {

Not sure there was possibly a use case for a one-second textual notification. 
With this patch, the `short` duration is actually useful for very short 
messages.

REPOSITORY
  R169 Kirigami

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

To: ngraham, #kirigami, mart
Cc: plasma-devel, apol, davidedmundson, mart, hein

Reply via email to