When downloads window is active, bookmarks in folders in bookmarks bar ignore tabbed browsing prefs

NEW
Unassigned

Status

Camino Graveyard
Tabbed Browsing
--
minor
12 years ago
8 years ago

People

(Reporter: froodian (Ian Leue), Unassigned)

Tracking

Details

Attachments

(1 attachment)

(Reporter)

Description

12 years ago
User-Agent:       Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.8) Gecko/20051127 Camino/1.0b1+
Build Identifier: Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.8) Gecko/20051127 Camino/1.0b1+

When the downloads folder is active, command-clicking on bookmarks conatained in folders in the bookmarks bar always opens the bookmark in a new window, even if the pref is set to open links in new tabs on Cmd-click.  It also ignores the "new windows load  in background" pref, and always loads in the foreground.

Reproducible: Always

Steps to Reproduce:
1. Open the downloads window
2. With the downloads window still in front, cmd-click on a folder in the bookmarks bar
3. Select a bookmark contained in said folder

Actual Results:  
Opens link in new window in foreground

Expected Results:  
Obey pref settings for cmd-clicking links
Yeah, I see this. It's kind of random case though. Not many users probably do this...
Status: UNCONFIRMED → NEW
Ever confirmed: true

Comment 2

12 years ago
I suspect what's going on here is the window controller is (rightly) seeing that it can't open a new tab in the frontmost window, and is therefore opening a new window.

What it *should* be doing is making a new tab in the window in which the click was registered.

cl

Comment 3

12 years ago
Further note: we do the right thing when the frontmost window is another browser window.

cl
(Reporter)

Comment 4

11 years ago
This also happens if another app is in front.
(Reporter)

Comment 5

11 years ago
Created attachment 231353 [details] [diff] [review]
Patch

I'm a little scared about this patch, just because it touches |loadBookmark| which I know is kinda fragile, but I think that conceptually it's the right change, and it doesn't seem to break anything...
Assignee: mikepinkerton → stridey
Status: NEW → ASSIGNED
Attachment #231353 - Flags: review?(stuart.morgan)
(Reporter)

Updated

11 years ago
Summary: When downloads folder is active, bookmarks in folders in bookmarks bar ignore tabbed browsing prefs → When downloads window is active, bookmarks in folders in bookmarks bar ignore tabbed browsing prefs
(Reporter)

Comment 6

11 years ago
Comment on attachment 231353 [details] [diff] [review]
Patch

This isn't really the way to do this.  Specifically, we want Cmd-Click on bookmark menu items (from the main bookmarks menu) to open in new windows in these cases - The only reason bookmark bar items shouldn't open in new windows is because they're "attached" to a window.

Codewise, items from the main bookmarks menu and the bookmark folder menus both call |openMenuBookmark| in MainController.  This code is all tied up in bug 333765, so it's gonna have to wait for that.
Attachment #231353 - Flags: review?(stuart.morgan) → review-
(Reporter)

Updated

11 years ago
Depends on: 333765
(Reporter)

Comment 7

11 years ago
This bug sucks.  Basically, the existing patch makes it so that cmd-click on menu bookmarks (from bookmarks bar or bookmarks menu) will always open in tabs in the *frontmost* browser window (assuming the tabs pref is set).

The problem is that we want

a) bookmarks from the bookmarks menu to always open in a new window if a browser window isn't frontmost, and follow the pref if a browser window is.

b) bookmarks from bookmarks bar menus to always follow the pref, opening in a new tab *in the browser window they were invoked from* if it's not frontmost.

It's the "in the browser window they were invoked from unless it's a bookmarks menu item, then in a new browser window" bit that's annoying.  I might realize a nice way to do this suddenly, but until then I'm not really interested in fixing this fringe case (despite the fact that I filed it when I was just a user).
Assignee: stridey → nobody
Status: ASSIGNED → NEW
QA Contact: tabbed.browsing

Comment 8

11 years ago
Can you look up the view hierarchy and see if the item has a browser window as one of its parent views? That'd be an easy way to differentiate between bookmarks menu items and bookmarks bar items.

Comment 9

10 years ago
(In reply to comment #8)
> Can you look up the view hierarchy and see if the item has a browser window as
> one of its parent views?

Gonna reply to my own comment so no one else wastes time trying to chase this down: the answer to the above is "no", according to everything I've been able to find. There is no way to know the view a menu was invoked for once the menu has been displayed. See also

http://lists.apple.com/archives/cocoa-dev/2006/Aug/msg01486.html

And since both menus are instances of BookmarkMenu, I agree with Ian that this bug totally sucks. I don't see any reasonable way to differentiate the two menus, which I guess makes this a CANTFIX unless anyone has any brilliant ideas.

Comment 10

8 years ago
I have a similar problem in my Mac OS 10.4.11   I cannot drag the URL to save it in my Bookmarks bar.  I am forced to open the pull-down menu and scroll or clic several times until I get to the folder in my bar where I want the bookmark to be. This is cumbersome.  Why, if it didn't happen with Firefox 3.0, is now happening with Firefox 3.5?  Are you going backwards?

Comment 11

8 years ago
This bug is a) for Camino, and b) about something completely different that what you are describing, so you should find/file a different bug.
You need to log in before you can comment on or make changes to this bug.