Skip to content

Conversation

@RichardFrangenberg
Copy link

@RichardFrangenberg RichardFrangenberg commented Mar 11, 2023

Removed a line which caused that value_changed was emitted twice when a vector widget value was changed.

Also fixed a problem that the _value variable of a vector wasn't replaced with a new list when a _NumberValueEdit was changed directly. This caused that the NodeGraph class didn't emit the property_changed signal when changing the number fields of a PropColorPickerRGB widget more than once.
This is because this if condition would be false in these cases:

if node.get_property(prop_name) != prop_value:

I hope it makes sense, but I can provide more details if needed.

@jchanvfx
Copy link
Owner

Awesome thanks for catching that 😃

@jchanvfx jchanvfx merged commit 069f0ee into jchanvfx:master Mar 12, 2023
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