What is faster ? the eye or the brain ?

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

diff --git a/src/klistview.cpp b/src/klistview.cpp
index e484c3fb31174e7281561a417e0480e454add4a7..fa78de638e3c4cb6dbc02b44ab954a425912a27d 100644 (file)
--- a/src/klistview.cpp
+++ b/src/klistview.cpp
@@ -120,7 +120,6 @@ const QModelIndexList &KListView::Private::intersectionSet(const QRect &rect)
         }
     }
 
         }
     }
 

-    int j = 0;

     for (int i = middle; i < proxyModel->rowCount(); i++)
     {
         index = elementDictionary[proxyModel->index(i, 0)];
     for (int i = middle; i < proxyModel->rowCount(); i++)
     {
         index = elementDictionary[proxyModel->index(i, 0)];


@@ -131,10 +130,9 @@ const QModelIndexList &KListView::Private::intersectionSet(const QRect &rect)
 
         // If we passed next item, stop searching for hits
         if (qMax(rect.bottomRight().y(), rect.topLeft().y()) <
 
         // If we passed next item, stop searching for hits
         if (qMax(rect.bottomRight().y(), rect.topLeft().y()) <

-                                                  indexVisualRect.topLeft().y())
+                                                  qMin(indexVisualRect.topLeft().y(),
+                                                       indexVisualRect.bottomRight().y()))

             break;
             break;

-
-        j++;

     }
 
     return intersectedIndexes;
     }
 
     return intersectedIndexes;


@@ -333,8 +331,7 @@ void KListView::Private::drawNewCategory(const QString &category,
     opt.direction = option.direction;
     opt.text = category;
 
     opt.direction = option.direction;
     opt.text = category;
 

-    if (option.rect.contains(listView->viewport()->mapFromGlobal(QCursor::pos())) &&
-        !mouseButtonPressed)
+    if ((category == hoveredCategory) && !mouseButtonPressed)

     {
         const QPalette::ColorGroup group =
                                           option.state & QStyle::State_Enabled ?
     {
         const QPalette::ColorGroup group =
                                           option.state & QStyle::State_Enabled ?


@@ -398,10 +395,12 @@ void KListView::Private::drawDraggedItems(QPainter *painter)
 {
     QStyleOptionViewItemV3 option = listView->viewOptions();
     option.state &= ~QStyle::State_MouseOver;
 {
     QStyleOptionViewItemV3 option = listView->viewOptions();
     option.state &= ~QStyle::State_MouseOver;

+    int dx;
+    int dy;

     foreach (const QModelIndex &index, listView->selectionModel()->selectedIndexes())
     {
     foreach (const QModelIndex &index, listView->selectionModel()->selectedIndexes())
     {

-        int dx = mousePosition.x() - initialPressPosition.x() + listView->horizontalOffset();
-        int dy = mousePosition.y() - initialPressPosition.y() + listView->verticalOffset();
+        dx = mousePosition.x() - initialPressPosition.x() + listView->horizontalOffset();
+        dy = mousePosition.y() - initialPressPosition.y() + listView->verticalOffset();

 
         option.rect = visualRect(index);
         option.rect.adjust(dx, dy, dx, dy);
 
         option.rect = visualRect(index);
         option.rect.adjust(dx, dy, dx, dy);


@@ -753,14 +752,10 @@ void KListView::setSelection(const QRect &rect,
     }
 
     selectionModel()->select(selection, flags);
     }
 
     selectionModel()->select(selection, flags);

-
-    viewport()->update();

 }
 
 void KListView::mouseMoveEvent(QMouseEvent *event)
 {
 }
 
 void KListView::mouseMoveEvent(QMouseEvent *event)
 {

-    d->mousePosition = event->pos();
-

     QListView::mouseMoveEvent(event);
 
     if ((viewMode() != KListView::IconMode) || !d->proxyModel ||
     QListView::mouseMoveEvent(event);
 
     if ((viewMode() != KListView::IconMode) || !d->proxyModel ||


@@ -769,9 +764,19 @@ void KListView::mouseMoveEvent(QMouseEvent *event)
         return;
     }
 
         return;
     }
 

-    event->accept();
+    d->mousePosition = event->pos();
+    d->hoveredCategory = QString();
+
+    // Redraw categories
+    foreach (const QString &category, d->categories)
+    {
+        if (d->categoryVisualRect(category).intersects(QRect(event->pos(), event->pos())))
+        {
+            d->hoveredCategory = category;
+        }

 
 

-    viewport()->update();
+        viewport()->update(d->categoryVisualRect(category));
+    }

 }
 
 void KListView::mousePressEvent(QMouseEvent *event)
 }
 
 void KListView::mousePressEvent(QMouseEvent *event)


@@ -784,8 +789,6 @@ void KListView::mousePressEvent(QMouseEvent *event)
         return;
     }
 
         return;
     }
 

-    event->accept();
-

     d->dragLeftViewport = false;
 
     if (event->button() == Qt::LeftButton)
     d->dragLeftViewport = false;
 
     if (event->button() == Qt::LeftButton)


@@ -798,8 +801,6 @@ void KListView::mousePressEvent(QMouseEvent *event)
         d->initialPressPosition.setX(d->initialPressPosition.x() +
                                                             horizontalOffset());
     }
         d->initialPressPosition.setX(d->initialPressPosition.x() +
                                                             horizontalOffset());
     }

-
-    viewport()->update();

 }
 
 void KListView::mouseReleaseEvent(QMouseEvent *event)
 }
 
 void KListView::mouseReleaseEvent(QMouseEvent *event)


@@ -812,8 +813,6 @@ void KListView::mouseReleaseEvent(QMouseEvent *event)
         return;
     }
 
         return;
     }
 

-    event->accept();
-

     d->mouseButtonPressed = false;
 
     QPoint initialPressPosition = viewport()->mapFromGlobal(QCursor::pos());
     d->mouseButtonPressed = false;
 
     QPoint initialPressPosition = viewport()->mapFromGlobal(QCursor::pos());


@@ -842,24 +841,24 @@ void KListView::mouseReleaseEvent(QMouseEvent *event)
 
     d->lastSelection = selectionModel()->selection();
 
 
     d->lastSelection = selectionModel()->selection();
 

-    viewport()->update();
+    if (d->hovered.isValid())
+        viewport()->update(d->visualRect(d->hovered));
+    else if (!d->hoveredCategory.isEmpty())
+        viewport()->update(d->categoryVisualRect(d->hoveredCategory));

 }
 
 void KListView::leaveEvent(QEvent *event)
 {
     QListView::leaveEvent(event);
 
 }
 
 void KListView::leaveEvent(QEvent *event)
 {
     QListView::leaveEvent(event);
 

-    d->hovered = QModelIndex();
-

     if ((viewMode() != KListView::IconMode) || !d->proxyModel ||
         !d->itemCategorizer)
     {
         return;
     }
 
     if ((viewMode() != KListView::IconMode) || !d->proxyModel ||
         !d->itemCategorizer)
     {
         return;
     }
 

-    event->accept();
-
-    viewport()->update();
+    d->hovered = QModelIndex();
+    d->hoveredCategory = QString();

 }
 
 void KListView::startDrag(Qt::DropActions supportedActions)
 }
 
 void KListView::startDrag(Qt::DropActions supportedActions)


@@ -872,16 +871,15 @@ void KListView::startDrag(Qt::DropActions supportedActions)
 
 void KListView::dragMoveEvent(QDragMoveEvent *event)
 {
 
 void KListView::dragMoveEvent(QDragMoveEvent *event)
 {

-    QListView::dragMoveEvent(event);
-
-    d->mousePosition = event->pos();
-

     if ((viewMode() != KListView::IconMode) || !d->proxyModel ||
         !d->itemCategorizer)
     {
     if ((viewMode() != KListView::IconMode) || !d->proxyModel ||
         !d->itemCategorizer)
     {

+        QListView::dragMoveEvent(event);

         return;
     }
 
         return;
     }
 

+    d->mousePosition = event->pos();
+

     if (d->mouseButtonPressed)
     {
         d->isDragging = true;
     if (d->mouseButtonPressed)
     {
         d->isDragging = true;


@@ -892,10 +890,6 @@ void KListView::dragMoveEvent(QDragMoveEvent *event)
     }
 
     d->dragLeftViewport = false;
     }
 
     d->dragLeftViewport = false;

-
-    event->accept();
-
-    viewport()->update();

 }
 
 void KListView::dragLeaveEvent(QDragLeaveEvent *event)
 }
 
 void KListView::dragLeaveEvent(QDragLeaveEvent *event)


@@ -909,10 +903,6 @@ void KListView::dragLeaveEvent(QDragLeaveEvent *event)
     }
 
     d->dragLeftViewport = true;
     }
 
     d->dragLeftViewport = true;

-
-    event->accept();
-
-    viewport()->update();

 }
 
 QModelIndex KListView::moveCursor(CursorAction cursorAction,
 }
 
 QModelIndex KListView::moveCursor(CursorAction cursorAction,

diff --git a/src/klistview_p.h b/src/klistview_p.h
index c0a4289a81e21d35ca82b34fd67b8091aa74eecd..2980e98d66cd4c8dd87a992dc10a834de2eb13d9 100644 (file)
--- a/src/klistview_p.h
+++ b/src/klistview_p.h
@@ -135,6 +135,7 @@ public:
     QHash<QString, QRect> categoriesPosition;
     QStringList categories;
     QModelIndexList intersectedIndexes;
     QHash<QString, QRect> categoriesPosition;
     QStringList categories;
     QModelIndexList intersectedIndexes;

+    QString hoveredCategory;

 
     // Attributes for speed reasons
     KSortFilterProxyModel *proxyModel;
 
     // Attributes for speed reasons
     KSortFilterProxyModel *proxyModel;