On Wed, 12 Aug 2026 06:25:38 GMT, Prasanta Sadhukhan <[email protected]> 
wrote:

>> Aqua classes for text components enables dragEnabled by default although 
>> setDragEnabled spec cites `"The default value of the dragEnabled property is 
>> false. "` and there is nothing in Aqua L&F that mentions otherwise as to why 
>> the default is overridden.
>> Other L&F dont override the mode and the code is there from macosx port days 
>> so it seems to be an oversight.
>> 
>> The code is deleted.
>> Test is added to check 
>>  - default value is honoured in all installed L&Fs
>>  - explicit application setting of drag mode is preserved on L&F change
>> 
>>  CI testing is ok.
>> 
>> ---------
>> - [x] I confirm that I make this contribution in accordance with the 
>> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai).
>
> Prasanta Sadhukhan has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   Check for headless

test/jdk/javax/swing/text/JTextComponent/TextComponentDragEnabledTest.java line 
37:

> 35:  * @summary Checks dragEnabled defaults and explicit settings across 
> installed L&Fs
> 36:  * @run main/othervm -Djava.awt.headless=true TextComponentDragEnabledTest
> 37:  */

I believe this sub-test will fail on the headless linux system since gtk will 
throw UnsupportedLookAndFeelException?

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

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

Reply via email to