> On July 17, 2011, 7:49 p.m., Aaron J. Seigo wrote: > > plasma/generic/applets/notifications/ui/stackdialog.cpp, lines 298-299 > > <http://git.reviewboard.kde.org/r/101981/diff/1/?file=27299#file27299line298> > > > > these could be merged into one if () statement.
I prefer small conditionals but ok :p - Alex ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/101981/#review4776 ----------------------------------------------------------- On July 17, 2011, 6:49 p.m., Alex Fiestas wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/101981/ > ----------------------------------------------------------- > > (Updated July 17, 2011, 6:49 p.m.) > > > Review request for Plasma. > > > Summary > ------- > > On resizeEvent adjustPosition will be called so the widget can update the > position in relation with the new size. The problem before this patch is that > the code is not taking into account the position of the notification, whether > it is closer to the top or to the bottom. > > The current behavior is correct if the notification is closer to the top but > it is incorrect if the notification is closer to the bottom since in that > case we want the notification to increment the Y if the old size was bigger > than the current size. > > I can't find better words to describe the problem, so I will attach a > screenshot. > > > Diffs > ----- > > plasma/generic/applets/notifications/ui/stackdialog.h 89c36a9 > plasma/generic/applets/notifications/ui/stackdialog.cpp 0d3f6a0 > > Diff: http://git.reviewboard.kde.org/r/101981/diff > > > Testing > ------- > > Everything I could think of: > 1-Notifications in each corner of the screen > 2-Notifications "gap" into a side. > 3-Notifications at the middle > > So far, so good! > > > Screenshots > ----------- > > Notification Wrong pos > http://git.reviewboard.kde.org/r/101981/s/204/ > > > Thanks, > > Alex > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
