I'm attaching various fixes I already have for the bookmark system here. They'll come as one big (well, not that big) patch - it should be easy to split into bite sized morsels if that's preferred.

This patch a) fixes the HTML export system (which is broken on 10.2, and won't even run on 10.1) and b) adds in support for exporting Safari.plist files. Beyond the new -writeSafariDictionary methods, I moved all the declarations for export/import keys into BookmarkItem.h and defined them in BookmarkItem.m.

This patch fixes a boo-boo on files which didn't have the toolbar flag set properly, and schmears out loading the site icons so there isn't such a delay on load.

This a) fixes a crash which can occur if a bookmark folder in the toolbar is accidentally dropped into itself on a drag & drop b) Makes sure the toolbar is visible on the first window if Camino was launched to handle an apple-event c) Preserves the order of bookmarks in a folder in the event of a mass drag & drop (current behavior reverses their order).

This prevents titles of bookmark from overlapping the "close/minimize/maximize" buttons on the Bookmark Info panel.

This a)makes sure the bookmarks are visible in the event Camino was launched via an apple-event b)pops open the search result window when you do a search, in the event it was hidden c)preserves the order of bookmarks dropped in a mass drag & drop (current behavior reverses their order in a folder) d)fixes some button states appropriately depending on what folder is selected e)gets rid of a useless method & cleans up a differnt method.

This preserves the top 10 list, toolbar bar, etc icons in the event you change the dock menu folder.

cc sbwoodside, pushing pinkerton's e-mails from this bug today into the triple digets ;)

Personally, I'd rather see a separate bug for each one. Just easier to keep track of (but maybe not to anyone else)

I agree with simon. I will split the bug and patches and make this one a META, with all bugs created from this one in the dependency tree.

nominee for 0.8

Which of these fixes the "add separator" button not moving?

re comment 11: bug 223413. It's just a replacement nib.

Removing dependency on bug 223576. How that's related to this bug in any way is beyond me.

Comment on attachment 133819 [details] [diff] [review] Bookmark Icon patch unset review requests since this is a meta bug now.

I added dependancy on bug 224629.

Nominating bug 199790

Removing bug 225276 from the list since it has nothing to do with bookmarks.

removing things we know aren't related to the bm cleanup and closing, since we're tracking the bugs individually now.