Review Request 126658: Notifications again not works correctly

[Anthony Fieroni]

> On Ян. 7, 2016, 5:36 преди обяд, Martin Klapetek wrote:
> > No.
> > 
> > I'm fine with disabling the first animation. Post a separate patch
> > for that and that only.
> > 
> > Any configuration issues that are left need to be identified
> > and fixed, not by reverting and moving things back again.

https://youtu.be/kavjfrRny9I
Video, flying with first test, see configuration.
Test mine, please, revert 126408.


- Anthony


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


On Ян. 7, 2016, 5:30 преди обяд, Anthony Fieroni wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/126658/
> -----------------------------------------------------------
> 
> (Updated Ян. 7, 2016, 5:30 преди обяд)
> 
> 
> Review request for Plasma, David Edmundson, Marco Martin, and Martin Klapetek.
> 
> 
> Repository: plasma-workspace
> 
> 
> Description
> -------
> 
> Review 126408 completely same flying and configurations issues
> 
> 
> Diffs
> -----
> 
>   applets/notifications/plugin/notificationshelper.cpp 15b4479 
>   applets/notifications/package/contents/ui/configNotifications.qml 95a8e59 
>   applets/notifications/package/contents/ui/Notifications.qml f479a65 
>   applets/notifications/lib/notificationsapplet.h 5b262f1 
>   applets/notifications/lib/notificationsapplet.cpp 891cdb0 
>   applets/notifications/plugin/notificationshelper.h 860a2da 
>   applets/notifications/package/contents/ui/ScreenPositionSelector.qml efff648 
> 
> Diff: https://git.reviewboard.kde.org/r/126658/diff/
> 
> 
> Testing
> -------
> 
> My code works *correctly*, revert patch from 126408 - configurations issues.
> Qt 5.5.1, libxcb 1.11.1 Arch Linux
> 
> 
> Thanks,
> 
> Anthony Fieroni
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160107/9ed8f468/attachment.html>