On a clean profile, I get three buckets after visiting a single page: * Today * Last 7 days * This month The second and third bucket don't contain additional information, even though their presence implies to me that there's more history available than just today's (which there isn't). On another profile I've limited history to 5 days. There, the "This month" bucket will always just be UI-noise (being equal to the "Last 7 days" bucket). So why not just hide the "Last 7 days" bucket when there are no pages older than yesterday and the "This month" bucket when there are no pages older than 7 days?
that's by design, see discussion in the original bug, overlapping was requested expliticly
(In reply to comment #1) This bug isn't about the fact that the buckets overlap, but that it's possible for several buckets to be identical, which IMO just makes the UI noisier and gives a slightly wrong impression (that there are history entries older than in the bucket above).
we can do that eventually, leave them overlapped but don't show if there are not additional entries.
Created attachment 370382 [details] [diff] [review] patch v1.0 So with this patch: "last 7 days" is visible only if there are visits older than 2 days but younger than 7 days "this month" is visible only if there are visits older than 7 days but younger than month's start (so first 7 days of the month it will be always hidden)
ps: contents of the containers do not change with this patch, they are simply hidden in above cases, but "last 7 days" will always contain everything from 7 days ago to now.
Comment on attachment 370382 [details] [diff] [review] patch v1.0 r=me, with tests for each of the containers that this change affects.
Created attachment 370532 [details] [diff] [review] patch v1.1 with a somehow deeper test, Dietrich, do you think this is enough or should i create a completely new unit test just for this?
Comment on attachment 370532 [details] [diff] [review] patch v1.1 thanks, that's great, r=me
Comment on attachment 370532 [details] [diff] [review] patch v1.1 Please land on trunk for baking ASAP.
Comment on attachment 370532 [details] [diff] [review] patch v1.1 asking approval after 2 days of baking, this makes the sidebar experience better not showing useless date containers, and is automatically tested.
Comment on attachment 370532 [details] [diff] [review] patch v1.1 a191=beltzner
verified with Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1b4pre) Gecko/20090414 Shiretoko/3.5b4pre
Bug 451915 - move Firefox/Places bugs to Firefox/Bookmarks and History. Remove all bugspam from this move by filtering for the string "places-to-b-and-h". In Thunderbird 3.0b, you do that as follows: Tools | Message Filters Make sure the correct account is selected. Click "New" Conditions: Body contains places-to-b-and-h Change the action to "Delete Message". Select "Manually Run" from the dropdown at the top. Click OK. Select the filter in the list, make sure "Inbox" is selected at the bottom, and click "Run Now". This should delete all the bugspam. You can then delete the filter. Gerv