From 5ba015fa56a2ae7e5b606ae9b687ce99ac2bc746 Mon Sep 17 00:00:00 2001 From: Rob Scheepmaker Date: Thu, 4 Dec 2008 23:57:47 +0000 Subject: [PATCH] Removed obsolete call to updateDialogPosition, and use this function in dialogSizeChanged() for consistent dialog position. This solves the problem of the dialog being moved twice when the size of the widget changes. svn path=/trunk/KDE/kdelibs/; revision=892689 --- popupapplet.cpp | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/popupapplet.cpp b/popupapplet.cpp index cf44ea69e..6470b6e8c 100644 --- a/popupapplet.cpp +++ b/popupapplet.cpp @@ -357,7 +357,6 @@ void PopupApplet::showPopup(uint popupDuration) { if (d->dialog && (formFactor() == Horizontal || formFactor() == Vertical)) { d->dialog->show(); - d->updateDialogPosition(); KWindowSystem::setState(d->dialog->winId(), NET::SkipTaskbar | NET::SkipPager); if (d->timer) { @@ -476,8 +475,7 @@ void PopupAppletPrivate::dialogSizeChanged() { //Reposition the dialog if (dialog) { - //dialog->updateGeometry(); - dialog->move(q->popupPosition(dialog->size())); + updateDialogPosition(); KConfigGroup sizeGroup = q->config(); sizeGroup = KConfigGroup(&sizeGroup, "PopupApplet"); @@ -503,7 +501,7 @@ void PopupAppletPrivate::updateDialogPosition() KConfigGroup sizeGroup = q->config(); sizeGroup = KConfigGroup(&sizeGroup, "PopupApplet"); - + Q_ASSERT(q->containment()); Q_ASSERT(q->containment()->corona()); const int width = qMin(sizeGroup.readEntry("DialogWidth", 0),