On 5/5/2016 2:24 PM, Sergey Bylokhov wrote:
On 05.05.16 14:11, Alexandr Scherbatiy wrote:
Note that the component GConfig is not necessary the config which is
used in drawing, and can be different from the GConfig of Graphics
passed to the paint() method.
This is still necessary change for
The fix looks good to me.
Thanks,
Alexandr.
On 5/9/2016 8:52 AM, Prem Balakrishnan wrote:
Hi Sergey,
Thankyou for the Review.
http://cr.openjdk.java.net/~pkbalakr/6421058/webrev.01/
Updated patch as per review comments.
[mainFrame.dispose(), by mistake I missed to uncomment which was comment
The first approach implies that a user should change all
PopupFactory.getPopup(owner, contents, x, y) calls to
OverridenPopupFactory.getPopup(owner, contents, x, y, true) to get a
heavy-weight popup in his code.
The second one looks better to me. It may have sense to restrict it only
to 2 p
Thanks, Looks fine.
I suppose that is a start. We will likely want more tests later.
One thing to fix before this is pushed :
45 }catch(Exception cnf) {
missing spaces here.
-phil.
On 05/06/2016 12:22 AM, Yuri Nesterenko wrote:
Hi Phil,
yes, thank you. I've added a simpler black box test
for both Lin
Hello All,
Please review the updated webrev.
http://cr.openjdk.java.net/~rchamyal/8147521/webrev.app.00/
Update: Added test case.
Regards,
Rajeev Chamyal
From: Rajeev Chamyal
Sent: 06 May 2016 17:14
To: Sergey Bylokhov; Alexander Scherbatiy; swing-dev@openjdk.java.net
Su