From: Peter Penz <peter.penz19@gmail.com>
Date: Thu, 14 Feb 2008 22:57:00 +0000 (+0000)
Subject: Bypass a temporary (?) regression (?) with the current Qt 4.4 beta version: it seems... 
X-Git-Url: https://cloud.milkyroute.net/gitweb/dolphin.git/commitdiff_plain/613a132a3697db603bca593a36f91ee3f9ccafdd?ds=inline

Bypass a temporary (?) regression (?) with the current Qt 4.4 beta version: it seems like QLayout::activate()/update() don't do a relayout anymore if the sizehint of a sub widget has been changed. I'll check this again with Qt 4.4.0.

svn path=/trunk/KDE/kdebase/apps/; revision=775119
---

diff --git a/src/infosidebarpage.cpp b/src/infosidebarpage.cpp
index 30c4467b4..943033876 100644
--- a/src/infosidebarpage.cpp
+++ b/src/infosidebarpage.cpp
@@ -172,6 +172,17 @@ void InfoSidebarPage::resizeEvent(QResizeEvent* event)
 
     // try to increase the preview as large as possible
     m_preview->setSizeHint(QSize(maxWidth, maxWidth));
+
+    // ---
+    // TODO: The following workaround was not necessary with Qt 4.3.x
+    // but is mandatory with the current Qt4.4 beta version. Check with
+    // Qt 4.4.0 whether the issue is Dolphin specific
+    // (see PixmapViewer::setSizeHint()) or a regression in Qt.
+    QVBoxLayout* vBox = static_cast<QVBoxLayout*>(layout());
+    vBox->removeWidget(m_preview);
+    vBox->insertWidget(0, m_preview);
+    // ---
+
     m_urlCandidate = m_shownUrl; // reset the URL candidate if a resizing is done
     m_timer->start(TimerDelay);