Bug 168004 - Timeline gives zero height column for dates with a single entry
Summary: Timeline gives zero height column for dates with a single entry
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:00 UTC by Gandalf Lechner
Modified: 2019-12-27 22:51 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In: 7.0.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:00:34 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

In the timeline view, if you select a time range (day, week, month or year) which has associated only a single picture, no column is displayed in the widget - both in the linear and the logarithmic one. Probably, there should be some minimum height making such entries visible.

Same situation in the 0.9 series.
Comment 1 caulier.gilles 2008-12-03 13:02:04 UTC
I can confirm this problem only with logarithmic scale...

Gilles
Comment 2 Mikolaj Machowski 2008-12-08 17:15:54 UTC
Confirming for 0.10b7svn both: linear and logarythmic scale for days, weeks and even months. Not only for one image but if there are several of them.
Comment 3 Marcel Wiesweg 2008-12-16 19:51:51 UTC
I have a patch here that draws the smallest visible rectangle (one-pixel line of blue, one-pixel line border, so 2px) when the value for a time span is not 0, but value / maxCount is sufficiently small that it would result in 0 pixels.
Now this is not linear any more for small values, but I think it's better for usability
Fine with you Gilles?
Comment 4 caulier.gilles 2008-12-16 21:00:11 UTC
Marcel,

yes, fine for me...

Gilles
Comment 5 Marcel Wiesweg 2008-12-16 21:51:11 UTC
SVN commit 897802 by mwiesweg:

If number of pictures is not null, use a minimum height of 1
(I first thought to use minimum height 2, but height 1, just a one-pixel line, works as well)

CCBUG: 168004

 M  +18 -5     timelinewidget.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=897802
Comment 6 caulier.gilles 2019-12-27 22:51:51 UTC
Not reproducible with 7.0.0-beta1