diff options
| author | Frank Reininghaus <[email protected]> | 2011-08-14 11:05:19 +0200 |
|---|---|---|
| committer | Frank Reininghaus <[email protected]> | 2011-08-14 11:05:19 +0200 |
| commit | 7b911df33ec83bb978985d50342ce1c27afb6425 (patch) | |
| tree | 54f39f397d436d491ac4a79eafefb34d7ea3824c /src/views/dolphinview.cpp | |
| parent | 98e859b0c31b0071a5bda673035e0ba39e7169bb (diff) | |
Two small selection improvements in DolphinView
1. Implement DolphinView::clearSelection().
2. Simplify DolphinView::invertSelection().
I found, fixed, and unit-tested a bug in the selection
manager which was uncovered by this change.
Diffstat (limited to 'src/views/dolphinview.cpp')
| -rw-r--r-- | src/views/dolphinview.cpp | 16 |
1 files changed, 3 insertions, 13 deletions
diff --git a/src/views/dolphinview.cpp b/src/views/dolphinview.cpp index f429529de..de54b9bb6 100644 --- a/src/views/dolphinview.cpp +++ b/src/views/dolphinview.cpp @@ -529,23 +529,13 @@ void DolphinView::selectAll() void DolphinView::invertSelection() { - KItemListSelectionManager* selectionManager = m_container->controller()->selectionManager(); - const QSet<int> selectedItems = selectionManager->selectedItems(); - QSet<int> invertedSelectedItems; - - const int maxIndex = fileItemModel()->count() - 1; - for (int i = 0; i <= maxIndex; ++i) { - if (!selectedItems.contains(i)) { - invertedSelectedItems.insert(i); - } - } - - selectionManager->setSelectedItems(invertedSelectedItems); + KItemListSelectionManager* selectionManager = m_container->controller()->selectionManager(); + selectionManager->setSelected(0, fileItemModel()->count(), KItemListSelectionManager::Toggle); } void DolphinView::clearSelection() { - //m_viewAccessor.itemView()->clearSelection(); + m_container->controller()->selectionManager()->clearSelection(); } void DolphinView::renameSelectedItems() |
