485703 - Hide date containers not containing additional entries compared to previous ones.

Status: VERIFIED FIXED

(Firefox :: Bookmarks & History, defect)

Description

[Simon Bünzli]

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?

Comment 1

[Marco Bonardo [:mak]]

that's by design, see discussion in the original bug, overlapping was requested expliticly

Comment 2

[Simon Bünzli]

(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).

Comment 3

[Marco Bonardo [:mak]]

we can do that eventually, leave them overlapped but don't show if there are not additional entries.

Comment 4

[Marco Bonardo [:mak]]

Attachment: 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)

Comment 5

[Marco Bonardo [:mak]]

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 6

[Dietrich Ayala (:dietrich)]

Comment on attachment 370382 [details] [diff] [review]
patch v1.0

r=me, with tests for each of the containers that this change affects.

Comment 7

[Marco Bonardo [:mak]]

Attachment: 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 8

[Dietrich Ayala (:dietrich)]

Comment on attachment 370532 [details] [diff] [review]
patch v1.1

thanks, that's great, r=me

Comment 9

[Mike Beltzner [:beltzner, not reading bugmail]]

Comment on attachment 370532 [details] [diff] [review]
patch v1.1

Please land on trunk for baking ASAP.

Comment 10

[Marco Bonardo [:mak]]

http://hg.mozilla.org/mozilla-central/rev/f0507c4d0abb

Comment 11

[Marco Bonardo [:mak]]

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 12

[Mike Beltzner [:beltzner, not reading bugmail]]

Comment on attachment 370532 [details] [diff] [review]
patch v1.1

a191=beltzner

Comment 13

[Marco Bonardo [:mak]]

http://hg.mozilla.org/releases/mozilla-1.9.1/rev/e6cd714d3451

Comment 14

[Tracy Walker [:tracy]]

verified with Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1b4pre)
Gecko/20090414 Shiretoko/3.5b4pre

Comment 15

[Gervase Markham [:gerv]]

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