On Mon, 19 Aug 2024 06:20:13 GMT, Abhishek Kumar <abhis...@openjdk.org> wrote:

>> mnemonics if gtk dialog were updated by the 
>> https://bugs.openjdk.org/browse/JDK-8080628, the save button was not changes 
>> since it is a default button in that dialog.
>
>> mnemonics if gtk dialog were updated by the 
>> https://bugs.openjdk.org/browse/JDK-8080628, the save button was not changes 
>> since it is a default button in that dialog.
> 
> Seems like the fix is to remove mnemonics for Metal Look and Feel. 
> Mnemonics for Open, Save and Cancel buttons are present on native side 
> (checked with gedit application on ubuntu 22.04). So, I think mnemonics for 
> Save button should also be there in GTK L&F.

> Hmm, the `&S` is already used by
> 
> ```
> FileChooser.pathLabel.textAndMnemonic=&Selection:
> ```
> 
> It looks like that `&S` is more common for the save action, so probably we 
> might want to change the `&Selection:` to something else, e.g. `S&election:` 
> Any thoughts?

Checked in native side and `&S` is the mnemonics for Save button. So, I think 
it's right to have `&S` as mnemonics for `Save` and as you suggested `&e` for 
`Selection`.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/20612#discussion_r1721265040

Reply via email to