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-122928 - merge to USD dialog window height - cannot scroll down #2348

Merged

Conversation

seando-adsk
Copy link
Collaborator

MAYA-122928 - merge to USD dialog window height - cannot scroll down

  • Made window/form/scroll layout changes to make it similar to a normal Maya option box with the same size/spacing.

* Made window/form/scroll layout changes to make it similar to a
  normal Maya option box with the same size/spacing.
@seando-adsk seando-adsk added the bug Something isn't working label May 11, 2022
@seando-adsk seando-adsk requested a review from ppt-adsk May 11, 2022 14:46
@@ -68,11 +68,19 @@ def _createMergeToUSDOptionsDialog(window, target):

windowLayout = cmds.setParent(query=True)

menuLayout = cmds.columnLayout("MergeToUSDOptionsDialogMenuLayout", adjustableColumn=True, parent=windowLayout)
menuBarLayout = cmds.menuBarLayout(parent=windowLayout)
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Basically what I did here was use the internal tool we have to inspect windows. I compared a normal Maya option box to this one and made adjustments here with form/scroll/column layouts.

@seando-adsk seando-adsk added the ready-for-merge Development process is finished, PR is ready for merge label May 12, 2022
@seando-adsk seando-adsk merged commit c2fc726 into dev May 12, 2022
@seando-adsk seando-adsk deleted the donnels/MAYA-122928/merge_to_usd_dialog_not_scrollable branch May 12, 2022 18:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working 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.

2 participants