ClearWindow.py adding to standard IDLE - 2 lines of code needs fix

[Aust01]

The ClearWindow.py extension needs fix:

1. On line 64, from idlelib.UndoDelegator import DeleteCommand the UndoDelegator does not exist. It is located in idlelib.undo FIX: from idlelib.undo import DeleteCommand

2. On the line 94, prompt = strip_ansi(sys.ps1), the sys.ps1 is not contained in sysmodule.c. my suggestion: line 93 add this FIX: sys.ps1 = '>>> '

[serwy]

What do you want to accomplish? Is ClearWindow.py under consideration for inclusion to IDLE itself?

IDLE in Python 3.6 had a large renaming of its contents, which IdleX did not support. Instead, IdleX forked IDLE from 3.5, before the renaming.

The strip_ansi is an artifact from when IdleX supported IPython.

[Aust01]

I already did my thing and I don't follow any news about IDLE so I can't tell if it is under consideration.

However, It should be included, otherwise It was a wasted time.

Anyway, thanks for replying

On Thursday, 29 August 2019, Roger D. Serwy [email protected] wrote:

What do you want to accomplish? Is ClearWindow.py under consideration for inclusion to IDLE itself?

IDLE in Python 3.6 had a large renaming of its contents, which IdleX did not support. Instead, IdleX forked IDLE from 3.5, before the renaming.

The strip_ansi is an artifact from when IdleX supported IPython.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/serwy/idlex/issues/9?email_source=notifications&email_token=AMHD6QW6HDM36U2QCPWXOFLQHBA4JA5CNFSM4HSARBTKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5P6HPY#issuecomment-526377919, or mute the thread https://github.com/notifications/unsubscribe-auth/AMHD6QTYNTSNAJCHAK7TUADQHBA4JANCNFSM4HSARBTA .