Re: [9] JDK-8049069 : JButton incorrect behaviour on button release

Hi, Ajit. Is this bug a regression of JDK-7088744 or JDK-8049069, or it works this way in jdk6 as well? I guess the code in jdk6 was something like this: public static boolean isLeftMouseButton(MouseEvent anEvent) { return ((anEvent.getModifiers() & InputEvent.BUTTON1_MASK) != 0);

Re: [9] Review Request for several test bugs: 8150535, 8151033, 8151037 etc.

Sergey, I'm sorry, filing JDK-8152693, I found an error in that version. A new one is http://cr.openjdk.java.net/~yan/8150535/webrev.02 The spawned processes were provided with addExports. Shame on me! -yan On 03/24/2016 05:34 PM, Sergey Bylokhov wrote: cc swing-dev On 24.03.16 17:17, Yuri N

Re: [9] Review Request for several test bugs: 8150535, 8151033, 8151037 etc.

cc swing-dev On 24.03.16 17:17, Yuri Nesterenko wrote: Hi, please review this test-only fix, a leftover from Jigsaw M3 integration. Some 7 tests fixed but note that not all of them always pass even fixed. Webrev: http://cr.openjdk.java.net/~yan/8150535/webrev.01 I need to file a bug about the

Re: [9] Review request for 8132119 Provide public API for text related methods in SwingUtilities2

On 24.03.16 16:52, Alexander Scherbatiy wrote: On 24/03/16 10:36, Semyon Sadetsky wrote: Hi Alexander, Could you answer one question: Why did you choose default interface methods to implement TextUIDrawing and not implement them in DefaultTextUIDrawing having declarations only in the interface?

Re: [9] Review request for 8132119 Provide public API for text related methods in SwingUtilities2

On 3/24/2016 4:52 PM, Alexander Scherbatiy wrote: On 24/03/16 10:36, Semyon Sadetsky wrote: Hi Alexander, Could you answer one question: Why did you choose default interface methods to implement TextUIDrawing and not implement them in DefaultTextUIDrawing having declarations only in the int

Re: [9] Review request for 8132119 Provide public API for text related methods in SwingUtilities2

On 24/03/16 10:36, Semyon Sadetsky wrote: Hi Alexander, Could you answer one question: Why did you choose default interface methods to implement TextUIDrawing and not implement them in DefaultTextUIDrawing having declarations only in the interface? AFAIK the common point of view is default met

Re: Review Request for 6439354 : Win L&F: TitledBorder colors are not from desktop

+1 thanks. On 22.03.16 11:56, Prem Balakrishnan wrote: Hi Sergey, Updated test as per the review comments. Webrev: http://cr.openjdk.java.net/~arapte/prem/6439354/webrev.01/ Regards, Prem -Original Message- From: Sergey Bylokhov Sent: Monday, March 21, 2016 7:31 PM To: Prem Balakris

Re: CFV: New Swing Group Member: Semyon Sadetsky

Vote: yes. Thanks, Alexandr. On 21/03/16 10:01, Alexander Scherbatiy wrote: I hereby nominate Semyon Sadetsky (OpenJDK user name: ssadetsky) to Membership in the Swing Group. Semyon is active member of Swing group and contributed a lot of fixes which include Swing TimerQueue race conditio

[9] JDK-8049069 : JButton incorrect behaviour on button release

Hi, Bug : https://bugs.openjdk.java.net/browse/JDK-8049069 Issue : A JButton which is pressed using left mouse button gets released if right mouse button is pressed and released. Root cause : --- In file BasicButtonListener.java, mousePressed() and mouseReleased() methods check wh

Re: [9] Review request for JDK-8150225 api/javax_swing/text/AbstractWriter/index_indent failed

Hi Rajeev, The fix looks fine to me. With Regards, Avik Niyogi -Original Message- From: Rajeev Chamyal Sent: 23 March 2016 14:45 To: Sergey Bylokhov; Alexander Scherbatiy; [email protected] Subject: RE: [9] Review request for JDK-8150225 a

Re: Review Request of 8137169 : [macosx] Incorrect minimal heigh of JTabbedPane with more tabs

Looks good to me. Regards, Rajeev Chamyal From: Avik Niyogi Sent: 24 March 2016 12:54 To: Rajeev Chamyal; Alexander Scherbatiy; Sergey Bylokhov; [email protected] Subject: Re: Review Request of 8137169 : [macosx] Incorrect minimal heigh of JTabbedPane with more tabs Hi All

Re: Review Request for 6439354 : Win L&F: TitledBorder colors are not from desktop

Looks good to me. Regards, Rajeev Chamyal -Original Message- From: Semyon Sadetsky Sent: 24 March 2016 11:45 To: Prem Balakrishnan; Sergey Bylokhov; Rajeev Chamyal; Alexander Scherbatiy; [email protected] Subject: Re: Review Request for 6439354 : Win L&F: TitledBorder colors ar

Re: Review Request of 8137169 : [macosx] Incorrect minimal heigh of JTabbedPane with more tabs

Hi All, Please review code changes as per inputs received. http://cr.openjdk.java.net/~aniyogi/8137169/webrev.03 As SCROLL_TAB_LAYOUT is the default layout for Aqua LAF, with implementation within the derived AquaTruncatedTabbedPaneLayout

Re: Review Request of 8137169 : [macosx] Incorrect minimal heigh of JTabbedPane with more tabs

Hello Avik, x variable on line 2195 is not used anywhere. Do we need for loop also? Regards, Rajeev Chamyal From: Avik Niyogi Sent: 24 March 2016 12:19 To: Alexander Scherbatiy Cc: Sergey Bylokhov; Rajeev Chamyal; [email protected] Subject: Re: Review Request of 8137169 : [m