Review Request 122230: Continuously update notification timestamp

Martin Klapetek martin.klapetek at gmail.com
Fri Jan 23 20:26:01 UTC 2015


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

Ship it!


Ship It!

- Martin Klapetek


On Jan. 23, 2015, 8:38 p.m., Kai Uwe Broulik wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/122230/
> -----------------------------------------------------------
> 
> (Updated Jan. 23, 2015, 8:38 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: plasma-workspace
> 
> 
> Description
> -------
> 
> While the popup is opened update the timestamps, especially the first ones (Just now, 10s ago, 30s ago) come in quick succession making it feel weird for them to only update when opening the dialog. Even simplifies the code now.
> 
> 
> Diffs
> -----
> 
>   applets/notifications/package/contents/ui/NotificationItem.qml c476749 
> 
> Diff: https://git.reviewboard.kde.org/r/122230/diff/
> 
> 
> Testing
> -------
> 
> Works
> 
> 
> Thanks,
> 
> Kai Uwe Broulik
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150123/6daf7b54/attachment.html>


More information about the Plasma-devel mailing list