RFR: 8274854: Mnemonics for menu containing numeric text not working [v2]
Michael Strauß
mstrauss at openjdk.java.net
Fri Oct 29 12:38:17 UTC 2021
On Fri, 29 Oct 2021 11:23:50 GMT, Ajit Ghaisas <aghaisas at openjdk.org> wrote:
>> Michael Strauß has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Removed isExtendedMnemonic check from isSimpleMnemonic
>
> modules/javafx.controls/src/main/java/com/sun/javafx/scene/control/behavior/TextBinding.java line 238:
>
>> 236: }
>> 237:
>> 238: return !isExtendedMnemonic(s, position);
>
> I am not sure why do we need to check for isExtendedMnemonic() inside isSimpleMnemonic() implementation.
> These two methods should be separate and it is up to the callers to check and take appropriate action.
Done.
-------------
PR: https://git.openjdk.java.net/jfx/pull/647
More information about the openjfx-dev
mailing list