X-Git-Url: https://cloud.milkyroute.net/gitweb/dolphin.git/blobdiff_plain/5e73577d6c3520fa3dfd63646795e628cf2a0614..60d555fa55f025f3ecaa82cf95f012cbbbeb8ddc:/src/tooltips/tooltipmanager.cpp

diff --git a/src/tooltips/tooltipmanager.cpp b/src/tooltips/tooltipmanager.cpp
index 489c6c2b6..ae514f896 100644
--- a/src/tooltips/tooltipmanager.cpp
+++ b/src/tooltips/tooltipmanager.cpp
@@ -50,12 +50,9 @@ ToolTipManager::ToolTipManager(QAbstractItemView* parent,
     m_waitOnPreviewTimer(0),
     m_item(),
     m_itemRect(),
-    m_preview(false),
     m_generatingPreview(false),
-    m_previewIsLate(false),
-    m_previewPass(0),
-    m_emptyRenderedKToolTipItem(0),
-    m_pix()
+    m_hasDefaultIcon(false),
+    m_previewPixmap()
 {
     KToolTip::setToolTipDelegate(g_delegate);
 
@@ -127,9 +124,8 @@ void ToolTipManager::requestToolTip(const QModelIndex& index)
         // only start the previewJob when the mouse has been over this item for 200 milliseconds,
         // this prevents a lot of useless preview jobs when passing rapidly over a lot of items
         m_previewTimer->start(200);
-        m_preview = false;
-        m_previewIsLate = false;
-        m_previewPass = 0;
+        m_previewPixmap = QPixmap();
+        m_hasDefaultIcon = false;
 
         m_timer->start(500);
     } else {
@@ -142,64 +138,33 @@ void ToolTipManager::hideToolTip()
     m_timer->stop();
     m_previewTimer->stop();
     m_waitOnPreviewTimer->stop();
-    m_previewIsLate = false;
     KToolTip::hideTip();
 }
 
 void ToolTipManager::prepareToolTip()
 {
     if (m_generatingPreview) {
-        if (m_previewPass == 1) {
-            // We waited 250msec and the preview is still not finished,
-            // so show the toolTip with a transparent image of maximal width.
-            // When the preview finishes, m_previewIsLate will cause
-            // a direct update of the tooltip, via m_emptyRenderedKToolTipItem.
-            QPixmap paddedImage(QSize(PREVIEW_WIDTH, 32));
-            m_previewIsLate = true;
-            paddedImage.fill(Qt::transparent);
-            KToolTipItem* toolTip = new KToolTipItem(paddedImage, m_item.getToolTipText());
-            m_emptyRenderedKToolTipItem = toolTip; // make toolTip accessible everywhere
-            showToolTip(toolTip);
-        }
-
-        ++m_previewPass;
         m_waitOnPreviewTimer->start(250);
-    } else {
-        // The preview generation has finished, find out under which circumstances.
-        if (m_preview && m_previewIsLate) {
-            // We got a preview, but it is late, the tooltip has already been shown.
-            // So update the tooltip directly.
-            if (m_emptyRenderedKToolTipItem != 0) {
-                m_emptyRenderedKToolTipItem->setData(Qt::DecorationRole, KIcon(m_pix));
-            }
-            return;
-        }
-
-        KIcon icon;
-        if (m_preview) {
-            // We got a preview.
-            icon = KIcon(m_pix);
-        } else {
-            // No preview, so use an icon.
-            // Force a 128x128 icon, a 256x256 one is far too big.
-            icon = KIcon(KIcon(m_item.iconName()).pixmap(ICON_WIDTH, ICON_HEIGHT));
-        }
+    }
 
-        KToolTipItem* toolTip = new KToolTipItem(icon, m_item.getToolTipText());
-        showToolTip(toolTip);
+    const QString text = m_item.getToolTipText();
+    if (!m_previewPixmap.isNull()) {
+        showToolTip(KIcon(m_previewPixmap), text);
+    } else if (!m_hasDefaultIcon) {
+        const QPixmap image(KIcon(m_item.iconName()).pixmap(ICON_WIDTH, ICON_HEIGHT));
+        showToolTip(image, text);
+        m_hasDefaultIcon = true;
     }
 }
 
-void ToolTipManager::showToolTip(KToolTipItem* tip)
+void ToolTipManager::showToolTip(const QIcon& icon, const QString& text)
 {
     if (QApplication::mouseButtons() & Qt::LeftButton) {
-        delete tip;
-        tip = 0;
-        // m_emptyRenderedKToolTipItem is an alias for tip.
-        m_emptyRenderedKToolTipItem = 0;
         return;
     }
 
+    KToolTipItem* tip = new KToolTipItem(icon, text);
+
     KStyleOptionToolTip option;
     // TODO: get option content from KToolTip or add KToolTip::sizeHint() method
     option.direction      = QApplication::layoutDirection();
@@ -211,15 +176,7 @@ void ToolTipManager::showToolTip(KToolTipItem* tip)
     option.state          = QStyle::State_None;
     option.decorationSize = QSize(32, 32);
 
-    QSize size;
-    if (m_previewIsLate) {
-        QPixmap paddedImage(QSize(PREVIEW_WIDTH, PREVIEW_HEIGHT));
-        KToolTipItem maxiTip(paddedImage, m_item.getToolTipText());
-        size = g_delegate->sizeHint(option, maxiTip);
-    }
-    else if (tip != 0) {
-        size = g_delegate->sizeHint(option, *tip);
-    }
+    const QSize size = g_delegate->sizeHint(option, *tip);
     const QRect desktop = QApplication::desktop()->screenGeometry(m_itemRect.bottomRight());
 
     // m_itemRect defines the area of the item, where the tooltip should be
@@ -254,6 +211,7 @@ void ToolTipManager::showToolTip(KToolTipItem* tip)
         y = desktop.bottom() - size.height();
     }
 
+    // the ownership of tip is transferred to KToolTip
     KToolTip::showTip(QPoint(x, y), tip);
 }
 
@@ -269,35 +227,24 @@ void ToolTipManager::startPreviewJob()
     connect(job, SIGNAL(gotPreview(const KFileItem&, const QPixmap&)),
             this, SLOT(setPreviewPix(const KFileItem&, const QPixmap&)));
     connect(job, SIGNAL(failed(const KFileItem&)),
-            this, SLOT(previewFailed(const KFileItem&)));
+            this, SLOT(previewFailed()));
 }
 
 
 void ToolTipManager::setPreviewPix(const KFileItem& item,
                                    const QPixmap& pixmap)
 {
-    if (m_item.url() != item.url()) {
-        m_generatingPreview = false;
-        return;
-    }
-
-    if (m_previewIsLate) {
-        // always use the maximal width
-        QPixmap paddedImage(QSize(PREVIEW_WIDTH, pixmap.height()));
-        paddedImage.fill(Qt::transparent);
-        QPainter painter(&paddedImage);
-        painter.drawPixmap((PREVIEW_WIDTH - pixmap.width()) / 2, 0, pixmap);
-        m_pix = paddedImage;
+    if ((m_item.url() != item.url()) || pixmap.isNull()) {
+        // an old preview or an invalid preview has been received
+        previewFailed();
     } else {
-        m_pix = pixmap;
+        m_previewPixmap = pixmap;
+        m_generatingPreview = false;
     }
-    m_preview = true;
-    m_generatingPreview = false;
 }
 
-void ToolTipManager::previewFailed(const KFileItem& item)
+void ToolTipManager::previewFailed()
 {
-    Q_UNUSED(item);
     m_generatingPreview = false;
 }