From: Peter Penz <peter.penz19@gmail.com>
Date: Sat, 15 Dec 2007 16:48:48 +0000 (+0000)
Subject: Fixed issue that Ctrl+A inverted the selection instead doing a "Select All". It seems... 
X-Git-Url: https://cloud.milkyroute.net/gitweb/dolphin.git/commitdiff_plain/ceafa5cc22d0450e2a0d27023c62f0d339c3ca10

Fixed issue that Ctrl+A inverted the selection instead doing a "Select All". It seems like a Qt issue, but I'm not 100 % sure - I'll try to make this reproducible with a Qt-only code after KDE 4.0 and submit a bug report to Trolltech...

BUG: 153938

svn path=/trunk/KDE/kdebase/apps/; revision=748823
---

diff --git a/src/dolphinview.cpp b/src/dolphinview.cpp
index 4db66cd4b..8502bfd22 100644
--- a/src/dolphinview.cpp
+++ b/src/dolphinview.cpp
@@ -320,7 +320,13 @@ bool DolphinView::supportsCategorizedSorting() const
 
 void DolphinView::selectAll()
 {
-    itemView()->selectAll();
+    QAbstractItemView* view = itemView();
+    // TODO: there seems to be a bug in QAbstractItemView::selectAll(); if
+    // the Ctrl-key is pressed (e. g. for Ctrl+A), selectAll() inverts the
+    // selection instead of selecting all items. This is bypassed for KDE 4.0
+    // by invoking clearSelection() first.
+    view->clearSelection();
+    view->selectAll();
 }
 
 void DolphinView::invertSelection()