gh-62142: IDLE - rework debugger UI#22947

roseman

updated to use ttk widgets throughout
simplified controls, changed go/step/etc. to image buttons
overall layout is a two-pane horizontal window (controls, stack, status on left, variables on right)
treeview widget used to replace custom scrolled-listbox stackviewer and custom canvas-based namespaceviewer
integrated locals/globals into a single treeview on right; can collapse/expand each independently instead of having checkboxes to control whether shown or not
removed scrolledlist.py (was previously used only by stackview)

Issue: IDLE Debugger: Improve GUI #62142

E-Paine

Is it possible to do without using preset image sizes and font? (if for some reason the default font is larger than normal, the difference is very obvious). Even if we cannot change the image and font sizes dynamically, I think we should avoid fixing the family.

- updated to use ttk widgets throughout - simplified controls, changed go/step/etc. to image buttons - overall layout is a two-pane horizontal window (controls, stack, status on left, variables on right) - treeview widget used to replace custom scrolled-listbox stackviewer and custom canvas-based namespaceviewer - integrated locals/globals into a single treeview on right; can collapse/expand each independently instead of having checkboxes to control whether shown or not

was being used by stack trace in debugger

- button captions now use TkIconFont (a bit bigger than before) - status line now uses default font, plus derives italic variation for errors

roseman

Oops, sorry about the committed merge (can you tell I'm not using git every day)? In any case, I've changed the hard-coded fonts as per Elisha's comment.

E-Paine

Massive visual improvement over the original! We have a few methods which don't do anything (add_varheader, mouse_moved_vars and leave_vars): I assume there's a plan for them but for now should we add a TODO to them?

terryjreedy

I only reviewed the comments, not the whole patch.

I verified that scrolled list is only used here and can go if not used here.
What is the replacement?

bedevere-bot

A Python core developer has requested some changes be made to your pull request before we can consider merging it. If you could please address their requests along with any other requests in other reviews from core developers that would be appreciated.

Once you have made the requested changes, please leave a comment on this pull request containing the phrase I have made the requested changes; please review again. I will then notify any core developers who have left a review that you're ready for them to take another look at this pull request.

terryjreedy

All REs with '' should be prefixed with 'r' because uses like . and \d not recognized by standard string processing are deprecated and will become errors in the future.

Co-authored-by: Terry Jan Reedy <tjreedy@udel.edu> Co-authored-by: E-Paine <63801254+E-Paine@users.noreply.github.com>

roseman

cleaned up the code and documentation in add_stackframee... not a shock but it was actually not doing anything as is because the 're' module hadn't been imported, which the bare exception oh-so-helpfully hid.. 😬

E-Paine

Just a general thing, is there a way to make the panedwindow more obvious? (when I first looked at the new design, I mistook it for padding until I looked at the code)

terryjreedy

Follow PEP-8 in comments and my comment on clarifying debugger separation.

The IDLE Doc says of debugger "This feature is still incomplete and somewhat experimental. " Would you consider this still true after this patch?

E-Paine

Ubuntu failure related to issue 42142 (I have noted this there)

terryjreedy

@E-Paine Thank you. GH will not let me download the failure log.

…idth

Co-authored-by: Terry Jan Reedy <tjreedy@udel.edu>

E-Paine

I believe all review comments have been addressed except for the lazy/greedy regex one
Does anyone have a high-dpi monitor to test this patch on? (I am concerned that the images will be too small so very hard to see on such a monitor)

roseman

related to issue #14111... i note that the 'stop' button in the in-progress debugger ui is actually doing 'quit' (and doesn't in fact work when in the middle of a busy loop).

i think it would make sense to add a 'pause' button beside go, which can be used to pause the running program, allowing it to be continued. i'm wondering if the stop button should be renamed or icon changed to make it more clear that it will terminate the running program altogether.

should probably add tool tips on the debugger controls as well

github-actions

This PR is stale because it has been open for 30 days with no activity.

github-actions

This PR is stale because it has been open for 30 days with no activity.

roseman requested a review from terryjreedy as a code owner October 24, 2020 20:42

the-knights-who-say-ni added the CLA signed label Oct 24, 2020

bedevere-bot added the awaiting review label Oct 24, 2020

roseman added 3 commits October 25, 2020 10:39

scrolledlist no longer needed …

3cabc0a

was being used by stack trace in debugger

adjust fonts to use standard named fonts …

a8b8da4

- button captions now use TkIconFont (a bit bigger than before) - status line now uses default font, plus derives italic variation for errors

roseman requested review from markshannon, methane and rhettinger as code owners October 25, 2020 17:44

roseman marked this pull request as draft October 25, 2020 18:15

roseman force-pushed the debugger-new-ui-17942 branch from 06a7185 to a8b8da4 Compare October 25, 2020 18:26

roseman marked this pull request as ready for review October 25, 2020 18:33

methane removed their request for review October 26, 2020 06:54

markshannon removed their request for review October 27, 2020 18:58

E-Paine reviewed Oct 31, 2020

View reviewed changes

terryjreedy requested changes Nov 1, 2020

View reviewed changes

bedevere-bot removed the awaiting review label Nov 1, 2020

bedevere-bot added the awaiting changes label Nov 1, 2020

roseman and others added 3 commits November 2, 2020 07:39

Apply suggestions from code review …

7787255

Co-authored-by: Terry Jan Reedy <tjreedy@udel.edu> Co-authored-by: E-Paine <63801254+E-Paine@users.noreply.github.com>

apply suggestions from code review

37399ed

apply suggestions from code review

041d552

E-Paine reviewed Nov 2, 2020

View reviewed changes

clean up based on review comments - re, exception

448527e

explicit tkinter imports as per review comments

a057032

terryjreedy requested changes Nov 4, 2020

View reviewed changes

roseman and others added 4 commits November 15, 2020 14:25

status line always goes below toolbar buttons, regardless of window w… …

4304969

…idth

Apply suggestions from code review (comments) …

812768f

Co-authored-by: Terry Jan Reedy <tjreedy@udel.edu>

missing import

72740a3

tweak comments as per review suggestions

f9f9331

github-actions Bot added the stale Stale PR or inactive for long period of time. label Jan 6, 2021

rhettinger removed their request for review May 3, 2022 06:15

ezio-melotti removed the CLA signed label Jul 13, 2022

github-actions Bot removed the stale Stale PR or inactive for long period of time. label Jul 30, 2022

terryjreedy added the topic-IDLE label Nov 2, 2022

terryjreedy self-assigned this Nov 2, 2022

terryjreedy changed the title ~~bpo-17942: major rework of debugger UI~~ Nov 2, 2022

terryjreedy changed the title ~~gh-62142: IDLE - major rework of debugger UI~~ Nov 2, 2022

terryjreedy mentioned this pull request Aug 6, 2024

IDLE Debugger: Improve GUI #62142

Open

github-actions Bot added the stale Stale PR or inactive for long period of time. label Apr 8, 2026

Conversation

roseman commented Oct 24, 2020 • edited by terryjreedy Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

E-Paine commented Oct 25, 2020

Uh oh!

roseman commented Oct 25, 2020

Uh oh!

E-Paine left a comment

Choose a reason for hiding this comment

Uh oh!

terryjreedy left a comment

Choose a reason for hiding this comment

Uh oh!

bedevere-bot commented Nov 1, 2020

Uh oh!

terryjreedy commented Nov 2, 2020

Uh oh!

roseman commented Nov 4, 2020

Uh oh!

E-Paine commented Nov 4, 2020

Uh oh!

terryjreedy left a comment

Choose a reason for hiding this comment

Uh oh!

E-Paine commented Nov 4, 2020

Uh oh!

terryjreedy commented Nov 4, 2020

Uh oh!

E-Paine commented Nov 17, 2020

Uh oh!

roseman commented Dec 6, 2020 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

github-actions Bot commented Jan 6, 2021

Uh oh!

github-actions Bot commented Apr 8, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

roseman commented Oct 24, 2020 •

edited by terryjreedy

Loading

roseman commented Dec 6, 2020 •

edited

Loading