[2.7] bpo-31500: IDLE: Scale default fonts on HiDPI displays. (GH-3639)#6585
[2.7] bpo-31500: IDLE: Scale default fonts on HiDPI displays. (GH-3639)#6585terryjreedy merged 11 commits into
Conversation
|
I didn't backport the changes to pyshell because that code doesn't exist in 2.7. |
Sorry, something went wrong.
|
And seems fix_scaling() is never called. |
Sorry, something went wrong.
I couldn't include #3664 because that code isn't present in 2.7. In |
Sorry, something went wrong.
Thanks. I couldn't backport the first call to fix_scaling() in pyshell, but I missed the second call, which should have been included. This should work now. |
Sorry, something went wrong.
|
It doesn't work because |
Sorry, something went wrong.
Apologies for that. I'm not able to successfully build over the 2.7 branch, so I couldn't test this. Update: I've finally figured out how to build 2.7! :-) I was able to call this on Ubuntu without any issues and it looked OK, but I never had the original HiDPI problem. |
Sorry, something went wrong.
|
I caught the same two omissions as Serhiy did, and then saw your explanations. 2.7 does not have the 8.5 warning box in PyShell because the 8.5 requirement is new in 3.6. |
Sorry, something went wrong.
|
Ping, @terryjreedy. ISTM this should go in (assuming it works). |
Sorry, something went wrong.
|
The patch is likely needed on 2.7 for people with HiRes screens, but I am always nervous patching 2.7. A factor here is that 2.7 is built against 8.5 instead of 8.6, at least on Windows and Mac. Cheryl tested on Ubuntu. I will test on Windows. Tal, did you test on Mac? I presume Serhiy tested this on Linux with a HiRes screen and it worked fine. |
Sorry, something went wrong.
As of 2.7.15, the default python.org installer for macOS supplies its own 8.6.8, like the current 3.x installers do. The deprecated 32-bit/64-bit installers for 2.7.15 and 3.6.x still use 8.5.x at the moment. FWIW, I haven't noticed any resolution issues on current versions of macOS when smoke testing IDLE 3.x or 2.7.x. |
Sorry, something went wrong.
|
The patch should only make a difference (improvement) if one has a high DPI screen, which makes it hard to test. Anyway, IDLE looks fine on Windows. Anyway, I will merge this now and try to remember to ask if someone has tried IDLE on Mac before 2.7.16 is released. |
Sorry, something went wrong.
(cherry picked from commit a96c96f
https://bugs.python.org/issue31500