On Thu, 4 Jul 2024 00:17:48 GMT, Harshitha Onkar <[email protected]> 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. > > @alisenchung I missed noticing it earlier, since this is an AWT test a more > appropriate location would be` java/awt/TextComponent`, currently it is in > swing test folder. @honkar-jdk @DamonGuy i've moved the test to the awt folder, can you take a look? ------------- PR Comment: https://git.openjdk.org/jdk/pull/19876#issuecomment-2251156593
