FIx looks good to me too.
Regards
Prasanta
On 1/20/2016 5:06 PM, Alexander Scherbatiy wrote:
The fix looks good to me.
Thanks,
Alexandr.
On 1/20/2016 9:52 AM, Rajeev Chamyal wrote:
Hello Alexandr,
Thanks for the review.
This issue is seen only if we have glue added to any SynthToolBar.
While doing layoutContainer we need to distribute the empty space
among glue and currently its done by finding minimum width of parent
using minimumLayoutSize which doesn't consider preferred size of
child comps.
The preferred layout size is not getting called in this case. I have
replaced the minimumLayoutSize with preferredLayoutSize and updated
the webrev.
http://cr.openjdk.java.net/~rchamyal/8146276/webrev.01/
Regards,
Rajeev Chamyal
-----Original Message-----
From: Alexander Scherbatiy
Sent: 19 January 2016 21:33
To: Rajeev Chamyal; Sergey Bylokhov; Prasanta Sadhukhan;
swing-dev@openjdk.java.net
Subject: Re: Review request for JDK-8146276 : Right aligned ToolBar
component does not appear
On 12/01/16 08:21, Rajeev Chamyal wrote:
Hello All,
Please review the following fix for Jdk9:
Bug : https://bugs.openjdk.java.net/browse/JDK-8146276
Webrev: http://cr.openjdk.java.net/~rchamyal/8146276/webrev.00/
Issue : Right aligned ToolBar component does not appear
Cause: While calculating the minimum layout size for the components
SynthToolBar is not checking if preferred size is set for the
components.
Fix: Updated the minimumLayoutSize method of SynthToolBarUI.java to
check preferred size of components as well.
Could you give more details why it is not enough to properly layout
buttons when the minimum layout size is only calculated on the
minimum size and the preferred layout size is based on the buttons
preferred size?
Thanks,
Alexandr.
Regards,
Rajeev Chamyal
