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
Fix dynamic selection #256
base: master
Are you sure you want to change the base?
Conversation
itemWasSingleClicked = QtCore.pyqtSignal() | ||
itemWasDoubleClicked = QtCore.pyqtSignal() | ||
|
||
DOUBLE_CLICK_TIMEOUT = 200 # ms |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
300
@@ -351,6 +351,9 @@ def child_type(self): | |||
else: | |||
raise TypeError("Attempted to get child types, but no Either.subtypes allow children.", self) | |||
|
|||
def get_child_type(self, type_cls): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
get_child_of_type
@@ -45,7 +45,8 @@ def rename_option(self, old_option, option): | |||
self._record_update(update) | |||
|
|||
def swap_options(self, option0, option1): | |||
placeholder = str(uuid.uuid4()) | |||
# TODO: fix this hack, only using <name> because it isn't shown in OptionTree.children |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fix in #260
Selecting an item in a
DynamicAttrsOf
orDynamicListOf
changes the navlist to aStaticAttrsOf
. This is incorrect behavior, the selected attributes type shouldn't be considered when determining the navlist class, the attribute containing the navlist should determine the navlist class.Single/Double click: fixes #173
Dynamic Navlist Mutation
swap_options
andrefresh
are called forDynamicListOf.up_clicked
anddown_clicked
.OptionTree.rename_attribute
Bugs:
EitherType(OneOfType())
is broken because theFieldWidget
cannot findEitherType.choices
. Ensure choices can be discovered.insert_items
dead code needs to be removedDocumentation
architecture.org
Further PRs (before v0.2 release):
DynamicListOf
andDynamicAttrsOf
so it follows an update-StateModel -> refresh navlist procedure to isolate the source of state to one place. This applies to all operations.max_renderable_field_widgets
so the universe of fields aren't rendered.