Re: Review Request for 8139169: Action registered for keyboard shortcut is called twice

2015-12-02 Thread Avik Niyogi
Hi All, Please find updated webrev.01 with inputs from comments below: http://cr.openjdk.java.net/~aniyogi/8139169/webrev.01/ With Regards, Avik Niyogi > On 02-Dec-2015, at 5:16 pm, Rajeev Chamyal wrote: > > Hello Avik, > > Can you use

Re: Review Request for 8139169: Action registered for keyboard shortcut is called twice

2015-12-02 Thread Rajeev Chamyal
Hello Avik,   Can you use @requires (os.family == "mac") instead of a method to check the OS type.   Regards, Rajeev Chamyal   From: Avik Niyogi Sent: 30 November 2015 17:22 To: Rajeev Chamyal; Alexander Scherbatiy; Sergey Bylokhov; swing-dev@openjdk.java.net Subject: Review Request for

Re: Review request for JDK-8075084 JOptionPane.showMessageDialog causes JScrollBar to move

2015-12-02 Thread Alexandr Scherbatiy
On 11/11/2015 7:47 AM, Rajeev Chamyal wrote: Hello All, Please review the following fix for Jdk9: Bug: https://bugs.openjdk.java.net/browse/JDK-8075084 Webrev:http://cr.openjdk.java.net/~rchamyal/8075084/webrev.00/ Issue: On running