Re: [9] Review request for 8174845 Bad scaling on Windows with large fonts with Java 9ea

2017-03-16 Thread Alexandr Scherbatiy
I prepared the image [1] which contains the middle.gif icon from the MenuSelectionManagerDemo drawn with scales from the top to the bottom 1, 1.25, 1.5, 2 and the interpolation rendering hint from the left to the right: not set, NEAREST_NEIGHBOR, BILINEAR, BICUBIC. If it may sense to add

Re: [9] Review request for 8174845 Bad scaling on Windows with large fonts with Java 9ea

2017-03-15 Thread Philip Race
I see the fix was just pushed. But my comments are clearly not addressed. -phil. On 3/15/17, 10:11 AM, Philip Race wrote: Does this actually try to address everything they complained about here http://stackoverflow.com/questions/41117190/java-9-on-windows-with-large-fonts which points to

Re: [9] Review request for 8174845 Bad scaling on Windows with large fonts with Java 9ea

2017-03-15 Thread Philip Race
Does this actually try to address everything they complained about here http://stackoverflow.com/questions/41117190/java-9-on-windows-with-large-fonts which points to an image : https://i.stack.imgur.com/fgpBu.png Check boxes are some of it but not all of it. There is what appears to be a

Re: [9] Review request for 8174845 Bad scaling on Windows with large fonts with Java 9ea

2017-03-15 Thread Sergey Bylokhov
> On 3/14/2017 7:31 PM, Sergey Bylokhov wrote: >> The initial image at [1] also show some issues in the submenu pointer and >> the menu corners, did we solve them already? > The submenu icon is updated in the current fix and the issue with a menu > corner is covered by the fix >JDK-8162350

Re: [9] Review request for 8174845 Bad scaling on Windows with large fonts with Java 9ea

2017-03-15 Thread Alexander Zvegintsev
Looks good to me; Thanks, Alexander. On 14/03/2017 13:54, Alexandr Scherbatiy wrote: Hello, Could you review the fix: bug: https://bugs.openjdk.java.net/browse/JDK-8174845 webrev: http://cr.openjdk.java.net/~alexsch/8174845/webrev.00 Metal JCheckBox and JMenuItem icons are updated to

Re: [9] Review request for 8174845 Bad scaling on Windows with large fonts with Java 9ea

2017-03-14 Thread Alexandr Scherbatiy
On 3/14/2017 7:31 PM, Sergey Bylokhov wrote: The initial image at [1] also show some issues in the submenu pointer and the menu corners, did we solve them already? The submenu icon is updated in the current fix and the issue with a menu corner is covered by the fix JDK-8162350

Re: [9] Review request for 8174845 Bad scaling on Windows with large fonts with Java 9ea

2017-03-14 Thread Sergey Bylokhov
The initial image at [1] also show some issues in the submenu pointer and the menu corners, did we solve them already? [1] http://stackoverflow.com/questions/41117190/java-9-on-windows-with-large-fonts > >

[9] Review request for 8174845 Bad scaling on Windows with large fonts with Java 9ea

2017-03-14 Thread Alexandr Scherbatiy
Hello, Could you review the fix: bug: https://bugs.openjdk.java.net/browse/JDK-8174845 webrev: http://cr.openjdk.java.net/~alexsch/8174845/webrev.00 Metal JCheckBox and JMenuItem icons are updated to be drawn by ovals, arcs and polygons. The [1] folder contains screenshots how icons