On Tue, 23 Sep 2025 17:47:17 GMT, Damon Nguyen <[email protected]> wrote:

>> Thanks, I reversed the order of the release.
>
>> Also you may want to check another test `EditableFocusTest` which also sets 
>> mnemonic "B" on a JButton and it passes in macOS..One difference is it uses 
>> setMnemonic(char) instead of setMnemonic(int)
> 
> I see that test's `setMnemonic`. This tabbedPane's API is forced to use 
> `setMnemonicAt(int, int)`. I figured, since mnemonics don't seem to be 
> supported on macOS anyway, it does not really make sense to ProblemList this 
> test for macOS only, and instead just exclude macOS from the test since the 
> test fails with the current implementation using `setMnemonicAt`.

It is not macOS but Aqua L&F problem it seems...
javax/swing/JTabbedPane/8134116/Bug8134116.java also uses `setMnemonicAt` but 
it uses Nimbus and it seems to pass on macOS as per CI
but I guess its ok to restrict as native macOS where it doesn't seem to work 
will run on its own Aqua L&F kind of interface

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27371#discussion_r2374127332

Reply via email to