----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127216/#review93133 -----------------------------------------------------------
Aside the two tems, there're some unrelated code changes (which i won't concern since i don't maintain this ;-) src/kstatusnotifieritem.cpp (line 425) <https://git.reviewboard.kde.org/r/127216/#comment63501> I assume this is the dealbreaker for the failing apps. This entire "restore position" mumbojumbo only makes sense if the associated widget can change, in which case this will break the positioning. => only store on minimizing and see what happens. src/kstatusnotifieritem.cpp (line 925) <https://git.reviewboard.kde.org/r/127216/#comment63500> you can skip NET::WMFrameExtents now - Thomas Lübking On Feb. 29, 2016, 5:18 a.m., Anthony Fieroni wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/127216/ > ----------------------------------------------------------- > > (Updated Feb. 29, 2016, 5:18 a.m.) > > > Review request for KDE Frameworks, Martin Gräßlin, Thomas Lübking, and Martin > Klapetek. > > > Bugs: 356523 > https://bugs.kde.org/show_bug.cgi?id=356523 > > > Repository: knotifications > > > Description > ------- > > Store position of widget before hide it > > > Diffs > ----- > > src/kstatusnotifieritem.cpp cf3e7b5 > src/kstatusnotifieritemprivate_p.h 8fdfd4c > > Diff: https://git.reviewboard.kde.org/r/127216/diff/ > > > Testing > ------- > > Tested on pixel ration = 1 > > > Thanks, > > Anthony Fieroni > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
