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

Autodesk/tremblp/maya 103495/contextual operations #303

Merged
merged 6 commits into from
Mar 3, 2020

Conversation

ppt-adsk
Copy link
Collaborator

No description provided.

@ppt-adsk
Copy link
Collaborator Author

Hi Pilar, we're adding you as a reviewer for contextual menu support through UFE. This will allow the Maya Outliner to pop up right click menu items that are USD-specific for USD selected items (e.g. visibility, variant set selection, etc.). We felt that either you or one of your colleagues at Blue Sky would be interested in this. Note that the version of UFE required to actually get this to work in Maya is not yet available externally, so you won't be able to try it out immediately. I realize it's a bit abstract until you see the UFE base class interface for it, but hopefully it will give you a flavor of what's possible, namely:

  • Data model specific right click menu items (specific to USD)
  • Extensible menu items
  • Full undo / redo capability, where appropriate
    Let me know if you feel you're not in a good position to provide a review. Thanks!

Copy link
Contributor

@pilarmolinalopez pilarmolinalopez left a comment

Choose a reason for hiding this comment

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

Took a look at the code and I agree with the missing documentation comments. We can't test this yet and I think we'll have more questions once we see the UFE base class and try to add our own contextual operations. But everything else looks to good to me! Thanks!

@cfmoore007
Copy link

Question - will it be possible to add drag/drop capability ? ( ie: from a custom asset manager UI to the outliner ) Thought I'd bring it up here in the 'context' of things :-)

We can log this as a separate github issue ( enhancement ).

@kxl-adsk kxl-adsk added pending beta This issue or PR will be testable in next beta release requires maya This issue requires changes in Maya ufe Related to UFE component in Maya ufe-usd Related to UFE-USD plugin in Maya-Usd labels Mar 3, 2020
@kxl-adsk
Copy link

kxl-adsk commented Mar 3, 2020

Given that @ppt-adsk is off and this PR is needed for dev branch to build against next Maya beta - I made requested improvements to the documentation.

@kxl-adsk
Copy link

kxl-adsk commented Mar 3, 2020

Question - will it be possible to add drag/drop capability ? ( ie: from a custom asset manager UI to the outliner ) Thought I'd bring it up here in the 'context' of things :-)
We can log this as a separate github issue ( enhancement ).

@cfmoore007 Yes, please log it.

@kxl-adsk kxl-adsk merged commit 2e7f85a into dev Mar 3, 2020
@kxl-adsk kxl-adsk deleted the Autodesk/tremblp/MAYA-103495/contextual_operations branch March 3, 2020 18:17
ppt-adsk pushed a commit that referenced this pull request Feb 28, 2023
* MAYA-126389 : fix selection color for wireframe display.

(cherry picked from commit 2498a978074c66f41942aa5f26e34c3e52a6f1b2)

* Adding more comments and factorize.

(cherry picked from commit 1491fecdbdf1b62b48f05c8704dca5a66a3f6931)

* Fixed a mistake in init color of kLead.

(cherry picked from commit 658bc3a453395e5d0038807ff8c025f909f67b29)

* MAYA-126389 : Fix wireframe colors for selection highlight.

* MAYA-126389 : Implement suggestions from code review and fix the first letter of the RenderItem name being eaten.

* Remove fix for MAYA-125923 and reopen the JIRA.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pending beta This issue or PR will be testable in next beta release requires maya This issue requires changes in Maya ufe Related to UFE component in Maya ufe-usd Related to UFE-USD plugin in Maya-Usd
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants