]> cloud.milkyroute.net Git - dolphin.git/blobdiff - src/kcategorizedview.cpp
projects / dolphin.git / blobdiff
? search:
summary | shortlog | log | commit | commitdiff | tree
raw | inline | side by side
Improve the categorized view for the list mode
[dolphin.git] / src / kcategorizedview.cpp
diff --git a/src/kcategorizedview.cpp b/src/kcategorizedview.cpp
index 8a0b3c8c570eca766407409f1e657c17ca7acbd7..217fef10fe899f12a86816d04d406d3ce95a25ce 100644 (file)
--- a/src/kcategorizedview.cpp
+++ b/src/kcategorizedview.cpp
@@ -126,15 +126,23 @@ QRect KCategorizedView::Private::visualRectInViewport(const QModelIndex &index)
 
     QRect retRect;
 
-    if (listView->layoutDirection() == Qt::LeftToRight)
+    if (listView->flow() == QListView::LeftToRight)
     {
-        retRect = QRect(listView->spacing(), listView->spacing() * 2 +
-                        categoryDrawer->categoryHeight(listView->viewOptions()), 0, 0);
+        if (listView->layoutDirection() == Qt::LeftToRight)
+        {
+            retRect = QRect(listView->spacing(), listView->spacing() * 2 +
+                            categoryDrawer->categoryHeight(index, listView->viewOptions()), 0, 0);
+        }
+        else
+        {
+            retRect = QRect(listView->viewport()->width() - listView->spacing(), listView->spacing() * 2 +
+                            categoryDrawer->categoryHeight(index, listView->viewOptions()), 0, 0);
+        }
     }
     else
     {
-        retRect = QRect(listView->viewport()->width() - listView->spacing(), listView->spacing() * 2 +
-                        categoryDrawer->categoryHeight(listView->viewOptions()), 0, 0);
+        retRect = QRect(listView->spacing(), listView->spacing() * 2 +
+                        categoryDrawer->categoryHeight(index, listView->viewOptions()), 0, 0);
     }
 
     int viewportWidth = listView->viewport()->width() - listView->spacing();
@@ -142,37 +150,61 @@ QRect KCategorizedView::Private::visualRectInViewport(const QModelIndex &index)
     int itemHeight;
     int itemWidth;
 
-    if (listView->gridSize().isEmpty())
+    if (listView->gridSize().isEmpty() && (listView->flow() == QListView::LeftToRight))
     {
         itemHeight = biggestItemSize.height();
         itemWidth = biggestItemSize.width();
     }
-    else
+    else if (listView->flow() == QListView::LeftToRight)
     {
         itemHeight = listView->gridSize().height();
         itemWidth = listView->gridSize().width();
     }
+    else if (listView->gridSize().isEmpty() && (listView->flow() == QListView::TopToBottom))
+    {
+        itemHeight = biggestItemSize.height();
+        itemWidth = listView->viewport()->width() - listView->spacing() * 2;
+    }
+    else
+    {
+        itemHeight = listView->gridSize().height();
+        itemWidth = listView->gridSize().width() - listView->spacing() * 2;
+    }
 
     int itemWidthPlusSeparation = listView->spacing() + itemWidth;
+    if (!itemWidthPlusSeparation)
+        itemWidthPlusSeparation++;
     int elementsPerRow = viewportWidth / itemWidthPlusSeparation;
     if (!elementsPerRow)
         elementsPerRow++;
 
-    int column = elementsInfo[index.row()].relativeOffsetToCategory % elementsPerRow;
-    int row = elementsInfo[index.row()].relativeOffsetToCategory / elementsPerRow;
+    int column;
+    int row;
 
-    if (listView->layoutDirection() == Qt::LeftToRight)
+    if (listView->flow() == QListView::LeftToRight)
     {
-        retRect.setLeft(retRect.left() + column * listView->spacing() +
-                        column * itemWidth);
+        column = elementsInfo[index.row()].relativeOffsetToCategory % elementsPerRow;
+        row = elementsInfo[index.row()].relativeOffsetToCategory / elementsPerRow;
+
+        if (listView->layoutDirection() == Qt::LeftToRight)
+        {
+            retRect.setLeft(retRect.left() + column * listView->spacing() +
+                            column * itemWidth);
+        }
+        else
+        {
+            retRect.setLeft(retRect.right() - column * listView->spacing() -
+                            column * itemWidth - itemWidth);
+
+            retRect.setRight(retRect.right() - column * listView->spacing() -
+                            column * itemWidth);
+        }
     }
     else
     {
-        retRect.setLeft(retRect.right() - column * listView->spacing() -
-                         column * itemWidth - itemWidth);
-
-        retRect.setRight(retRect.right() - column * listView->spacing() -
-                         column * itemWidth);
+        elementsPerRow = 1;
+        column = elementsInfo[index.row()].relativeOffsetToCategory % elementsPerRow;
+        row = elementsInfo[index.row()].relativeOffsetToCategory / elementsPerRow;
     }
 
     foreach (const QString &category, categories)
@@ -189,7 +221,7 @@ QRect KCategorizedView::Private::visualRectInViewport(const QModelIndex &index)
 
         retRect.setTop(retRect.top() +
                        (rowsInt * itemHeight) +
-                       categoryDrawer->categoryHeight(listView->viewOptions()) +
+                       categoryDrawer->categoryHeight(index, listView->viewOptions()) +
                        listView->spacing() * 2);
 
         if (listView->gridSize().isEmpty())
@@ -225,8 +257,7 @@ QRect KCategorizedView::Private::visualRectInViewport(const QModelIndex &index)
     return retRect;
 }
 
-QRect KCategorizedView::Private::visualCategoryRectInViewport(const QString &category)
-                                                                           const
+QRect KCategorizedView::Private::visualCategoryRectInViewport(const QString &category) const
 {
     QRect retRect(listView->spacing(),
                   listView->spacing(),
@@ -260,6 +291,11 @@ QRect KCategorizedView::Private::visualCategoryRectInViewport(const QString &cat
     if (!elementsPerRow)
         elementsPerRow++;
 
+    if (listView->flow() == QListView::TopToBottom)
+    {
+        elementsPerRow = 1;
+    }
+
     foreach (const QString &itCategory, categories)
     {
         if (itCategory == category)
@@ -273,7 +309,7 @@ QRect KCategorizedView::Private::visualCategoryRectInViewport(const QString &cat
 
         retRect.setTop(retRect.top() +
                        (rowsInt * itemHeight) +
-                       categoryDrawer->categoryHeight(listView->viewOptions()) +
+                       categoryDrawer->categoryHeight(index, listView->viewOptions()) +
                        listView->spacing() * 2);
 
         if (listView->gridSize().isEmpty())
@@ -283,7 +319,7 @@ QRect KCategorizedView::Private::visualCategoryRectInViewport(const QString &cat
         }
     }
 
-    retRect.setHeight(categoryDrawer->categoryHeight(listView->viewOptions()));
+    retRect.setHeight(categoryDrawer->categoryHeight(index, listView->viewOptions()));
 
     return retRect;
 }
@@ -396,7 +432,7 @@ void KCategorizedView::Private::updateScrollbars()
     QModelIndex lastIndex = categoriesIndexes.isEmpty() ? QModelIndex() : categoriesIndexes[categories.last()].last();
 
     int lastItemBottom = cachedRectIndex(lastIndex).top() +
-                         listView->spacing() + (listView->gridSize().isEmpty() ? 0 : listView->gridSize().height()) - listView->viewport()->height();
+                         listView->spacing() + (listView->gridSize().isEmpty() ? biggestItemSize.height() : listView->gridSize().height()) - listView->viewport()->height();
 
     listView->horizontalScrollBar()->setRange(0, 0);
 
@@ -426,8 +462,7 @@ void KCategorizedView::Private::drawDraggedItems(QPainter *painter)
 
 void KCategorizedView::Private::layoutChanged(bool forceItemReload)
 {
-    if ((listView->viewMode() == KCategorizedView::IconMode) && proxyModel &&
-        categoryDrawer && proxyModel->isCategorizedModel() &&
+    if (proxyModel && categoryDrawer && proxyModel->isCategorizedModel() &&
         ((forceItemReload ||
           (modelSortRole != proxyModel->sortRole()) ||
           (modelSortColumn != proxyModel->sortColumn()) ||
@@ -447,8 +482,7 @@ void KCategorizedView::Private::layoutChanged(bool forceItemReload)
             modelCategorized = proxyModel->isCategorizedModel();
         }
     }
-    else if ((listView->viewMode() == KCategorizedView::IconMode) && proxyModel &&
-             categoryDrawer && proxyModel->isCategorizedModel())
+    else if (proxyModel && categoryDrawer && proxyModel->isCategorizedModel())
     {
         updateScrollbars();
     }
@@ -485,6 +519,8 @@ KCategorizedView::KCategorizedView(QWidget *parent)
     : QListView(parent)
     , d(new Private(this))
 {
+    setHorizontalScrollMode(QAbstractItemView::ScrollPerPixel);
+    setVerticalScrollMode(QAbstractItemView::ScrollPerPixel);
 }
 
 KCategorizedView::~KCategorizedView()
@@ -566,8 +602,7 @@ void KCategorizedView::setModel(QAbstractItemModel *model)
 
 QRect KCategorizedView::visualRect(const QModelIndex &index) const
 {
-    if ((viewMode() != KCategorizedView::IconMode) || !d->proxyModel ||
-        !d->categoryDrawer || !d->proxyModel->isCategorizedModel())
+    if (!d->proxyModel || !d->categoryDrawer || !d->proxyModel->isCategorizedModel())
     {
         return QListView::visualRect(index);
     }
@@ -649,8 +684,7 @@ void KCategorizedView::setCategoryDrawer(KCategoryDrawer *categoryDrawer)
 
 QModelIndex KCategorizedView::indexAt(const QPoint &point) const
 {
-    if ((viewMode() != KCategorizedView::IconMode) || !d->proxyModel ||
-        !d->categoryDrawer || !d->proxyModel->isCategorizedModel())
+    if (!d->proxyModel || !d->categoryDrawer || !d->proxyModel->isCategorizedModel())
     {
         return QListView::indexAt(point);
     }
@@ -688,8 +722,7 @@ void KCategorizedView::reset()
 
 void KCategorizedView::paintEvent(QPaintEvent *event)
 {
-    if ((viewMode() != KCategorizedView::IconMode) || !d->proxyModel ||
-        !d->categoryDrawer || !d->proxyModel->isCategorizedModel())
+    if (!d->proxyModel || !d->categoryDrawer || !d->proxyModel->isCategorizedModel())
     {
         QListView::paintEvent(event);
         return;
@@ -832,8 +865,7 @@ void KCategorizedView::resizeEvent(QResizeEvent *event)
     d->categoriesPosition.clear();
     d->forcedSelectionPosition = 0;
 
-    if ((viewMode() != KCategorizedView::IconMode) || !d->proxyModel ||
-        !d->categoryDrawer || !d->proxyModel->isCategorizedModel())
+    if (!d->proxyModel || !d->categoryDrawer || !d->proxyModel->isCategorizedModel())
     {
         return;
     }
@@ -844,8 +876,7 @@ void KCategorizedView::resizeEvent(QResizeEvent *event)
 void KCategorizedView::setSelection(const QRect &rect,
                                     QItemSelectionModel::SelectionFlags flags)
 {
-    if ((viewMode() != KCategorizedView::IconMode) || !d->proxyModel ||
-        !d->categoryDrawer || !d->proxyModel->isCategorizedModel())
+    if (!d->proxyModel || !d->categoryDrawer || !d->proxyModel->isCategorizedModel())
     {
         QListView::setSelection(rect, flags);
         return;
@@ -941,6 +972,8 @@ void KCategorizedView::setSelection(const QRect &rect,
         }
 
         int itemWidthPlusSeparation = spacing() + itemWidth;
+        if (!itemWidthPlusSeparation)
+            itemWidthPlusSeparation++;
         int elementsPerRow = viewportWidth / itemWidthPlusSeparation;
         if (!elementsPerRow)
             elementsPerRow++;
@@ -986,8 +1019,7 @@ void KCategorizedView::mouseMoveEvent(QMouseEvent *event)
 {
     QListView::mouseMoveEvent(event);
 
-    if ((viewMode() != KCategorizedView::IconMode) || !d->proxyModel ||
-        !d->categoryDrawer || !d->proxyModel->isCategorizedModel())
+    if (!d->proxyModel || !d->categoryDrawer || !d->proxyModel->isCategorizedModel())
     {
         return;
     }
@@ -1085,8 +1117,7 @@ void KCategorizedView::mouseReleaseEvent(QMouseEvent *event)
 
     QListView::mouseReleaseEvent(event);
 
-    if ((viewMode() != KCategorizedView::IconMode) || !d->proxyModel ||
-        !d->categoryDrawer || !d->proxyModel->isCategorizedModel())
+    if (!d->proxyModel || !d->categoryDrawer || !d->proxyModel->isCategorizedModel())
     {
         return;
     }
@@ -1102,7 +1133,7 @@ void KCategorizedView::mouseReleaseEvent(QMouseEvent *event)
         {
             if (d->categoryVisualRect(category).contains(event->pos()))
             {
-                QItemSelection selection;
+                QItemSelection selection = selectionModel()->selection();
                 QModelIndexList indexList = d->categoriesIndexes[category];
 
                 foreach (const QModelIndex &index, indexList)
@@ -1112,7 +1143,7 @@ void KCategorizedView::mouseReleaseEvent(QMouseEvent *event)
                     selection << QItemSelectionRange(selectIndex);
                 }
 
-                selectionModel()->select(selection, QItemSelectionModel::Select);
+                selectionModel()->select(selection, QItemSelectionModel::SelectCurrent);
 
                 break;
             }
@@ -1200,8 +1231,7 @@ void KCategorizedView::dragMoveEvent(QDragMoveEvent *event)
     QListView::dragMoveEvent(event);
 #endif
 
-    if ((viewMode() != KCategorizedView::IconMode) || !d->proxyModel ||
-        !d->categoryDrawer || !d->proxyModel->isCategorizedModel())
+    if (!d->proxyModel || !d->categoryDrawer || !d->proxyModel->isCategorizedModel())
     {
         return;
     }
@@ -1233,26 +1263,14 @@ QModelIndex KCategorizedView::moveCursor(CursorAction cursorAction,
                                          Qt::KeyboardModifiers modifiers)
 {
     if ((viewMode() != KCategorizedView::IconMode) ||
-         !d->proxyModel ||
-         !d->categoryDrawer ||
-          d->categories.isEmpty() ||
-         !d->proxyModel->isCategorizedModel()
-       )
+        !d->proxyModel ||
+        !d->categoryDrawer ||
+         d->categories.isEmpty() ||
+        !d->proxyModel->isCategorizedModel())
     {
         return QListView::moveCursor(cursorAction, modifiers);
     }
 
-    QModelIndex current = selectionModel()->currentIndex();
-
-    if (!current.isValid())
-    {
-        current = model()->index(0, 0, QModelIndex());
-        selectionModel()->select(current, QItemSelectionModel::NoUpdate);
-        d->forcedSelectionPosition = 0;
-
-        return current;
-    }
-
     int viewportWidth = viewport()->width() - spacing();
     int itemWidth;
 
@@ -1266,10 +1284,34 @@ QModelIndex KCategorizedView::moveCursor(CursorAction cursorAction,
     }
 
     int itemWidthPlusSeparation = spacing() + itemWidth;
+    if (!itemWidthPlusSeparation)
+        itemWidthPlusSeparation++;
     int elementsPerRow = viewportWidth / itemWidthPlusSeparation;
     if (!elementsPerRow)
         elementsPerRow++;
 
+    QModelIndex current = selectionModel()->currentIndex();
+
+    if (!current.isValid())
+    {
+        if (cursorAction == MoveEnd)
+        {
+            current = model()->index(model()->rowCount() - 1, 0, QModelIndex());
+            d->forcedSelectionPosition = d->elementsInfo[current.row()].relativeOffsetToCategory % elementsPerRow;
+        }
+        else
+        {
+            current = model()->index(0, 0, QModelIndex());
+            d->forcedSelectionPosition = 0;
+        }
+
+        return current;
+    }
+    else if (!current.isValid())
+    {
+        return QModelIndex();
+    }
+
     QString lastCategory = d->categories.first();
     QString theCategory = d->categories.first();
     QString afterCategory = d->categories.first();
@@ -1420,8 +1462,7 @@ void KCategorizedView::rowsInserted(const QModelIndex &parent,
 {
     QListView::rowsInserted(parent, start, end);
 
-    if ((viewMode() != KCategorizedView::IconMode) || !d->proxyModel ||
-        !d->categoryDrawer || !d->proxyModel->isCategorizedModel())
+    if (!d->proxyModel || !d->categoryDrawer || !d->proxyModel->isCategorizedModel())
     {
         d->forcedSelectionPosition = 0;
         d->elementsInfo.clear();
@@ -1522,8 +1563,7 @@ void KCategorizedView::rowsRemoved(const QModelIndex &parent,
                                    int start,
                                    int end)
 {
-    if ((viewMode() == KCategorizedView::IconMode) && d->proxyModel &&
-        d->categoryDrawer && d->proxyModel->isCategorizedModel())
+    if (d->proxyModel && d->categoryDrawer && d->proxyModel->isCategorizedModel())
     {
         // Force the view to update all elements
         rowsInsertedArtifficial(QModelIndex(), 0, d->proxyModel->rowCount() - 1);
@@ -1532,8 +1572,7 @@ void KCategorizedView::rowsRemoved(const QModelIndex &parent,
 
 void KCategorizedView::updateGeometries()
 {
-    if ((viewMode() != KCategorizedView::IconMode) || !d->proxyModel ||
-        !d->categoryDrawer || !d->proxyModel->isCategorizedModel())
+    if (!d->proxyModel || !d->categoryDrawer || !d->proxyModel->isCategorizedModel())
     {
         QListView::updateGeometries();
         return;
@@ -1572,6 +1611,8 @@ void KCategorizedView::currentChanged(const QModelIndex &current,
     }
 
     int itemWidthPlusSeparation = spacing() + itemWidth;
+    if (!itemWidthPlusSeparation)
+        itemWidthPlusSeparation++;
     int elementsPerRow = viewportWidth / itemWidthPlusSeparation;
     if (!elementsPerRow)
         elementsPerRow++;