----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102671/#review6772 -----------------------------------------------------------
kdeui/dialogs/kdialog.cpp <http://git.reviewboard.kde.org/r/102671/#comment6004> Are you sure it should not be availableGeometry(widget) so it is consistant with the other case? (or maybe the other case is wrong as well, wr really want to center on the center of the screen, not its available geometry). - Olivier On Sept. 23, 2011, 12:23 p.m., Thomas Gahr wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/102671/ > ----------------------------------------------------------- > > (Updated Sept. 23, 2011, 12:23 p.m.) > > > Review request for kdelibs. > > > Summary > ------- > > This patch changes KDialog::screenRect to call > QDesktopWidget::screenGeometry(widget) instead of QDesktopWidget::geometry(), > which returns the wrong geometry if a second monitor has been connected and > disconnected. > This fixes a bug described here: > http://article.gmane.org/gmane.comp.kde.devel.core/71875 > The testcase and results can be found here: > http://article.gmane.org/gmane.comp.kde.devel.core/71911 > > Behaviour of QDesktopWidget::geometry() vs. > QDesktopWidget::screenGeometry(widget) are equivalent in case of non-faulty > ::geometry() and fix the problem in case of faulty ::geometry(). So I > wouldn't consider this a workaround that needs to be maintained/changed back > once (if) the bug in Qt has been fixed. On the contrary: using > ::screenGeometry(widget) is more verbose than ::geometry(), which make more > sense in the context of KDialog::screenRect(QWidget* widget,int screen) IMHO > > > Diffs > ----- > > kdeui/dialogs/kdialog.cpp 0cabb85 > > Diff: http://git.reviewboard.kde.org/r/102671/diff > > > Testing > ------- > > Tested successfully with 4.7.0 on Fedora 15 and current master... and on > branch KDE/4.7 - as expected everything works fine > > > Thanks, > > Thomas > >
