Hello,

 Could you review the updated fix:
   http://cr.openjdk.java.net/~alexsch/8076545/webrev.01/

  - inverted scale is used
  - round is used instead of ceil

On 7/9/2015 2:32 AM, Phil Race wrote:
Hi,

- Note that the size you are re-scaling is the real pixel size GDI says to use, so this is presumably to prevent double-scaling ? I can't really tell from
  the attached images if the size the user perceives in the Win L&F is
  the same as the rest of the native desktop or much bigger.
  In other words is Win L&F just right and Metal too small, or is
  Metal just right and Win L&F too big ?
The Win L&F is twice bigger than Metal L&F, window, and component sizes.

- why use ceil instead of rounding ? For 120 dpi and a reported 14 pixel font you probably want the closest int to "11.2" which is 11 but you will return 12.

- I am not familiar with what the effect is of the change in ThemeReader_getSize.
  Can you explain this.
It returns size of requested native controls. For example, Windows L&F can request a size of the expanded or collapsed Tree icon.


- Sergey's point about multi-mon should be checked out.
Windows 8.1 has option "Let me choose one scaling level for all my displays". If I unset it I am able to change the size of all items. However, the DPI which returns GetDPIForMonitor is still 2 on HiDPI displays.

   Thanks,
   Alexandr.


-phil.

On 7/3/2015 10:06 AM, Alexander Scherbatiy wrote:

Hello,

Could you review the fix:
  bug: https://bugs.openjdk.java.net/browse/JDK-8076545
  webrev: http://cr.openjdk.java.net/~alexsch/8076545/webrev.00

Windows L&F fonts and size of components are rescaled according to the system DPI.

Thanks,
Alexandr.



Reply via email to