enthought / enthought/traitsui

Remove imports from `traitsui.editors.__init__`

Open
#1,759 0 comments 0 reactions 0 assignees View on GitHub
component: core difficulty: easy type: cleanup
Dominant language
Python
Stars
306
Forks
99
PR merge metrics
No merged PRs in 30d

Description

A long time ago, all editor factories were imported directly from `traitsui.editors`. Over a decade ago, the `traitsui.editors.api` module was added, but import from `traitsui.editors` was still kept for backwards compatibility.

However this causes problems with circular imports in a number of places (eg. see discussion here: https://github.com/enthought/traitsui/blob/9f4acb87865e331b566628b9e4ecc08bec61e8d7/traitsui/null/rgb_color_trait.py#L145-L151).

We should remove the imports from `traitsui.editors.__init__` and fix situations where we are working around circular imports.

This would technically be a breaking change, but the impact is like very low at this point.

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.