minor fixes regarding inactive columns

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

diff --git a/src/dolphincolumnview.cpp b/src/dolphincolumnview.cpp
index 1268064156b6d87cbc4b7fb29377b7244b210a0c..3389e15bf02e4151b8885e156c7062ca41b6dbd1 100644 (file)
--- a/src/dolphincolumnview.cpp
+++ b/src/dolphincolumnview.cpp
@@ -69,6 +69,9 @@ protected:
     virtual void paintEvent(QPaintEvent* event);
     virtual void contextMenuEvent(QContextMenuEvent* event);
 
     virtual void paintEvent(QPaintEvent* event);
     virtual void contextMenuEvent(QContextMenuEvent* event);
 

+protected slots:
+    virtual void selectionChanged(const QItemSelection& selected, const QItemSelection& deselected);
+

 private:
     /** Used by ColumnWidget::setActive(). */
     void activate();
 private:
     /** Used by ColumnWidget::setActive(). */
     void activate();


@@ -204,9 +207,6 @@ void ColumnWidget::dropEvent(QDropEvent* event)
 
 void ColumnWidget::mousePressEvent(QMouseEvent* event)
 {
 
 void ColumnWidget::mousePressEvent(QMouseEvent* event)
 {

-    QListView::mousePressEvent(event);
-
-    bool requestActivation = false;

     const QModelIndex index = indexAt(event->pos());
     if (index.isValid()) {
         // A click on an item has been done. Only request an activation
     const QModelIndex index = indexAt(event->pos());
     if (index.isValid()) {
         // A click on an item has been done. Only request an activation


@@ -218,22 +218,21 @@ void ColumnWidget::mousePressEvent(QMouseEvent* event)
         if (item != 0) {
             if (item->isDir()) {
                 m_childUrl = item->url();
         if (item != 0) {
             if (item->isDir()) {
                 m_childUrl = item->url();

+                viewport()->update();

            } else {
            } else {

-                requestActivation = true;
+                m_view->requestActivation(this);

             }
         }
     } else {
         // a click on the viewport has been done
             }
         }
     } else {
         // a click on the viewport has been done

-        requestActivation = true;
+        m_view->requestActivation(this);

 
         // Swallow mouse move events if a click is done on the viewport. Otherwise the QColumnView
         // triggers an unwanted loading of directories on hovering folder items.
         m_swallowMouseMoveEvents = true;
     }
 
 
         // Swallow mouse move events if a click is done on the viewport. Otherwise the QColumnView
         // triggers an unwanted loading of directories on hovering folder items.
         m_swallowMouseMoveEvents = true;
     }
 

-    if (requestActivation) {
-        m_view->requestActivation(this);
-    }
+    QListView::mousePressEvent(event);

 }
 
 void ColumnWidget::mouseMoveEvent(QMouseEvent* event)
 }
 
 void ColumnWidget::mouseMoveEvent(QMouseEvent* event)


@@ -253,23 +252,25 @@ void ColumnWidget::mouseReleaseEvent(QMouseEvent* event)
 
 void ColumnWidget::paintEvent(QPaintEvent* event)
 {
 
 void ColumnWidget::paintEvent(QPaintEvent* event)
 {

-    // indicate the shown URL of the next column by highlighting the shown folder item
-    const QAbstractProxyModel* proxyModel = static_cast<const QAbstractProxyModel*>(m_view->model());
-    const KDirModel* dirModel = static_cast<const KDirModel*>(proxyModel->sourceModel());
-    const QModelIndex dirIndex = dirModel->indexForUrl(m_childUrl);
-    const QModelIndex proxyIndex = proxyModel->mapFromSource(dirIndex);
-    if (proxyIndex.isValid()) {
-        const QRect itemRect = visualRect(proxyIndex);
-        QPainter painter(viewport());
-        painter.save();
-
-        QColor color = KColorScheme(KColorScheme::View).foreground();
-        color.setAlpha(32);
-        painter.setPen(Qt::NoPen);
-        painter.setBrush(color);
-        painter.drawRect(itemRect);
-
-        painter.restore();
+    if (!m_childUrl.isEmpty()) {
+        // indicate the shown URL of the next column by highlighting the shown folder item
+        const QAbstractProxyModel* proxyModel = static_cast<const QAbstractProxyModel*>(m_view->model());
+        const KDirModel* dirModel = static_cast<const KDirModel*>(proxyModel->sourceModel());
+        const QModelIndex dirIndex = dirModel->indexForUrl(m_childUrl);
+        const QModelIndex proxyIndex = proxyModel->mapFromSource(dirIndex);
+        if (proxyIndex.isValid() && !selectionModel()->isSelected(proxyIndex)) {
+            const QRect itemRect = visualRect(proxyIndex);
+            QPainter painter(viewport());
+            painter.save();
+
+            QColor color = KColorScheme(KColorScheme::View).foreground();
+            color.setAlpha(32);
+            painter.setPen(Qt::NoPen);
+            painter.setBrush(color);
+            painter.drawRect(itemRect);
+
+            painter.restore();
+        }

     }
 
     QListView::paintEvent(event);
     }
 
     QListView::paintEvent(event);


@@ -298,6 +299,15 @@ void ColumnWidget::contextMenuEvent(QContextMenuEvent* event)
     }
 }
 
     }
 }
 

+void ColumnWidget::selectionChanged(const QItemSelection& selected, const QItemSelection& deselected)
+{
+    // inactive views should not have any selection
+    if (!m_active) {
+        clearSelection();
+    }
+    QListView::selectionChanged(selected, deselected);
+}
+

 void ColumnWidget::activate()
 {
     const QColor bgColor = KColorScheme(KColorScheme::View).background();
 void ColumnWidget::activate()
 {
     const QColor bgColor = KColorScheme(KColorScheme::View).background();