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

Move all Shader Node related code and assets to Material Support module #402

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

neph1
Copy link
Contributor

@neph1 neph1 commented Oct 19, 2022

This is an intermediate step which may not be 100% complete yet. I'll use this PR to change icons in the Shader Editor as well.

I found it odd that these classes and icons were part of the "core", rather than "materialdefinition" package. Seemed like an oversight that no one bothered to fix.

If there's a good reason why it should be in core, let me know, and I'll close this PR.

@neph1 neph1 requested a review from MeFisto94 October 19, 2022 19:12
@neph1 neph1 marked this pull request as ready for review October 23, 2022 07:35
@neph1 neph1 added this to the 3.6 milestone Jan 3, 2023
@neph1 neph1 removed this from the 3.6 milestone Jan 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant