Skip to content

Conversation

@kitchoi
Copy link
Contributor

@kitchoi kitchoi commented Aug 18, 2020

This is a draft PR which I will close immediately. This contains proof-of-concept code which I used to experiment with ways to implement the UITester. I also tested the API with a number of editors: ButtonEditor, TextEditor, ListEditor, EnumEditor, TableEditor, TreeEditor, RangeEditor as well as the UI panel specific TabbedFoldGroupEditor.

Qt/Wx toolkit compatibility was tested for ButtonEditor and RangeEditor only.

It is open for the reference of #1149.

Caution:

  • Various classes and variables may be renamed in the future.
  • Since I was writing this mainly for testing a concept / demo, there are quite a few undocumented functions... please feel free to ask about them. And the structures may not be the most optimal.
  • Where there are documentation, they might also have been outdated.
  • It does contain more than what we need for Extend UITester API #1149.

Comment on lines +242 to +244
editor : Editor
An instance of Editor. It is assumed to be in a state after the UI
has been initialized but before it is disposed of.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

The editor attribute will be renamed, and it may not be an instance of traitsui Editor.
#1107 will clarify that.

@kitchoi kitchoi closed this Aug 18, 2020
@kitchoi kitchoi deleted the ui-tester-draft branch August 18, 2020 13:25
@kitchoi kitchoi restored the ui-tester-draft branch August 18, 2020 13:25
Comment on lines +11 to +12
self.editor_to_action_to_handler = {}
self.editor_to_location_solver = {}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

These two are independent, so we might as well have a registry for interaction, and a separate registry for locations.

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.

1 participant