Minefield trunk debug, source about 4-5 days old, 10.3.9 (ppc build with 10.3.0 sdk) STR: 1) Open Minefield 2) Launch Venkman 3) Once Venkman has opened, do "Minefield" --> "About Minefield" Results: Nothing happens. 4) Switch focus to Minefield and do "Minefield" --> "About Minefield" Results: Nothing happens. 5) Now, switch focus to Venkman and try to open Preferences from the Minefield menu (or shortcut) Results: No pref window opens and the Console window spits out an assertion: ###!!! ASSERTION: Carbon event for menu hit not sent!: 'err == noErr', file /Users/Stefan/debug/mozilla/widget/src/cocoa/nsMenuBarX.mm, line 1115 If you switch focus to Minefield you can open the pref window (but I get another, probably known, assertion 3 times). I haven't checked, but I'm pretty sure this is old (from the cocoa-switch)

"Preferences" is fixed in Venkman now because of the fix for bug 368911. I'm looking into the "About Minefield" part.

This fixes it, we just need to do the same thing with content nodes for the "About" menu item that we do for prefs and quit. We should have done this a long time ago.

Nice, this also hides it :-)

Comment on attachment 259741 [details] [diff] [review] fix v1.0 rs=pink

fixed on trunk

Comment on attachment 259854 [details] [diff] [review] followup separator fix, v1.0 r=mano.

Comment on attachment 259855 [details] [diff] [review] remove browser workarounds, v1.0 r=mano, thanks.

followup separator fix and remove browser workarounds checked in on trunk

Because of the last check-in the 'Report Web Forgery' menu item has moved on Windows. It is now grouped with the check for updates menu item because it inserts itself before the menu separator with id = aboutSeparator.