From: Peter Penz <peter.penz19@gmail.com>
Date: Fri, 22 Feb 2008 10:08:11 +0000 (+0000)
Subject: SVN_SILENT: updated comment with the Qt-bugtracker ID
X-Git-Url: https://cloud.milkyroute.net/gitweb/dolphin.git/commitdiff_plain/1abc518e7a492917ffaca4de880e641eb1bf406c

SVN_SILENT: updated comment with the Qt-bugtracker ID

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

diff --git a/src/dolphindetailsview.cpp b/src/dolphindetailsview.cpp
index 678343503..e80b75119 100644
--- a/src/dolphindetailsview.cpp
+++ b/src/dolphindetailsview.cpp
@@ -237,30 +237,17 @@ void DolphinDetailsView::mouseMoveEvent(QMouseEvent* event)
             setSelection(selRect, QItemSelectionModel::Select);
         }
 
-        // TODO: see comment at end of method
+        // TODO: enable QTreeView::mouseMoveEvent(event) again, as soon
+        // as the Qt-issue #199631 has been fixed.
         // QTreeView::mouseMoveEvent(event);
         QAbstractItemView::mouseMoveEvent(event);
         updateElasticBand();
     } else {
-        // TODO: see comment at end of method
-        //QTreeView::mouseMoveEvent(event);
+        // TODO: enable QTreeView::mouseMoveEvent(event) again, as soon
+        // as the Qt-issue #199631 has been fixed.
+        // QTreeView::mouseMoveEvent(event);
         QAbstractItemView::mouseMoveEvent(event);
     }
-
-    // The original implementation of QTreeView::mouseMoveEvent() looks like this (Qt4.4):
-    //
-    // void QTreeView::mouseMoveEvent(QMouseEvent *event)
-    // {
-    //     Q_D(QTreeView);
-    //     if (d->itemDecorationAt(event->pos()) == -1) // ### what about expanding/collapsing state ?
-    //         QAbstractItemView::mouseMoveEvent(event);
-    // }
-    //
-    // This prevents that the signal 'entered()' is emitted when the mouse is above a decoration,
-    // although the hovered item has changed. The SelectionManager is connected to 'entered()'
-    // and assumes that the old item is selected. This is currently bypassed by skipping
-    // the base implementation and invoking QAbstractItemView::mouseMoveEvent() directly.
-    // Submitted a bug report to Trolltech, bug ID is still pending.
 }
 
 void DolphinDetailsView::mouseReleaseEvent(QMouseEvent* event)