On Fri, 12 Dec 2025 08:12:29 GMT, Prasanta Sadhukhan <[email protected]> 
wrote:

>> Issue is if one internal frame is open and maximized and another internal 
>> frame is created which is initialized with `setMaximum(true)` then after 
>> opening the second internal frame the new internal frame is opened as 
>> non-maximized
>> 
>> The fix makes sure the 2nd internal frame remains maximised if it is set 
>> with maximized attrbute, that is
>> If the current frame is maximized, transfer that attribute to the frame 
>> being activated. 
>> which is also mentioned in this comment 
>> https://github.com/openjdk/jdk/blob/46ee8d550e42f756dd04fa67ed549c93b632df77/src/java.desktop/windows/classes/com/sun/java/swing/plaf/windows/WindowsDesktopManager.java#L70-L71
>>  
>> This code seems to be added for 
>> [JDK-5036083](https://bugs.openjdk.org/browse/JDK-5036083) which expects
>> When a frame is maximized and then minimized, the next frame should NOT be 
>> maximized.
>> 
>> It still honours that fix as the test mentioned in JDK-5036083 works as 
>> expected as mentioned above along with fixing the present issue..
>
> Prasanta Sadhukhan has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   Test fix

Looks reasonable.

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

Marked as reviewed by kizune (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/28502#pullrequestreview-3576256087

Reply via email to