<Swing Dev> [10][JDK-6714836] JRootPane.getMaximumSize() returns a width of 0

Prasanta Sadhukhan prasanta.sadhukhan at oracle.com
Mon Sep 4 12:14:57 UTC 2017


Looks fine. But, remove @author tag as we do not use it anymore. Also, 
@modules tag is used to add some internal modules if test needs one, 
here it is not needed, so you remove that too. Add @run tag.

Regards
Prasanta
On 9/4/2017 3:46 PM, Krishna Addepalli wrote:
>
> Hi Prasanta,
>
> Thanks for bringing that up. I have updated the webrev with the test 
> case.
>
> JDK 10 Webrev: 
> http://cr.openjdk.java.net/~pkbalakr/Krishna/6714836/webrev.01/ 
> <http://cr.openjdk.java.net/%7Epkbalakr/Krishna/6714836/webrev.01/>
>
> Krishna
>
> *From:*Prasanta Sadhukhan
> *Sent:* Thursday, August 24, 2017 1:50 PM
> *To:* Krishna Addepalli <krishna.addepalli at oracle.com>; 
> swing-dev at openjdk.java.net
> *Subject:* Re: <Swing Dev> [10][JDK-6714836] 
> JRootPane.getMaximumSize() returns a width of 0
>
> You can add a regression test with the fix as JBS already has one.
>
> Regards
> Prasanta
>
> On 8/24/2017 12:37 PM, Krishna Addepalli wrote:
>
>     Hi All,
>
>     Bug : JDK- 6714836
>     <https://bugs.openjdk.java.net/browse/JDK-6714836>
>     <%3chttps:/bugs.openjdk.java.net/browse/JDK-6714836%3e>
>
>     JDK 10 Webrev:
>     http://cr.openjdk.java.net/~pkbalakr/Krishna/6714836/webrev00/
>     <http://cr.openjdk.java.net/%7Epkbalakr/Krishna/6714836/webrev00/>
>
>     The issue is that when a text label is added to JRootPane and
>     query the maximumLayoutSize, it is returning 0. The root cause is
>     because, JRootPane decides to provide the Minimum width of Menubar
>     (which in this case is 0) and the content pane (which contains the
>     JLabel). Actually, it should return the maximum of the two, since
>     that is what is the layout size needed.
>
>     Thanks,
>
>     Krishna
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/swing-dev/attachments/20170904/7be30cdc/attachment.html>


More information about the swing-dev mailing list