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

@prsadhuk @DamonGuy can you take a look at the latest changes?

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

PR Comment: https://git.openjdk.org/jdk/pull/19876#issuecomment-2276206648

Reply via email to