Patch review: 321928 Pre-defined date ranges

[Stephane Delcroix <stephane delcroix org>]

http://bugzilla.gnome.org/show_bug.cgi?id=321928

Quick patch review

Patch cleanly against current CVS.
It could be useful, clearly.

I only see one minor issue... there are no default in the dropdown list. It's a
bit confusing at first use. And it's not very user friendly.
A good default could be "custom" or "last week".

Speaking of usability, I guess that 'last year' is better than 'last 360 days',
and 'Previous Week (mon-sun)' could be replaced by 'Previous week (8-14
May)'...

my 2 cents, for what it's worth

rgds,

Stephane
-- 
Stephane Delcroix
stephane delcroix org