-
Notifications
You must be signed in to change notification settings - Fork 668
Fix dropdown input nodes when running from player with Dynamo open #13403
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
Conversation
QilongTang
left a comment
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.
A few comments
QilongTang
left a comment
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.
One comment then LGTM
|
Waiting for all PR checks to finish |
|
Reported regression is a glitch |
|
@twastvedt I assume the actual testing will be done by Vera team on this |
|
@QilongTang Yes we will! |
Purpose
Fixes AGD-2777
When running a graph from Dynamo Player while the Dynamo UI was open, the value and UI display of dropdown nodes was not updated consistently.
DropDown.UpdateValueCoreuses theSelectedIndexsetter instead of its own logic.DropDown.SelectedIndexsetter runsRaisePropertyChanged("SelectedString"). (Previously it changedselectedStringbut did not alert the UI of the change.)Opportunistic fixes:
DropDown.SelectedStringsetter runsRaisePropertyChanged("SelectedIndex"). (Previously it changedselectedIndexbut did not alert the UI of the change.)DropDown.SelectedStringsetter does nothing if the given value is the same as the current value.Declarations
Check these if you believe they are true
*.resxfilesReviewers
@saintentropy