Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SplitWidget fixes #977

Merged
merged 3 commits into from
Jul 13, 2021
Merged

SplitWidget fixes #977

merged 3 commits into from
Jul 13, 2021

Conversation

corranwebster
Copy link
Contributor

@corranwebster corranwebster commented Jul 13, 2021

Some improvements to the SplitWidget class and related classes (particularly SplitPanel)

  • clean up of traits (including adding an "Orientation" trait)
  • deprecate automatic creation of widgets in SplitPanel.__init__ (part of The _create method should be public #729)
  • add an Orientation trait and use it where applicable

- clean up of traits (including adding an "Orientation" trait)
- deprecate automatic creation of widgets in __init__
- general clean-up
@rahulporuri rahulporuri self-requested a review July 13, 2021 12:13
Copy link
Contributor

@rahulporuri rahulporuri left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM with one small comment

Comment on lines +37 to +42
warnings.warn(
"automatic widget creation is deprecated and will be removed "
"in a future Pyface version, use create=False and explicitly "
"call create() for future behaviour",
PendingDeprecationWarning,
)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same comment here regarding an issue to handle this deprecation - looks like a single issue will suffice to handle all such deprecations related to the create method.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same reply as before - I've updated #729 to add the sequence around this.

Actually, should probably add #729 to the 8.0 milestone.

@corranwebster corranwebster merged commit 131a3ce into master Jul 13, 2021
@corranwebster corranwebster deleted the enh/split-widget-fixes branch July 13, 2021 12:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants