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