EMSUSD-3340 support drag-and-drop material in AE - #4671
Merged
Conversation
pierrebai-adsk
added
adsk
Collaborator
Author
|
Without LookdevX: |
Collaborator
Author
|
With LookdevX: |
pierrebai-adsk
force-pushed
the
bailp/EMSUSD-3340/ae-DnD-material
branch
from
July 20, 2026 20:27
5c93d02 to
fc302fe
Compare
New drag-and-drop text field: - Added `DragAndDropTextField` class to make a MEL textField support drag-and-drop of text. - The class supports validation of the dropped value. - The class has callbacks for user text changes and drag-and-drop changes. Enhance AE material UI: - Use the drag-and-drop text filed in the material custom control. - Added `AssignedMaterialUI` to replace `TextField` for clarity. - Added undo label to metadata UI. - Added goto-prim button for all fields with prim paths. - Don't create the graph buttons when there is no LookdevX. Edit restrictions for material commands: - Added edit restriction enforcement in the bind, unbind and strength commands. - Made the unbind command not unbind collections. (We're not managing collections.) - Added unit tests for restrictions. - Edit-routing for the material binding commands as property routing for the property "material:binding". - We don't distinguish benteen purpose and strength for routing to avoid having too many different routings.
pierrebai-adsk
force-pushed
the
bailp/EMSUSD-3340/ae-DnD-material
branch
from
July 20, 2026 20:43
fc302fe to
6a4e52b
Compare
barbalt
reviewed
Jul 21, 2026
barbalt
reviewed
Jul 21, 2026
barbalt
previously approved these changes
Jul 21, 2026
barbalt
left a comment
barbalt
left a comment
Collaborator
There was a problem hiding this comment.
small questions. Looks good overall
Removed unused variable and moved a comment to its proper place.
barbalt
approved these changes
Jul 21, 2026
pierrebai-adsk
added
the
ready-for-merge
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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.
New drag-and-drop text field:
DragAndDropTextFieldclass to make a MEL textField support drag-and-drop of text.Enhance AE material UI:
AssignedMaterialUIto replaceTextFieldfor clarity.Edit restrictions for material commands: