Description
In two places IDLE uses incorrect unit for screen dimension:
cpython/Lib/idlelib/configdialog.py
Line 114 in 983efcf
cpython/Lib/idlelib/searchbase.py
Line 89 in 983efcf
It uses "5px", but the only documented valid suffixes are "c", "i", "m" and "p". Tk ignores the rest in versions < 8.7, but in 8.7 and 9.0 this is an error. And this is for good, because "px" did not mean pixels, as was expected, but printer's points (1/72 inch).
If we want to keep the same look, we should change "px" to "p". But if it originally should be in pixels, we should remove the suffix. In all other places padding is specified in pixels, and this makes sense, so I believe the latter option is better.
Linked PRs
Metadata
Metadata
Assignees
Projects
Status