Name: boT120536 Date: 06/03/2001


java version "1.4.0-beta"
Java(TM) 2 Runtime Environment, Standard Edition (build 1.4.0-beta-b65)
Java HotSpot(TM) Client VM (build 1.4.0-beta-b65, mixed mode)

The new focus rework was suppose to solve the problem of posted popups not
being able to be navigated with the keyboard. This seems to work OK but there
is still a serious problem. If a popup is posted it temporarily steals the focus
(as a result of what was done for 1.4), but menu accelerators(like from menubar
menus) are still firing while the popup is posted! This is obviously bad so I
hope it can be fixed in time for 1.4FCS.

Here is a simple testcase to show the bug. After invoking:

   1. RMB to post the popup menu on the desktop
   2. Hit "1", "2" or "3", as these are menu accelerators
      for the items in the menubar menu
   3. You'll see that the accelerators fire by checking
      stdout(where the menu's action listeners print)

import java.awt.*;
import java.awt.event.*;
import javax.swing.*;

public class MenuFocusBug extends JFrame {

   public static void main(String[] args) {
      new MenuFocusBug();
   }
   public MenuFocusBug() {
      super("Post Popup, Then Press \"1,2 or3\"");
      addWindowListener(new WindowAdapter() {
         public void windowClosing(WindowEvent e) {
            System.exit(0);
         }}
      );
      // --- Create the popup menu, with mnemonics
      final JPopupMenu popup = new JPopupMenu();
      popup.add(new JMenuItem("Foo" , 'F'));
      popup.add(new JMenuItem("Bar" , 'B'));
      popup.add(new JMenuItem("Hello", 'H'));
      popup.add(new JMenuItem("World", 'W'));

      // --- Create a desktop that posts the popup
      final JDesktopPane dtp = new JDesktopPane();
      dtp.setOpaque(true);
      dtp.setBackground(Color.black);
      setContentPane(dtp);
      dtp.addMouseListener(new MouseAdapter() {
         public void mouseReleased(MouseEvent e) {
            if (e.isPopupTrigger()) {
               popup.show(dtp, e.getX(), e.getY());
            }
         }
      });

      // --- Create a menubar menu, that uses menu accelerators
      JMenuBar mb = new JMenuBar();
      setJMenuBar(mb);
      JMenu menu = new JMenu("Test Menu w/ 3 Items");
      createMenuItem(menu);
      createMenuItem(menu);
      createMenuItem(menu);
      mb.add(menu);

      pack();
      setBounds(300,300,400,200);
      setVisible(true);
   }
   private int cnt = 1;
   private void createMenuItem(JMenu menu) {

      JMenuItem mi = menu.add(new JMenuItem("Item #" + cnt));
      mi.setAccelerator(KeyStroke.getKeyStroke(Integer.toString(cnt)));
      final int _cnt = cnt;
      mi.addActionListener(new ActionListener() {
         public void actionPerformed(ActionEvent e) {
            System.out.println("MenuBar Item #"+_cnt);
         }
      });
      cnt++;
   }
}
(Review ID: 125051)
======================================================================