Bug 168005

Summary: Week view broken in timeline
Product: [Applications] digikam Reporter: Gandalf Lechner <gandalflechner>
Component: Searches-TimeLineAssignee: Digikam Developers <digikam-bugs-null>
Status: RESOLVED FIXED    
Severity: normal    
Priority: NOR    
Version First Reported In: 0.10.0   
Target Milestone: ---   
Platform: unspecified   
OS: Linux   
Latest Commit: Version Fixed/Implemented In: 0.10.0
Sentry Crash Report:

Description Gandalf Lechner 2008-08-01 17:06:27 UTC 
Version:           0.10.0-beta3 (rev.: 840686) (using 4.1.00 (KDE 4.1.0), Kubuntu packages)
Compiler:          gcc
OS:                Linux (x86_64) release 2.6.24-17-generic

While day, month and year view work in the timeline, the week view is broken: Almost all entries are missing, I can see columns only for two weeks over a span of several years (which is not consistent with the correctly working year view, for example)
Comment 1 caulier.gilles 2008-08-01 18:09:16 UTC 
Marcel, 

it's certainly an issue with new database search tool.

Gandalf,

Is that the problem do not exist with 0.9.4 ?

Gilles
Comment 2 Gandalf Lechner 2008-08-01 18:13:10 UTC 
No, it's fine in 0.9.4
Comment 3 caulier.gilles 2008-10-12 18:07:47 UTC 
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