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

This review has been submitted with commit 2df720859b5164143c064d9aa4f491bde920a0ad by Nikita Churaev to branch KDE/4.8.

- Commit


On December 31st, 2011, 5:18 a.m., Nikita Churaev wrote:

Review request for Plasma.
By Nikita Churaev.

Updated Dec. 31, 2011, 5:18 a.m.

Description

Right-aligned popups are one pixel away from right edge of the screen and top-aligned popups (when the panel is on top) are one pixel inside panel. This is because the bug in QRect, where right() and bottom() value return value that is less than the true value by one. This is a feature-bug that Qt developers aren't going to fix because of compatiability reasons. My patch replaces rect.right() and rect.bottom() with (rect.x() + rect.width()) and (rect.y() + rect.height()) respectively.

Testing

Works as expected.

Diffs

  • plasma/corona.cpp (366a9df)

View Diff