FIX: Parameter editor not reacting to changes to InputSettings #2130
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Fix for a bug (ISXB-1531) in parameter UITK editor which wouldn't update its visual representation if setting default values would be changed while the editor is still open. Additionally did some slight refactoring to eliminate some redundant code.
I did not attempt to fix the missing tri-state representation which should really be done to make this implementation better.
Testing status & QA
Tested manually by changing settings default values and toggling defaults on/off.
Overall Product Risks
Comments to reviewers
Recommend testing different scenarios of switching settings, changing default etc with asset editor open.
Checklist
Before review:
Changed,Fixed,Addedsections.Area_CanDoX,Area_CanDoX_EvenIfYIsTheCase,Area_WhenIDoX_AndYHappens_ThisIsTheResult.During merge:
NEW: ___.FIX: ___.DOCS: ___.CHANGE: ___.RELEASE: 1.1.0-preview.3.After merge: