Re: Updating the TimeLine Limits

["Thomas Van Machelen" <thomas vanmachelen gmail com>]

2006/5/27, Thomas Van Machelen <thomas vanmachelen gmail com>:
> Hi all,
>
> On Wed, 2006-05-24 at 14:38 +0200, Stephane Delcroix wrote:
> > A new patch to solve also the case of 'Set Date Range'
> > http://bugzilla.gnome.org/attachment.cgi?id=66120&action=view
> >
>
> Cool stuff, i like both the "Clear Date Range" context menu and the fact
> that the date range is actually visible now.  However you can find a new
> patch in attachment that:
>
> * fixes a bug in the sliding of the end limit (it took a month too much
> off)
> * only shows the Clear Data Range menu item when a range actually has
> been set
> * removes some of the code duplication you introduced in the TimeAdaptor
> class.

Stephane posted a new version of the patch to bugzilla [1].

I think it really improves f-spot by connecting two features that were
previously distinct: updating the date range now updates the timeline
widget and vice versa.  This makes both sliding the limits in the
timeline and setting the date range more intuitive to use.

Please test and post comments.

Regards,
Thomas

[1] http://bugzilla.gnome.org/show_bug.cgi?id=342694