On Tue, 6 Aug 2024 21:36:15 GMT, Alisen Chung <ach...@openjdk.org> wrote:
>> Currently the bug described in the issue is that the colors of the
>> TextComponents do not change when setting TextComponents to uneditable. The
>> default uneditable color (SystemColor.control) happens to be the same as the
>> default for the editable color for some L&Fs, so the fix may not be
>> initially noticeable. However, the bug still exists where the the color is
>> not being changed when changing between editable and uneditable. You can
>> check by changing TextComponent.getBackground() code to return Color.GRAY on
>> line 342 and you can see that TextComponents are not changing to a gray
>> background when set to uneditable.
>>
>> This fix adds a private setBackground method in TextComponent so that
>> TextArea and TextField can change the background color to the correct color
>> (SystemColor.control) when set uneditable by overriding the TextComponent
>> setEditable. You can verify the fix by changing this color to Color.GRAY and
>> verifying the backgrounds change to gray when the TextComponents are
>> disabled.
>
> Alisen Chung has updated the pull request incrementally with one additional
> commit since the last revision:
>
> use getbackground
src/java.desktop/share/classes/java/awt/TextField.java line 495:
> 493: public synchronized void setEditable(boolean b) {
> 494: super.setEditable(b);
> 495: Color defaultBackground =
> UIManager.getColor("TextField.background");
It still uses UIManager.getColor here and this property is again set in L&Fs
mentioned for`TextArea.background`
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/19876#discussion_r1711448046
