Review Request 120993: Cleanup Notifications.qml

[Kai Uwe Broulik]

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


Sorry, I forgot to adjust the addNotification call in Jobs.qml

- Kai Uwe Broulik


On Nov. 4, 2014, 8:08 nachm., Kai Uwe Broulik wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/120993/
> -----------------------------------------------------------
> 
> (Updated Nov. 4, 2014, 8:08 nachm.)
> 
> 
> Review request for Plasma and Martin Klapetek.
> 
> 
> Repository: plasma-workspace
> 
> 
> Description
> -------
> 
> This cleans up the code in Notifications.qml a bit
>  - Some minor code style stuff
>  - new Array() ? []
>  - Pass the notification object we get from the data engine to addNotification directly to avoid this ugly function with dozen parameters and creating the object ourselves which we alredy almost get for free already
>  - Push actions into array directly instead of creating a new Object, settings its properties and then pushing it
> 
> 
> Diffs
> -----
> 
>   applets/notifications/package/contents/ui/Notifications.qml 47a475a 
> 
> Diff: https://git.reviewboard.kde.org/r/120993/diff/
> 
> 
> Testing
> -------
> 
> Didn't notice anything unusual after running knotificationsdbustest
> 
> 
> Thanks,
> 
> Kai Uwe Broulik
> 
>

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