RFR: 8258970: Disabled JPasswordField foreground color is wrong with GTK LAF [v2]

Abhishek Kumar abhiscxk at openjdk.org
Fri Sep 1 08:00:33 UTC 2023


On Fri, 1 Sep 2023 04:17:04 GMT, Prasanta Sadhukhan <psadhukhan at openjdk.org> wrote:

>> Abhishek Kumar has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Fix extended for TextField, FormattedTextField, Spinner
>
> src/java.desktop/unix/native/libawt_xawt/awt/gtk3_interface.c line 2413:
> 
>> 2411:             && state_type == GTK_STATE_INSENSITIVE && color_type == TEXT_FOREGROUND) {
>> 2412:             widget_type = TEXT_AREA;
>> 2413:         }
> 
> I guess l2403 has same checks which can be clubbed together
> 
> if (widget_type == TEXT_FIELD || widget_type == PASSWORD_FIELD
>             || widget_type == SPINNER_TEXT_FIELD || widget_type == FORMATTED_TEXT_FIELD) {
>         if (state_type == GTK_STATE_SELECTED && color_type == TEXT_BACKGROUND} 
>             || state_type == GTK_STATE_INSENSITIVE && color_type == TEXT_FOREGROUND) {
>             widget_type = TEXT_AREA;
>     }
> }
> 
> 
> What do you think?

I think it's better to clubbed them, updated as per suggestion.

> test/jdk/javax/swing/JPasswordField/TestDisabledPasswordFieldForegroundColor.java line 67:
> 
>> 65:         boolean testFail = false;
>> 66:         robot = new Robot();
>> 67:         robot.setAutoDelay(1000);
> 
> autoDelay normally is 100ms

Done.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/15263#discussion_r1312697730
PR Review Comment: https://git.openjdk.org/jdk/pull/15263#discussion_r1312697914


More information about the client-libs-dev mailing list