[Digikam-devel] [Bug 168005] Week view broken in timeline

[Gilles Caulier]

http://bugs.kde.org/show_bug.cgi?id=168005


Gilles Caulier caulier gilles gmail com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|                            |FIXED




--- Comment #3 from Gilles Caulier <caulier gilles gmail com>  2008-10-12 18:07:47 ---
Fixed by marcel in current svn :

SVN commit 870444 by mwiesweg:

Change the cursor to PointingHand on the exact spot where a timeline
selection is possible, to indicate that it is just this area you are supposed
to click.

 M  +26 -2     timelinewidget.cpp


--- trunk/extragear/graphics/digikam/utilities/timeline/timelinewidget.cpp
#870443:870444
@@ -1520,12 +1520,36 @@

 void TimeLineWidget::mouseMoveEvent(QMouseEvent *e)
 {
+    // set cursor shape to indicate selection area
+    QRect selectionArea;
+    selectionArea.setTop(height() - d->bottomMargin + 1);
+    selectionArea.setLeft(0);
+    selectionArea.setBottom(height());
+    selectionArea.setRight(width());
+
+    bool sel;
+    QDateTime selEndDateTime;
+
+    if (selectionArea.contains(e->pos()))
+    {
+        selEndDateTime = dateTimeForPoint(e->pos(), &sel);
+        SelectionMode unused;
+        bool hasSelectableDates = statForDateTime(selEndDateTime, &unused);
+        if (hasSelectableDates)
+            setCursor(Qt::PointingHandCursor);
+        else
+            unsetCursor();
+    }
+    else
+        unsetCursor();
+
+    // handle move event
    if (d->validMouseEvent == true)
    {
        QPoint pt(e->x(), e->y());

-        bool sel;
-        QDateTime selEndDateTime = dateTimeForPoint(pt, &sel);
+        if (selEndDateTime.isNull())

Gilles Caulier


-- 
Configure bugmail: http://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.