Jesse Ruderman Reporter
	Description • 17 years ago

Steps to reproduce:
1. Cmd+N.
2. Ctrl+click an item on the bookmarks toolbar.

Result: "Get Info", etc. are disabled.  Only "New Bookmark..." and other "New" items are enabled.

	Joey Minta
	Comment 1 • 17 years ago

This works fine if you do right click (enable two-finger clicking), but fails if you do ctrl+click.  The difference is in the timing of toolbar.xml's mousedown and the onpopupshowing event for the popup.  For the two-finger clicking, the mousedown fires first (which means there's a selection, so the appropriate nodes are enabled).  For ctrl+click, onpopupshowing is fired first, so there's no selection, and the items are disabled.  This can be fixed by putting http://mxr.mozilla.org/seamonkey/source/browser/components/places/content/placesOverlay.xul#103
into a timeout, but I'm wondering if this isn't a lower level event-bug for mac.

	Jesse Ruderman Reporter
	Comment 2 • 17 years ago

Colin, please see comment 1.

	Asaf Romano (gone)
	Comment 3 • 17 years ago

bug 368397 was supposed to fix this...

	Josh Aas Assignee
	Comment 4 • 17 years ago

I worked on this for a bit today, it is a cocoa widgets bug. I know what is going on, should have a patch soon. Basically we send a context menu event from menuForEvent: and then the mouse event, then another context menu event.

	Josh Aas Assignee
	Comment 5 • 17 years ago

The patch on bug 389542 fixes this, but I don't understand why yet. Still reading through it, just wanted to note that here.

	Steven Michaud [:smichaud] (Retired)
	Comment 6 • 17 years ago

Fixed by patch for bug 389542, which just landed on trunk.