On 5/11/2016 1:17 PM, Rajeev Chamyal wrote:
Hello All,
Please review the updated webrev.
Bug : https://bugs.openjdk.java.net/browse/JDK-8147521
Webrev: http://cr.openjdk.java.net/~rchamyal/8147521/webrev.03/
Update: Added a new protected method getpop in PopupFactory.java with a Boolean
parameter for specifying heavy weight popup.
The provided method looks good to me.
I would leave the javadoc review to the native speakers. What I only
see that there is a missed space in {@codey}, typo in "paramaters" word
and probably there should be a comma before "otherwise false".
Thanks,
Alexandr.
Regards,
Rajeev Chamyal
-----Original Message-----
From: Alan Snyder [mailto:javali...@cbfiddle.com]
Sent: 11 May 2016 03:39
To: Alexandr Scherbatiy
Cc: swing-dev@openjdk.java.net
Subject: Re: <Swing Dev> [9] Review request for JDK-8147521 [macosx] Internal
API Usage: setPopupType used to force creation of heavyweight popup
I use only heavy weight.
On May 10, 2016, at 12:46 PM, Alexandr Scherbatiy
<alexandr.scherba...@oracle.com> wrote:
Do you need to use medium-weight popups in your application or
light/heavy-weight popups are enough?
Thanks,
Alexandr.
On 5/10/2016 10:23 PM, Alan Snyder wrote:
On May 10, 2016, at 5:58 AM, Sergey Bylokhov <sergey.bylok...@oracle.com> wrote:
Hi, Alan.
Can you please take a look to the proposed solutions? Thanks.
Approach 2 matches what I currently do. The problem noted by Rajeev does not
happen because my popup factory calls setPopupType() on each call to the public
getPopup() before invoking the superclass method.
I think the original version of Approach 1 would work. My factory would
override the public getPopup() method and pass true to the five parameter
method.
I think the revised version of Approach 1 does not work for me because the new
flag is only tested if the first attempt to create a popup fails.