After renaming an item the view should be scrolled in a way to still have a fully visible renamed item. The implementation required a lot of more code changes as such a fix should require: QAbstractItemView::scrollTo() cannot be used directly (inconsistent default behavior in QListView and QTreeView, a special case for the column view), so the communication has to be done with the DolphinController...

BUG: 185191

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

1 files changed, 13 insertions, 0 deletions

diff --git a/src/dolphincontroller.h b/src/dolphincontroller.h
index 29dd7dfb1..ba1036060 100644
--- a/src/dolphincontroller.h
+++ b/src/dolphincontroller.h
@@ -73,6 +73,7 @@ class QWidget;
  * - setShowPreview()
  * - indicateActivationChange()
  * - setZoomLevel()
+ * - triggerScrollToCurrentItem()
  */
 class LIBDOLPHINPRIVATE_EXPORT DolphinController : public QObject
 {
@@ -190,6 +191,12 @@ 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.
      */
@@ -369,6 +376,12 @@ 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();