On Fri, 10 Jul 2026 05:05:39 GMT, Jyothilekshmi <[email protected]> wrote:

>> Please review this
>> 
>> Platforms tested:
>> Windows 11, macOS, Linux, Ubuntu
>> 
>> Summary:
>> Updated `PanelRepaint` to use `BorderLayout` for the test frame and its main 
>> panel instead of relying on `FlowLayout` with manually assigned bounds.
>> 
>> - [x] I confirm that I make this contribution in accordance with the 
>> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai).
>
> Jyothilekshmi has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Fix PanelRepaint viewport sizing

It is really not easy to know what the original test author was doing here and 
why but I'd suppose that s/he had a reason for creating this custom scroll 
panel. I think the test could just make the top-level window non-resizable and 
leave it at that. The rest is irrelevant to what it is testing.
The bug & PR summary seems quite wrong to me (the test does NOT fail), as well 
as being confusing.
Better might be  "PanelRepaint.java should not be resizable".

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

PR Comment: https://git.openjdk.org/jdk/pull/31687#issuecomment-5007160852

Reply via email to