Closed Bug 359465 Opened 18 years ago Closed 15 years ago

[rfe] change history sidebar group by date to be like IE7's

Categories

(Firefox :: Bookmarks & History, enhancement)

enhancement
Not set
normal

Tracking

()

RESOLVED DUPLICATE of bug 390614

People

(Reporter: moco, Unassigned)

Details

Attachments

(1 file)

[rfe] change history sidebar group by date to be like IE7's

spun off of bug #359346
some things that IE 7 does:

* calendar icons, instead of folder icon (for day containers)
* weekly groupings (3 weeks ago, 2 weeks ago, last week)
* named days (Monday, Wednesday instead of x days ago)
* sort order (most recent is last)
Severity: normal → enhancement
I realized you should be doing this grouping differently than you are, as it will give poor performance grouping every single item in history (though possibly not worse than the Mork one).

You should have some root thing (probably this is hardcoded somewhere for now, in the original design it would be a bookmark folder) that contains queries. These queries are for "Today", "Yesterday", etc...

Querying by time is *very* fast. This prevents you from having to schlep *all* of history into memory and sort it and group it. Only those folder that the user has opened will be populated and kept up-to-date, which will happen on demand. Initial population will be instant.
brett, was your comment for this bug, for bug #358784 (where I've got an implementaion of GROUP_BY_DATE)
I wasn't sure where to put it, I just wanted you guys to keep that in mind. Your group by date implementation is probably fine for now, since it gives parity. You can worry about performance later. Just realize that this really needs to be changed before shipping.
> I wasn't sure where to put it

ok, thanks brett.  I've taken it (and your follow up comment) to bug #358784
This will be much easier and faster now when the bug 385245 has landed. Please note that other views of the sidebar are candidates for better grouping too - see bug 417262.
Component: History → Places
QA Contact: history → places
not as IE7 but splitting has changed in bug 390614, we can further tweak splitting in new followups, there's no need to hold this bug, there's no evidence that ie7 splitting is better than ours atm.
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → DUPLICATE
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
Component: Places → Bookmarks & History
QA Contact: places → bookmarks
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: