After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 342694 - Timeline not updated when changing or clearing Date Range
Timeline not updated when changing or clearing Date Range
Status: RESOLVED FIXED
Product: f-spot
Classification: Other
Component: Browsing
0.1.x
Other Linux
: Normal normal
: ---
Assigned To: F-spot maintainers
F-spot maintainers
Depends on:
Blocks:
 
 
Reported: 2006-05-23 12:47 UTC by Stephane Delcroix
Modified: 2006-06-06 15:24 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
reset timeline limits on 'clear date range' (1.71 KB, patch)
2006-05-23 14:53 UTC, Stephane Delcroix
none Details | Review
the mandatory screenshot... (117.24 KB, image/jpeg)
2006-05-23 14:54 UTC, Stephane Delcroix
  Details
update timeline limits with Set or Clear Date Range (4.76 KB, patch)
2006-05-24 12:26 UTC, Stephane Delcroix
none Details | Review
updated patch (7.01 KB, patch)
2006-05-30 09:58 UTC, Stephane Delcroix
committed Details | Review

Description Stephane Delcroix 2006-05-23 12:47:44 UTC
You can set a date range graphically using the 'borders' of the timeline.

But when you select a date range or clear the date range from the 'find' menu, the graphical timeline is not updated.

Way to reproduce:

1) 
- Find > Set Date Range 
- the timeline is unchanged

2)
- Select a Date Range using the timeline
- Find > Clear Date Range
- the timeline is not reset
Comment 1 Stephane Delcroix 2006-05-23 14:53:53 UTC
Created attachment 66061 [details] [review]
reset timeline limits on 'clear date range'

A small patch to solve the second case of the bug report.

Now, when you choose 'Clear Date Range', the limits of the TimeLine are reset.
This patch also add an entry to the popup of the timeline to 'Clear Date Range'

See also the screenshot below

Any comments ?
Comment 2 Stephane Delcroix 2006-05-23 14:54:48 UTC
Created attachment 66062 [details]
the mandatory screenshot...
Comment 3 Bengt Thuree 2006-05-23 15:30:35 UTC
Please also see bug #324908
Comment 4 Stephane Delcroix 2006-05-24 12:26:05 UTC
Created attachment 66120 [details] [review]
update timeline limits with Set or Clear Date Range

A new version of the patch to handle both cases.

- the timeline is updated when you set a date range
- the timeline is updated when you clear the date range
- the selection is adjusted when you reverse the order
- the selection is restored when you change from 'directory view' to 'month view'
- a 'clear date range' menu item is added in the popup in 'month view'

Someone to review it ??
Comment 5 Stephane Delcroix 2006-05-30 09:58:50 UTC
Created attachment 66465 [details] [review]
updated patch

An updated patch, after review, comments and a patch from Thomas Van Machelen.

Quoting Thomas:
<snip>
* 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.
<snap>
Comment 6 Larry Ewing 2006-06-06 15:24:47 UTC
Looks good,committed.