SUGGESTED FIX
Name: pzR10082 Date: 02/05/2002
------- BasicPopupMenuUI.java -------
*** /tmp/sccs.gCa4Vd Tue Feb 5 16:20:18 2002
--- BasicPopupMenuUI.java Tue Feb 5 16:17:17 2002
***************
*** 54,60 ****
popupMenu.getLayout() instanceof UIResource)
popupMenu.setLayout(new DefaultMenuLayout(popupMenu, BoxLayout.Y_AXIS));
- popupMenu.setFocusable(false);
popupMenu.setOpaque(true);
LookAndFeel.installBorder(popupMenu, "PopupMenu.border");
LookAndFeel.installColorsAndFont(popupMenu,
--- 54,59 ----
***************
*** 837,845 ****
MenuSelectionManager msm = (MenuSelectionManager)ev.getSource();
MenuElement[] p = msm.getSelectedPath();
JPopupMenu popup = getActivePopup(p);
! if (popup != null && popup instanceof BasicComboPopup) {
! // Combo popups don't transfer focus anywhere, so we do
! // nothing special.
return;
}
--- 836,843 ----
MenuSelectionManager msm = (MenuSelectionManager)ev.getSource();
MenuElement[] p = msm.getSelectedPath();
JPopupMenu popup = getActivePopup(p);
! if (popup != null && !popup.isFocusable()) {
! // Do nothing for non-focusable popups
return;
}
###@###.### 2002-02-05
======================================================================
|
EVALUATION
Name: pzR10082 Date: 02/05/2002
It appears we can safely use the focusable property of the popup
menu. This means, if a popup is not focusable, we will not move focus
anywhere when it shows up. For a developer it is sufficient to make
their popups non-focusable.
###@###.### 2002-02-05
======================================================================
|
Java
