On Tue, 29 Aug 2023 18:38:35 GMT, Damon Nguyen <dngu...@openjdk.org> wrote:
>> Previously, a new key combination involving the option key was added to Aqua >> LAF for JTextAreas. In doing so, some code was removed that created this >> regression. The regression caused the right option key on Mac OS to >> incorrectly show another KeyPressed event instead of a KeyReleased event >> when pressing and releasing the key. Additionally, the location of the key >> was 'standard' instead of 'right'. Adding back the key mask and its >> following code resolves the issue and doesn't cause any other CI tests to >> fail. >> >> The headful test included displays the key events as they're pressed. After >> the changes, the test correctly shows the right option key's KeyPressed and >> KeyReleased events and shows the location as 'right'. > > Damon Nguyen has updated the pull request incrementally with one additional > commit since the last revision: > > Move curly brace LGTM. Manual test looks good. I guess we cannot automate this test using Robot since the VK_ALT defaults to the left option keypress. ------------- Marked as reviewed by honkar (Committer). PR Review: https://git.openjdk.org/jdk/pull/15432#pullrequestreview-1601067843