Stupid me: The inconsistent behavior between QListView::scrollTo() and QTreeView::scrollTo() has been fixed in Qt 4.5 :-) I should have tested this _before_ committing the huge patch for bug 185191... So:

* Remove the workaround in DolphinIconsView::scrollTo() to bypass an issue in Qt 4.4 (-> fixed in Qt 4.5)
* Revert the "use the DolphinController"-parts of commit 930754 and just use QAbstractItemView::scrollTo() instead.

CCBUG: 185191

svn path=/trunk/KDE/kdebase/apps/; revision=930777

1 files changed, 0 insertions, 13 deletions

diff --git a/src/dolphincontroller.h b/src/dolphincontroller.h
index ba1036060..29dd7dfb1 100644
--- a/src/dolphincontroller.h
+++ b/src/dolphincontroller.h
@@ -73,7 +73,6 @@ class QWidget;
  * - setShowPreview()
  * - indicateActivationChange()
  * - setZoomLevel()
- * - triggerScrollToCurrentItem()
  */
 class LIBDOLPHINPRIVATE_EXPORT DolphinController : public QObject
 {
@@ -191,12 +190,6 @@ public:
     int zoomLevel() const;
 
     /**
-     * Triggers the view implementation to assure having a fully visible
-     * current item. The signal scrollToCurrentItem() will be emitted.
-     */
-    void triggerScrollToCurrentItem();
-
-    /**
      * Tells the view implementation to zoom out by emitting the signal zoomOut()
      * and is invoked by the abstract Dolphin view.
      */
@@ -376,12 +369,6 @@ signals:
      */
     void hideToolTip();
 
-    /**
-     * Is emitted if the view implementation should scroll to the current item, so
-     * that it is fully visible.
-     */
-    void scrollToCurrentItem();
-
 private slots:
     void updateMouseButtonState();