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

created a new hero, an adaptation of keeper of the light called : the… #12

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

jeltedeproft
Copy link
Collaborator

added new hero, keeper of the light aka the lightswitcher

@darklordabc
Copy link
Owner

darklordabc commented Mar 19, 2017

Resolved conflicts and made some small improvements.
Also, I don't think the ultimate works, was it working for you? When I use it, nothing happens. @jeltedeproft

@jeltedeproft
Copy link
Collaborator Author

it was a bug with the name of the modifier, reuploading now

@jeltedeproft
Copy link
Collaborator Author

i updated the changes and pushed to remote, do i have to create a new pull request now?

@darklordabc
Copy link
Owner

no

@darklordabc
Copy link
Owner

@jeltedeproft looks pretty good, ultimate is working great. Could you add a tooltip for this modifier please? It looks a bit tacky without a title and description. And then we can merge it.
image

@jeltedeproft
Copy link
Collaborator Author

of course, there you go, added a fitting description. im still in the proces of learning blender to make custom models, sometime in the future i'll create a custom model for this hero as well

@darklordabc
Copy link
Owner

Sorry I'll merge this soon.

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.

2 participants