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)
Firefox
Bookmarks & History
Tracking
()
RESOLVED
DUPLICATE
of bug 390614
People
(Reporter: moco, Unassigned)
Details
Attachments
(1 file)
18.24 KB,
image/png
|
Details |
[rfe] change history sidebar group by date to be like IE7's spun off of bug #359346
Reporter | ||
Comment 1•18 years ago
|
||
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)
Reporter | ||
Comment 2•18 years ago
|
||
Reporter | ||
Updated•18 years ago
|
Severity: normal → enhancement
Comment 3•18 years ago
|
||
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.
Reporter | ||
Comment 4•18 years ago
|
||
brett, was your comment for this bug, for bug #358784 (where I've got an implementaion of GROUP_BY_DATE)
Comment 5•18 years ago
|
||
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.
Reporter | ||
Comment 6•18 years ago
|
||
> I wasn't sure where to put it ok, thanks brett. I've taken it (and your follow up comment) to bug #358784
Comment 7•16 years ago
|
||
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
Comment 8•15 years ago
|
||
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
Comment 9•15 years ago
|
||
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.
Description
•