Skip to content
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

MAYA-129141 - As a user, I'd like to set my anonymous layers as relative in Bulk Save's 'Save As' dialog #3104

Merged
merged 1 commit into from
May 19, 2023

Conversation

vlasovi
Copy link
Collaborator

@vlasovi vlasovi commented May 18, 2023

Ability to bulk-save layers with relative paths

Copy link
Collaborator

@pierrebai-adsk pierrebai-adsk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll wait for the answer before approving.

@vlasovi vlasovi added the ready-for-merge Development process is finished, PR is ready for merge label May 19, 2023
@vlasovi
Copy link
Collaborator Author

vlasovi commented May 19, 2023

@seando-adsk This PR can be merged now. The failing preflight is due to a known issue in testUsdExportPackage, which is not related to my changes.

Comment on lines +123 to +127
$titleCaption = "Save As";
$actionCaption = "Save";
} else {
$titleCaption = "Set As";
$actionCaption = "Set";
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These strings should really be in resource dictionary for future translation. But actually these strings are probably already in one (or more) of the Maya dictionary files. Search for uses of uiRes and you can see how we can directly use a string from Maya.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@seando-adsk If it's ok with you, I would have this PR merged as is and apply this change with my next PR, as there will be more changes going into Batch Save UI.

@seando-adsk seando-adsk added the enhancement New feature or request label May 19, 2023
@seando-adsk seando-adsk merged commit 360bd55 into dev May 19, 2023
@seando-adsk seando-adsk deleted the vlasovi/MAYA-129141 branch May 19, 2023 18:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request ready-for-merge Development process is finished, PR is ready for merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants