This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/101981/

This review has been submitted with commit 7e8675b11baedc95eb388665f5f44f214b187a86 by Alex Fiestas to branch KDE/4.7.

- Commit


On July 17th, 2011, 6:49 p.m., Alex Fiestas wrote:

Review request for Plasma.
By Alex Fiestas.

Updated July 17, 2011, 6:49 p.m.

Description

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.

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!

Diffs

  • plasma/generic/applets/notifications/ui/stackdialog.h (89c36a9)
  • plasma/generic/applets/notifications/ui/stackdialog.cpp (0d3f6a0)

View Diff

Screenshots

Notification Wrong pos