> The test instructions say that disabled PopupMenus should not have shortcuts 
> shown, but on MacOS, these shortcuts still appear. When checking native 
> MacOS15 behavior, disabled PopupMenus still have shortcuts shown. Since the 
> test doesn't modify the popup's shortcuts other than adding the shortcut for 
> `A`, it makes sense that the result matches native behavior. So, I modified 
> the test instructions instead to exclude MacOS from this step.

Damon Nguyen has updated the pull request incrementally with one additional 
commit since the last revision:

  Review comments

-------------

Changes:
  - all: https://git.openjdk.org/jdk/pull/23402/files
  - new: https://git.openjdk.org/jdk/pull/23402/files/145cf9fd..1cc6c40e

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=23402&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=23402&range=00-01

  Stats: 2 lines in 1 file changed: 0 ins; 1 del; 1 mod
  Patch: https://git.openjdk.org/jdk/pull/23402.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/23402/head:pull/23402

PR: https://git.openjdk.org/jdk/pull/23402

Reply via email to