enthought / enthought/traitsui

Refactor Undo/Redo

Open
#1,060 1 comment 0 reactions 0 assignees View on GitHub
difficulty: intermediate type: important type: refactor
Dominant language
Python
Stars
306
Forks
99
PR merge metrics
No merged PRs in 30d

Description

The undo/redo code is not particularly clean or well-written. It is also not API-compatible with the apptools Undo/Redo system.

A proper re-write might mean bringing apptools Undo/Redo into Pyface (at least the core, but possibly all of it, so it can be used there) and then making the TraitsUI implement or adapt to the API for its undo/redo, so that it can integrate with application-level undo/redo.

While I suspect that this code is not heavily used externally to TraitsUI, there may need to take some care not to break the API exposed to editors and handlers without a clear migration path.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.