----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126658/#review90725 -----------------------------------------------------------
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. - Martin Klapetek On Jan. 7, 2016, 4:30 a.m., Anthony Fieroni wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/126658/ > ----------------------------------------------------------- > > (Updated Jan. 7, 2016, 4:30 a.m.) > > > 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 > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel