Closed Bug 372025 Opened 18 years ago Closed 18 years ago

Duplicates in the history menu are not collapsed

Categories

(Firefox :: Bookmarks & History, defect, P2)

defect

Tracking

()

RESOLVED FIXED
Firefox 3 alpha3

People

(Reporter: asaf, Assigned: asaf)

References

Details

(Whiteboard: [Fx2-parity])

Attachments

(1 file)

Duplicated items in the history menu are not collapsed. The code for collapsing duplicates lives in the treeviewer implementation for some reason.
Flags: blocking-firefox3?
Whiteboard: [Fx2-parity]
For some sortings, you don't want duplicates collapsed, and you want to collapse them in different ways depending on the sorting. For example, sort by date you want adjacent duplicates collapsed, but not if you visited them at different times. If you then sort by title, you want all instances of that page collapsed. This is why collapsing is done in the view. If you do not ever want duplicates, you can query for urls instead of visits.
Assignee: nobody → mano
Priority: -- → P2
Target Milestone: --- → Firefox 3 alpha3
Attached patch as simpleSplinter Review
Many thanks, Brett.
Attachment #256710 - Flags: review?(sspitzer)
Status: NEW → ASSIGNED
Comment on attachment 256710 [details] [diff] [review] as simple r=sspitzer, as you are switching from RESULTS_AS_VISIT (1) to RESULTS_AS_URI (0), per brettw's suggestion.
Attachment #256710 - Flags: review?(sspitzer) → review+
mozilla/browser/base/content/browser-menubar.inc 1.110
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Flags: blocking-firefox3?
Resolution: --- → FIXED
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: