-
Notifications
You must be signed in to change notification settings - Fork 201
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
EMSUSD-550 - Disable Material section when Mesh type is excluded from export #3421
EMSUSD-550 - Disable Material section when Mesh type is excluded from export #3421
Conversation
Hi @seando-adsk , just realize I have some debug code forgot to remove. Added another commit |
} else if($optionBreakDown[0] == "exportCameras"){ | ||
mayaUsdTranslatorExport_SetOptionMenuByBool($optionBreakDown[1], $enable, "exportInstancesPopup"); | ||
} else if($optionBreakDown[0] == "exportLights") { | ||
mayaUsdTranslatorExport_SetOptionMenuByBool($optionBreakDown[1], $enable, "exportInstancesPopup"); |
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.
Why were these lines removed?
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.
Those lines are some debug code I used before and forgot to remove. They are totally useless now...
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.
Why would they be useless. Someone can call this function adding those strings to the input options to control the starting value of these two export options.
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.
Oh no those flags are not longer available. Now we use excludeExportTypes
flag, so it will be excludeExportTypes=[Cameras, Lights]
like this. These two flags are abandoned. It's my fault I forgot to delete them here...
Disable the material section when user excludes Meshes from export. Currently the material is attached to the mesh so when the mesh is not exported, the material can not be exported either.