On Fri, 14 Jun 2024 20:28:11 GMT, Phil Race <p...@openjdk.org> wrote:

> Hmm. So .. new public API ? Is this absolutely necessary ?
I don't see why an app would need to call this directly.

`setMnemonicHidden` can be changed to a `protected` member as you pointed out 
it may not be required by an app to call this method.
But I guess the `isMnemonicHidden` should be public API.

> And it would need a CSR, and it is too late for 23 anyway.

Will update the `@since to 24` for `isMnemonicHidden` method if we are ok with 
`isMnemonicHidden` method's access modifier.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/18992#discussion_r1644412541

Reply via email to