Review Request 121360: Rework Plasma's notification positioning code

[Kai Uwe Broulik]

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


Overall the notification appearance is much cleaner, they don't overlap anymore and when they're updated they don't just flicker re-appear but fade out and fade in again.
However, the vertical positioning is completely off with the notifications dancing around the entire height of the screen from which it never recovers when you had multiple notifications appearing in quick succession.


applets/notifications/plugin/notificationshelper.cpp
<https://git.reviewboard.kde.org/r/121360/#comment50826>

    Use QScopedPointer



applets/notifications/plugin/notificationshelper.cpp
<https://git.reviewboard.kde.org/r/121360/#comment50823>

    New style connect?



applets/notifications/plugin/notificationshelper.cpp
<https://git.reviewboard.kde.org/r/121360/#comment50824>

    Heh.


- Kai Uwe Broulik


On Jan. 3, 2015, 12:26 vorm., Martin Klapetek wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/121360/
> -----------------------------------------------------------
> 
> (Updated Jan. 3, 2015, 12:26 vorm.)
> 
> 
> Review request for Plasma and Kai Uwe Broulik.
> 
> 
> Bugs: 339732
>     https://bugs.kde.org/show_bug.cgi?id=339732
> 
> 
> Repository: plasma-workspace
> 
> 
> Description
> -------
> 
> There can easily be situations where the popups could overlap one another or result in strange animations. This patch rewrites the notifications so that all actions such as show/reposition/hide are handled from a one single place and every action is properly queued and protected around, which makes it more robust, more predictive and less chaotic. There's also a slight delay between every action so it's also visually much more cleaner and easier to see what's going on. 
> 
> 
> Diffs
> -----
> 
>   applets/notifications/package/contents/ui/NotificationPopup.qml 4491230 
>   applets/notifications/plugin/notificationshelper.h af8f6fa 
>   applets/notifications/plugin/notificationshelper.cpp 425f0d6 
> 
> Diff: https://git.reviewboard.kde.org/r/121360/diff/
> 
> 
> Testing
> -------
> 
> Tested whole day plus stress-tested with something like for i in {1..200}; do notify-send "$i - $RANDOM" "$RANDOM sdf sdf sdfwefhsdjfnskdfbkwefnos igodsfgn sodifgj asodfgnsdlfgdf g"; done executed from 4 terminals at once, all works fine and as expected.
> 
> 
> Thanks,
> 
> Martin Klapetek
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150103/c866862b/attachment-0001.html>