don't reset the x-position to 0 in the Column View, if a new directory is loaded

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

diff --git a/src/dolphinview.cpp b/src/dolphinview.cpp
index 65ec4c9d305f7da815d676c798b5c5c5c56dca65..d69626f65f645715bfb5a657be8ef788e3499a40 100644 (file)
--- a/src/dolphinview.cpp
+++ b/src/dolphinview.cpp
@@ -61,7 +61,7 @@ DolphinView::DolphinView(QWidget* parent,
                          Mode mode) :
     QWidget(parent),
     m_active(true),
-    m_blockContentsMovedSignal(false),
+    m_loadingDirectory(false),
     m_initializeColumnView(false),
     m_mode(mode),
     m_topLayout(0),
@@ -337,7 +337,7 @@ void DolphinView::setContentsPosition(int x, int y)
     view->horizontalScrollBar()->setValue(x);
     view->verticalScrollBar()->setValue(y);
 
-    m_blockContentsMovedSignal = false;
+    m_loadingDirectory = false;
 }
 
 QPoint DolphinView::contentsPosition() const
@@ -541,7 +541,7 @@ void DolphinView::startDirLister(const KUrl& url, bool reload)
     }
 
     m_cutItemsCache.clear();
-    m_blockContentsMovedSignal = true;
+    m_loadingDirectory = true;
 
     m_dirLister->stop();
 
@@ -754,7 +754,12 @@ void DolphinView::updateSortOrder(Qt::SortOrder order)
 
 void DolphinView::emitContentsMoved()
 {
-    if (!m_blockContentsMovedSignal) {
+    // only emit the contents moved signal if:
+    // - no directory loading is ongoing (this would reset the contents position
+    //   always to (0, 0))
+    // - if the Column View is active: the column view does an automatic
+    //   positioning during the loading operation, which must be remembered
+    if (!m_loadingDirectory || isColumnViewActive()) {
         const QPoint pos(contentsPosition());
         emit contentsMoved(pos.x(), pos.y());
     }

diff --git a/src/dolphinview.h b/src/dolphinview.h
index ebca53e99ae400da8696fdcb0ad43e068bfc3eb6..3d0f9a548ddb13ce361d280d28c9ce9bc882ed71 100644 (file)
--- a/src/dolphinview.h
+++ b/src/dolphinview.h
@@ -527,7 +527,7 @@ private:
     };
 
     bool m_active;
-    bool m_blockContentsMovedSignal;
+    bool m_loadingDirectory;
     bool m_initializeColumnView;
     Mode m_mode;