Bug 168005 - Week view broken in timeline
Summary: Week view broken in timeline
Status: RESOLVED FIXED
Alias: None
Product: digikam
Classification: Applications
Component: Searches-TimeLine (show other bugs)
Version: 0.10.0
Platform: unspecified Linux
: NOR normal
Target Milestone: ---
Assignee: Digikam Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-08-01 17:06 UTC by Gandalf Lechner
Modified: 2012-06-27 11:14 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In: 0.10.0


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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