Hi Rajeev,
Since this bug is closed now, you should open a new one. In this new
bug, indicate it's a regression by adding 'regression' and
'regression_8067346' labels to it, also add 'relates to' link to the
original bug that caused this regression.
Regards,
Alexey
On 05.10.2015 7:20, Rajeev Chamyal wrote:
Hello Alexey,
Thanks for noticing it. Yes, we need to resolve it.
Alexander, Should I create a new bug for this or I can submit against the
existing bug.
Regards,
Rajeev Chamyal
-----Original Message-----
From: Alexey Ivanov
Sent: Friday, October 02, 2015 7:30 PM
To: Alexander Scherbatiy; Rajeev Chamyal
Cc: swing-dev@openjdk.java.net
Subject: Re: <Swing Dev> Review request for JDK-8067346: Swing submenu has a
changed starting offset
Hi Alexandr, Rajeev,
Isn't there a copy-paste bug in OSInfo.java?
windowsVersionMap.put(WINDOWS_VISTA.toString(), WINDOWS_7);
I believe it should have been
windowsVersionMap.put(WINDOWS_7.toString(), WINDOWS_7);
I understand it's a bit too late since the fix is pushed now but it might be
problem and should be resolved.
Regards,
Alexey
On 30.09.2015 14:45, Alexander Scherbatiy wrote:
The fix looks good to me.
Thanks,
Alexandr.
On 9/30/2015 1:01 PM, Rajeev Chamyal wrote:
Hello Alexander,
Thanks for the review. I have updated webrev as per review comments.
Bug: https://bugs.openjdk.java.net/browse/JDK-8067346
Webrev: http://cr.openjdk.java.net/~psadhukhan/rajeev/8067346/webrev.02/
Regards,
Rajeev Chamyal
-----Original Message-----
From: Alexander Scherbatiy
Sent: Tuesday, September 29, 2015 6:59 PM
To: Rajeev Chamyal
Cc: Sergey Bylokhov; Alexander Zvegintsev; swing-dev@openjdk.java.net
Subject: Re: Review request for JDK-8067346: Swing submenu has a
changed starting offset
On 9/29/2015 12:38 PM, Rajeev Chamyal wrote:
Hello All,
Please review the following updated webrev.
Bug: https://bugs.openjdk.java.net/browse/JDK-8067346
Webrev:
http://cr.openjdk.java.net/~psadhukhan/rajeev/8067346/webrev.01/
Verified the fix on windows10 and windows8.
bug8067346.java.html
66 try {
67 // set windows look and feel
68 UIManager.setLookAndFeel(LOOK_FEEL);
69 } catch (UnsupportedLookAndFeelException e) {
70 System.out.println("Windows look and feel is not
supported");
71 } catch (ClassNotFoundException e) {
72 } catch (InstantiationException e) {
73 } catch (IllegalAccessException e) {
74 }
Is the test designed to work only with Windows L&F? If yes, it
needs just to pass the test instead of running it with the default
one when the Windows L&F is not available.
Thanks,
Alexandr.
Regards,
Rajeev Chamyal
-----Original Message-----
From: Sergey Bylokhov
Sent: Wednesday, September 23, 2015 11:44 PM
To: Rajeev Chamyal; Alexander Zvegintsev; swing-dev@openjdk.java.net;
Alexander Scherbatiy
Subject: Re: Review request for JDK-8067346: Swing submenu has a
changed starting offset
Hi, Rajeev.
Can you check Windows 10 as well? Thanks.
On 22.09.15 13:09, Rajeev Chamyal wrote:
Hi,
Please review the following fix for jdk9:
Bug:https://bugs.openjdk.java.net/browse/JDK-8067346
Webrev:
http://cr.openjdk.java.net/~psadhukhan/rajeev/8067346/webrev.00/
WindowsLookAndFeel.java overrides basic look and feel properties
based on different OS versions XP, Vista etc.
Current implementation is applying Vista defaults to Windows7 as
well and as a result submenu on windows7 has changed offset.
Added checks in WindowsLookAndFeel.java to skip Vista menu defaults
for Windows7.
Verified changes on Netbeans9 Dev build and SwingSet2 applications.
Regards,
Rajeev Chamyal
