RFR: 8078471: The BACKSPACE key doesn't work with the special options"-client -Dswing.defaultlaf=com.sun.java.swing.plaf.gtk.GTKLookAndFeel" [v2]

[Abhishek Kumar]

On Thu, 24 Nov 2022 13:00:52 GMT, Prasanta Sadhukhan <psadhukhan at openjdk.org> wrote:

> I wonder how it works in Nimbus as this mapping is not there in Synth/NimbusLookAndFeel and still it works and both GTK/Nimbus extends Synth. Can you please check?


 There is a mapping for squarebutton in skin.laf `<uiProperty name="squareButton" type="BOOLEAN" value="false"/>`. 
Setting it true is having similar effect in Nimbus LAF also. So, I think in Nimbus LAF the squarebutton property is set through skin.laf file.

> You have a similar fix for GTK https://github.com/openjdk/jdk/commit/9d0009e92b790b43153e3db353db775e6ff731cb
Can it be reused in this case also?

Don't think it can be reused in this case as the event is triggered from either double click or keyboard (Enter) on "../" directory present in JList. 
I will verify this once.

> Also, is it not possible to create automated test, like calling JFileChooser.getCurrentDirectory() and fail if it's not the parent directory (or if it still remains in present directory) after BACKSPACE is pressed?

I tried implementing an automated test case but in GTK LAF the cursor is focused on textfield and the backspace event is not triggered to change directory.

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

PR: https://git.openjdk.org/jdk/pull/11291