]> cloud.milkyroute.net Git - dolphin.git/commitdiff
Fix issue that the horizontal scrollbar is not updated correctly when resizing the...
authorPeter Penz <peter.penz19@gmail.com>
Fri, 5 Nov 2010 15:31:23 +0000 (15:31 +0000)
committerPeter Penz <peter.penz19@gmail.com>
Fri, 5 Nov 2010 15:31:23 +0000 (15:31 +0000)
svn path=/trunk/KDE/kdebase/apps/; revision=1193329

src/views/dolphincolumnviewcontainer.cpp

index 7346532a16f4274e380ccdd0e4a0093eb0080005..31ec21083640d02b1622a4f23719a7f4adb4a87a 100644 (file)
@@ -331,15 +331,12 @@ void DolphinColumnViewContainer::layoutColumns()
         contentWidth += column->maximumWidth();
     }
 
-    if (horizontalScrollBar()->pageStep() != contentWidth) {
-        disconnect(horizontalScrollBar(), SIGNAL(valueChanged(int)),
-                this, SLOT(moveContentHorizontally(int)));
-
+    const int scrollBarMax = contentWidth - viewport()->width();
+    const bool updateScrollBar =    (horizontalScrollBar()->pageStep() != contentWidth)
+                                 || (horizontalScrollBar()->maximum()  != scrollBarMax);
+    if (updateScrollBar) {
         horizontalScrollBar()->setPageStep(contentWidth);
-        horizontalScrollBar()->setRange(0, contentWidth - viewport()->width());
-
-        connect(horizontalScrollBar(), SIGNAL(valueChanged(int)),
-                this, SLOT(moveContentHorizontally(int)));
+        horizontalScrollBar()->setRange(0, scrollBarMax);
     }
 }
 
@@ -351,13 +348,13 @@ void DolphinColumnViewContainer::requestActivation(DolphinColumnView* column)
     if (focusProxy() != column) {
         setFocusProxy(column);
     }
-    
+
     if (!column->isActive()) {
         // Deactivate the currently active column
         if (m_index >= 0) {
             m_columns[m_index]->setActive(false);
         }
-        
+
         // Get the index of the column that should get activated
         int index = 0;
         foreach (DolphinColumnView* currColumn, m_columns) {