Review Request 120997: Add timestamp to notification history

Thomas Pfeiffer colomar at autistici.org
Wed Nov 5 09:41:44 UTC 2014


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


Great work, this looks very useful, and big bonus points for implementing the relative timestamps!

One thing: What happens with notifications from previous days? This sounds a bit like an edge case, but for notifications that stay indefinitely and users (mostly devs and admins I guess) that don't reboot their computer daily, ir would be relevant.

We could then just write "Yesterday", "2 days ago" etc. If this is not too much work, I'd switch from time to "Yesterday" at t-24h. Users should be able to realuze that a timestamp which is later than the current time is likely from yesterday, but beyond 24hrs, it could be confusing.

- Thomas Pfeiffer


On Nov. 4, 2014, 9:33 nachm., Kai Uwe Broulik wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/120997/
> -----------------------------------------------------------
> 
> (Updated Nov. 4, 2014, 9:33 nachm.)
> 
> 
> Review request for Plasma, KDE Usability and Martin Klapetek.
> 
> 
> Repository: plasma-workspace
> 
> 
> Description
> -------
> 
> This simple patch shows the time when a notification was added to the history.
> 
> Of course would be fancy to have "1min ago", "5min ago" etc but that's much more complicated since we cannot listen to changes of a Date() object but would need to pull in the dataengine and then we get timezone sync issues and what not. So this just stores a Date() object and displays it as hours.
> 
> 
> Diffs
> -----
> 
>   applets/notifications/package/contents/ui/Notifications.qml 47a475a 
>   applets/notifications/package/contents/ui/NotificationDelegate.qml 75bcfd5 
>   applets/notifications/package/contents/ui/NotificationItem.qml 724cc13 
> 
> Diff: https://git.reviewboard.kde.org/r/120997/diff/
> 
> 
> Testing
> -------
> 
> Pretty nifty! The looks could be improved a bit I guess.
> 
> 
> File Attachments
> ----------------
> 
> Notification history with timestamp
>   https://git.reviewboard.kde.org/media/uploaded/files/2014/11/04/20f3c850-0f64-4807-bbde-064bf730d830__notificationtimestamp.png
> Multiple notifications with relative time
>   https://git.reviewboard.kde.org/media/uploaded/files/2014/11/04/f4979255-2971-4458-bf2c-ec05819591e5__notificationtimestamp1.png
> 
> 
> Thanks,
> 
> Kai Uwe Broulik
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20141105/1c23ff0c/attachment.html>


More information about the Plasma-devel mailing list