I've taken it upon myself to rework/refactor the unix menu code, after trying to fix bug 3852 (it's fixed). So far, I've managed to cut out something on the order of 500 loc, by refactoring some of the common code into private member functions, and eliminating some custom label/menuitem setting code which was just unnecessary. I just have to fix the keyboard accelerators and it looks like the menus should be in nice shape.
I do wonder about one thing though: This line of code in af/xap/xap_UnixFrame.cpp around line 244-- is this really necessary (intercepting and ignoring every keypress that isn't in an alt-combination)? The menu mnemonics won't work with this enabled, and Abi seems to behave as expected with the code commented out. If no one says anything, I'm going to remove this code. Cheers, Will Lachance [EMAIL PROTECTED]
