On Mon, 12 Oct 2020 08:01:43 GMT, Tejpal Rebari <treb...@openjdk.org> wrote:
> Please review the following fix for jdk16. > > Issue : There is a JToggleButton that will post/take down a JPopupMenu when > the button is selected. If the button is selected and the menu is not posted > the action listener will post the menu. If the button is selected and the > menu is displayed the action listener will take the menu down. The use case > is: > 1 - select button > 2 - menu posted > 3 - select button > 4 - menu taken down > > With MetalLookAndFeel the above use case works fine, but with > WindowsLookAndFeel the second button selection does not fire the > actionPerformed event, button needs to be selected third time for the menu > to be taken down. > > The issue is that the button must be selected twice after the menu is posted > to have the actionPerformed event to fire when using the Windows look and > feel. > > Fix : MouseGrabber is not removed while uninstalling the listeners in the > BasicPopupMenuUI. > By removing the mouseGrabber in the uninstallListeners() methods fixes this > issue. > > Added a test to test the same in all the LookAndFeels This pull request has now been integrated. Changeset: 65ca5c66 Author: Tejpal Rebari <treb...@openjdk.org> URL: https://git.openjdk.java.net/jdk/commit/65ca5c66 Stats: 169 lines in 5 files changed: 165 ins; 0 del; 4 mod 8048109: JToggleButton does not fire actionPerformed under certain conditions Reviewed-by: serb, psadhukhan, vdyakov ------------- PR: https://git.openjdk.java.net/jdk/pull/600