bpo-17942: Improve IDLE debugger user interface#2454
Conversation
|
@roseman, thanks for your PR! By analyzing the history of the files in this pull request, we identified @kbkaiser, @terryjreedy and @ned-deily to be potential reviewers. |
Sorry, something went wrong.
|
Turn Debugger on, Run file, and |
Sorry, something went wrong.
terryjreedy
left a comment
There was a problem hiding this comment.
See posted error
Sorry, something went wrong.
|
Thanks Terry. I've added the missing routine to filelist.py in my branch. It looks like this is automatically added to this pull request...? (excuse the newbie question) |
Sorry, something went wrong.
|
Yes, when one pushes a commit to a branch on a github repository that is the basis for a pull request, then the commit appears on the pull request. You probably pushed to 'origin'. If I created a local pr_2454 branch and made some commits, and did 'git push https://github.com/roseman pr_2454:debugger-ui-17942', those commits would become part of this. Please ask questions. The devguide is both helpful and confusing. |
Sorry, something went wrong.
|
To try and help move older pull requests forward, we are going through and backfilling 'awaiting' labels on pull requests that are lacking the label. Based on the current reviews, the best we can tell in an automated fashion is that a core developer requested changes to be made to this pull request. If/when the requested changes have been made, please leave a comment that says, |
Sorry, something went wrong.
|
From AppVeyor Add 'r' prefix to regex |
Sorry, something went wrong.
|
replaced by newer PR |
Sorry, something went wrong.
Previous versions were dependent on a parallel 'ui' library that handled some of the 8.4/8.5 differences in Tk, as well as provided some utilities. In terms of functionality, this version omits the tooltips found in the original one (code left but commented out to indicate where they'd go), but otherwise is the same. Some pieces, e.g. context menu bindings, windowing system calls, etc. should probably still be pulled into a separate ui module.
https://bugs.python.org/issue17942