➜
This issue trackerhas been migrated toGitHub, and is currentlyread-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.
Created on2017-10-24 13:20 byserhiy.storchaka, last changed2022-04-11 14:58 byadmin. This issue is nowclosed.
| Pull Requests | |||
|---|---|---|---|
| URL | Status | Linked | Edit |
| PR 4106 | merged | serhiy.storchaka,2017-10-24 13:23 | |
| PR 4154 | merged | python-dev,2017-10-28 00:22 | |
| Messages (9) | |||
|---|---|---|---|
| msg304909 -(view) | Author: Serhiy Storchaka (serhiy.storchaka)* | Date: 2017-10-24 13:20 | |
The proposed patch makes the font sample in IDLE font configuration dialog editable. This allows users to test fonts with arbitrary samples. | |||
| msg305098 -(view) | Author: Terry J. Reedy (terry.reedy)* | Date: 2017-10-27 03:47 | |
Thanks for the patch. Adding the feature is somehow easier than I expected. After moving the sample text to module level, which I considered doing before, saving edits for the duration of an IDLE session turned out to also be easy. With 11 point Lucida Console, there is room for 5 more lines, without erasing anything, before anything scrolls off the top.I expect that saving changes across IDLE sessions would be much harder and likely not worth the effort. I think that exploring font choices is likely rare enough that there is little need to do so. | |||
| msg305128 -(view) | Author: Cheryl Sabella (cheryl.sabella)* | Date: 2017-10-27 17:33 | |
I don't know if saving the changes would be too difficult.1. Create a new config file for the text (I think it would clutter the existing config files, but could also add it there).2. Load font_sample_text from the config file.3. In apply(), write the text back to the config file.Issues:1. Have a reset button so that the original sample text could be restored at some point? If so, then need to store the original (current) text somewherem maybe a .def and .cfg version like for the other settings? A reset button would also require some screen space from the text frame. | |||
| msg305135 -(view) | Author: Terry J. Reedy (terry.reedy)* | Date: 2017-10-27 21:18 | |
Cheryl, thanks for explaining, in part, why I don't want to do this ;-). Using the the current .def, .cfg system would mean that the default sample would be frozen. I want to be able to change it. | |||
| msg305141 -(view) | Author: Cheryl Sabella (cheryl.sabella)* | Date: 2017-10-27 23:25 | |
Makes sense. I guess I didn't think of it being frozen because I was thinking of it being more like the Recent Files list, but pre-populated if it were empty. As you said, probably not worth the effort. :-) | |||
| msg305143 -(view) | Author: Terry J. Reedy (terry.reedy)* | Date: 2017-10-28 00:22 | |
New changeseted6554c487fb2403bc88be6deee611c7a4171d33 by Terry Jan Reedy (Serhiy Storchaka) in branch 'master':bpo-31860: Make the font sample in the IDLE font configuration dialog editable. (#4106)https://github.com/python/cpython/commit/ed6554c487fb2403bc88be6deee611c7a4171d33 | |||
| msg305144 -(view) | Author: Terry J. Reedy (terry.reedy)* | Date: 2017-10-28 00:59 | |
New changeset6a2957de08e0c2d73f3124d12874b408cda4633d by Terry Jan Reedy (Miss Islington (bot)) in branch '3.6':bpo-31860: Make the font sample in the IDLE font configuration dialog editable. (GH-4106) (#4154)https://github.com/python/cpython/commit/6a2957de08e0c2d73f3124d12874b408cda4633d | |||
| msg310469 -(view) | Author: Terry J. Reedy (terry.reedy)* | Date: 2018-01-23 04:32 | |
Postscript: I read a week or so ago that a new Windows 10 font chooser will have this feature. There was not much detail and I believe it is currently only part of the Preview package. | |||
| msg333935 -(view) | Author: Terry J. Reedy (terry.reedy)* | Date: 2019-01-18 03:01 | |
Closing#31777 as duplicate of this. | |||
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2022-04-11 14:58:53 | admin | set | github: 76041 |
| 2019-01-18 03:01:39 | terry.reedy | link | issue31777 superseder |
| 2019-01-18 03:01:30 | terry.reedy | set | messages: +msg333935 |
| 2018-01-23 04:32:07 | terry.reedy | set | messages: +msg310469 |
| 2017-10-28 12:55:22 | terry.reedy | set | pull_requests: -pull_request4125 |
| 2017-10-28 01:39:39 | python-dev | set | pull_requests: +pull_request4125 |
| 2017-10-28 01:03:14 | terry.reedy | set | status: open -> closed resolution: fixed stage: patch review -> resolved |
| 2017-10-28 01:02:32 | terry.reedy | set | pull_requests: -pull_request4123 |
| 2017-10-28 00:59:40 | terry.reedy | set | messages: +msg305144 |
| 2017-10-28 00:39:04 | python-dev | set | pull_requests: +pull_request4123 |
| 2017-10-28 00:22:57 | python-dev | set | stage: commit review -> patch review pull_requests: +pull_request4122 |
| 2017-10-28 00:22:52 | terry.reedy | set | messages: +msg305143 |
| 2017-10-27 23:25:30 | cheryl.sabella | set | messages: +msg305141 |
| 2017-10-27 21:18:00 | terry.reedy | set | messages: +msg305135 |
| 2017-10-27 17:33:31 | cheryl.sabella | set | nosy: +cheryl.sabella messages: +msg305128 |
| 2017-10-27 03:47:28 | terry.reedy | set | messages: +msg305098 stage: patch review -> commit review |
| 2017-10-24 13:23:25 | serhiy.storchaka | set | keywords: +patch stage: patch review pull_requests: +pull_request4076 |
| 2017-10-24 13:20:40 | serhiy.storchaka | create | |