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

Using unique names for nodes of different types #43

Closed
LeonCheung opened this issue Oct 25, 2014 · 11 comments
Closed

Using unique names for nodes of different types #43

LeonCheung opened this issue Oct 25, 2014 · 11 comments

Comments

@LeonCheung
Copy link
Contributor

Hi, since I got used to add nodes by searching, I think this would be a much quicker way to add nodes (for me at least :P). However, just found it didn't work well with the recent changes.

cofl

You can see from the screenshot, I can hardly tell which "Animate" or "Math" is the one I really want. So I think unique names for them will be helpful here?

@LeonCheung LeonCheung changed the title Using unique names for nodes with different types Using unique names for nodes of different types Oct 25, 2014
@JacquesLucke
Copy link
Owner

Do you want to do that youself?
Go to the nodes folder. There is a file called mn_node_list and there you can change the menu-name for every node very easy

@LeonCheung
Copy link
Contributor Author

You mean it is something that should be DIY locally? Btw, I find four "Combine" nodes in the search menu, which is really confusing I suppose LOL. And, some nodes are not quite aligned with its default name, for example, Substrings is called "Slice" now, but the title for the new-created one is still called "Substrings". I think they should use the same title, right?

@JacquesLucke
Copy link
Owner

yeah you're right.
I never used the search, so I forget that. These 4 combine nodes would make sense if you would only use the normal menu.
Vector -> Combine
Color -> Combine
Text -> Combine, etc.

But as I said. feel free to change the names in this file to something more useful:
https://github.com/JacquesLucke/animation-nodes/blob/master/nodes/mn_node_list.py
and then make a Pull Request again :)

I'm doing something else currently

@LeonCheung
Copy link
Contributor Author

Sure, I'm glad to help on improvement on usibility. But I still want to discuss the neccessarity with you, in case you may disagree with it :) Believe me, Adding nodes by searching is the fastest way to do so. Fine, I'll try to find all things I can think of on naming things and pull, then leave them to you to decide whether to merge or not. :P

@JacquesLucke
Copy link
Owner

I plan to integrate some Pie menus in the future for some nodes. This is even faster I think :p

Ok thank you

@LeonCheung
Copy link
Contributor Author

Omg, that would a great idea! Yeah, it is even more possible since the new 2.72 feature allow things like triggering sub-menus more conveniently.

@JacquesLucke
Copy link
Owner

yeees
but this is all stuff for the future (for me at least)

I mostly work on some internal stuff and the more difficult nodes

Everything else should be much easier then

@JacquesLucke
Copy link
Owner

thanks. these changes are really good :)

@JacquesLucke
Copy link
Owner

should the sound nodes also have Sound in their name?
then you can search for Sound and get the two nodes

@LeonCheung
Copy link
Contributor Author

That would be nice. The previous pulls focused more on distinguishability on convenient searching. I leave them untouched because they got their unique titles (so far). But Adding sound would be even nicer for searching especially. :)

@JacquesLucke
Copy link
Owner

Feel free to do so hehe

kdsin23 pushed a commit to kdsin23/animation_nodes that referenced this issue Sep 21, 2020
Fix Slice List node for negative start index.
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

No branches or pull requests

2 participants