350482 - Make bookmarks bar the default collection selection on add bookmark dialog

Status: VERIFIED FIXED

(Camino Graveyard :: Toolbars & Menus, enhancement)

Description

[froodian (Ian Leue)]

Per a status meeting, as a stop-gap measure for bug 287708 (and to allow dialog-less bookmarking to be useful), we should make the default selection of cmd-D the bookmarks bar (and not the bookmarks folder) when invoked for the first time after launch.

Comment 1

[froodian (Ian Leue)]

Attachment: Patch

Comment 2

[Bruce Davidson]

Comment on attachment 235757 [details] [diff] [review]
Patch

r+ = me.

Comment 3

[Mike Pinkerton (not reading bugmail)]

Comment on attachment 235757 [details] [diff] [review]
Patch

sr=pink

Comment 4

[Chris Lawson (gone)]

How long are we going to be stopping this gap? :-p Hopefully we get something finished before we ship another release version because this has the potential to annoy people who are used to the old behaviour.

cl

Comment 5

[Håkan Waara]

Comment on attachment 235757 [details] [diff] [review]
Patch

Your |bookmarkManager| variable could be used in more places in the same scope, where [BookmarkManager sharedBookmarkManager]| currently is. Reuse, reuse, reuse! :)

Comment 6

[froodian (Ian Leue)]

Attachment: Patch

Oops.  Yeah, I missed one.  Nice catch.

Comment 7

[froodian (Ian Leue)]

Checked in on trunk and 1.8branch.

Comment 8

[Chris Casciano]

@ comment #4


like me

Comment 9

[Kevin Parichan]

Camino should remember what I last selected in the bookmarks dialog pop up between launches of the application. Not just for the current session.

The new behavior (default to bookmarks bar) is very annoying, especially if you have a long bookmarks menu like I do. I forget that it doesn't save the selection between launches and end up putting all the bookmarks into the bookmarks bar, which I have hidden.

Comment 10

[Samuel Sidler (old account; do not CC)]

(In reply to comment #9)
> Camino should remember what I last selected in the bookmarks dialog pop up
> between launches of the application. Not just for the current session.

That's filed as bug 328737.

Comment 11

[Ben Fowler]

(In reply to comment #10)
> (In reply to comment #9)
> > Camino should remember what I last selected in the bookmarks dialog pop up
> > between launches of the application. Not just for the current session.

I am not surprised that people are complaining about the '"Add Bookmark" to Bookmark Bar policy' over-riding the user's wish!

A better stop gap measure would be to put New Bookmarks at the top of the Bookmark list, or in a dated folder at that place (much like the History list); or better still in an alternative Bookmark Bar labeled 'Incoming', from which they could be dragged to the "All Bookmarks" screen at the user's lesiure.

My Bookmark Bar is ten rows deep (owing to the lack of usability of the Bookmark menu and the "All Bookmarks" screen) and I have to keep it hidden: Adding bookmarks to /dev/null makes more sense than the existing policy, and I guess that this what many users are actually doing nowadays!

I expect that these ideas have been considered and rejected, but I ask: Would a patch to do this be considered? 

Comment 12

[Stuart Morgan]

(In reply to comment #11)
> Would a patch to do this be considered? 

No. Bug 328737 is the right way to fix this.