----------------------------------------------------------- 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 > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel