[3.6] bpo-27099: IDLE - Convert built-in extensions to regular featur…#3487
Merged
terryjreedy merged 1 commit intopython:3.6python/cpython:3.6from Sep 11, 2017
terryjreedy:backport-58fc71c-3.6terryjreedy/cpython:backport-58fc71c-3.6Copy head branch name to clipboard
Merged
[3.6] bpo-27099: IDLE - Convert built-in extensions to regular featur…#3487terryjreedy merged 1 commit intopython:3.6python/cpython:3.6from terryjreedy:backport-58fc71c-3.6terryjreedy/cpython:backport-58fc71c-3.6Copy head branch name to clipboard
terryjreedy merged 1 commit intopython:3.6python/cpython:3.6from
terryjreedy:backport-58fc71c-3.6terryjreedy/cpython:backport-58fc71c-3.6Copy head branch name to clipboard
Conversation
…es (pythonGH-2494) About 10 IDLE features were implemented as supposedly optional extensions. Their different behavior could be confusing or worse for users and not good for maintenance. Hence the conversion. The main difference for users is that user configurable key bindings for builtin features are now handled uniformly. Now, editing a binding in a keyset only affects its value in the keyset. All bindings are defined together in the system-specific default keysets in config- extensions.def. All custom keysets are saved as a whole in config- extension.cfg. All take effect as soon as one clicks Apply or Ok. The affected events are '<<force-open-completions>>', '<<expand-word>>', '<<force-open-calltip>>', '<<flash-paren>>', '<<format-paragraph>>', '<<run-module>>', '<<check-module>>', and '<<zoom-height>>'. Any (global) customizations made before 3.6.3 will not affect their keyset- specific customization after 3.6.3. and vice versa. Inital patch by Charles Wohlganger, revised by Terry Jan Reedy. (cherry picked from commit 58fc71c)
Contributor
|
While this is difficult to review in detail, it looks good overall. +1 on moving forward with it. |
Member
Author
|
Thank you for taking a look. I used the news blurb to tell users what I think they need to know. I just posted to the issue a high-level 'developer' view of what the patch does. Even with that in mind, reviewing was tough. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
…es (GH-2494)
About 10 IDLE features were implemented as supposedly optional
extensions. Their different behavior could be confusing or worse for
users and not good for maintenance. Hence the conversion.
The main difference for users is that user configurable key bindings
for builtin features are now handled uniformly. Now, editing a binding
in a keyset only affects its value in the keyset. All bindings are
defined together in the system-specific default keysets in config-
extensions.def. All custom keysets are saved as a whole in config-
extension.cfg. All take effect as soon as one clicks Apply or Ok.
The affected events are '<>', '<>',
'<>', '<>', '<>',
'<>', '<>', and '<>'. Any
(global) customizations made before 3.6.3 will not affect their keyset-
specific customization after 3.6.3. and vice versa.
Inital patch by Charles Wohlganger, revised by Terry Jan Reedy.
(cherry picked from commit 58fc71c)
https://bugs.python.org/issue27099