Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[3.8] bpo-37530: simplify, optimize and clean up IDLE code context (GH-14675) #14807

Merged
merged 1 commit into from
Jul 17, 2019

Conversation

miss-islington
Copy link
Contributor

@miss-islington miss-islington commented Jul 17, 2019

  • Only create CodeContext instances for "real" editors windows, but
    not e.g. shell or output windows.
  • Remove configuration update Tk event fired every second, by having
    the editor window ask its code context widget to update when
    necessary, i.e. upon font or highlighting updates.
  • When code context isn't being shown, avoid having a Tk event fired
    every 100ms to check whether the code context needs to be updated.
  • Use the editor window's getlineno() method where applicable.
  • Update font of the code context widget before the main text widget
    (cherry picked from commit 7036e1d)

Co-authored-by: Tal Einat [email protected]

https://bugs.python.org/issue37530

…H-14675)

* Only create CodeContext instances for "real" editors windows, but
  not e.g. shell or output windows.
* Remove configuration update Tk event fired every second, by having
  the editor window ask its code context widget to update when
  necessary, i.e. upon font or highlighting updates.
* When code context isn't being shown, avoid having a Tk event fired
  every 100ms to check whether the code context needs to be updated.
* Use the editor window's getlineno() method where applicable.
* Update font of the code context widget before the main text widget
(cherry picked from commit 7036e1d)

Co-authored-by: Tal Einat <[email protected]>
@miss-islington
Copy link
Contributor Author

@taleinat: Status check is done, and it's a success ✅ .

@miss-islington miss-islington merged commit 6cf7c45 into python:3.8 Jul 17, 2019
@miss-islington miss-islington deleted the backport-7036e1d-3.8 branch July 17, 2019 08:44
@miss-islington
Copy link
Contributor Author

@taleinat: Status check is done, and it's a success ✅ .

1 similar comment
@miss-islington
Copy link
Contributor Author

@taleinat: Status check is done, and it's a success ✅ .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
performance Performance or resource usage
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants