enthought / enthought/pyface

Review/clean-up Field class

Open
#1,021 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
115
Forks
54
PR merge metrics
No merged PRs in 30d

Description

Once #1020 is merged the `Field` class and the associated testing code have become very thin and have no toolkit-specific code in them.

We should either:
- simplify things (eg. move the implementation of `Field` out of the toolkit specific code, remove empty classes)
- find some new toolkit-dependent functionality that really belongs on a generic `Field` and implement it
- (in the case of testing code) see if there are common things in the subclasses that belong in the mixin.

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.