On Thu, 30 Jul 2026 19:19:51 GMT, Phil Race <[email protected]> wrote:

>> test/jdk/javax/swing/text/JTextComponent/TextComponentDragEnabledTest.java 
>> line 36:
>> 
>>> 34: 
>>> 35: import javax.swing.JEditorPane;
>>> 36: import javax.swing.JPasswordField;
>> 
>> `JPasswordField` is not currently used, but it could be added to the test, 
>> along with `JFormattedTextField`.
>
> Isn't JPasswordField special ? I'd expect it to never allow the password to 
> be dragged out.
> i.e although it can't stop someone setting dragEnabled to true, it should 
> ignore the setting.
> But if this test just verifies nothing changes the default setting, that's 
> OK. I'd just want to make it clear that it is ignored.

> `JPasswordField` is not currently used, but it could be added to the test, 
> along with `JFormattedTextField`.

Seems the JFormattedTextField should be covered as well.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/32101#discussion_r3738437896

Reply via email to