Cleanup: Remove unnecessary "add node search" #112056

Merged
Hans Goudey merged 2 commits from HooglyBoogly/blender:cleanup-add-node-search-remove into main 2023-09-11 18:36:17 +02:00

2 Commits

Author SHA1 Message Date
e0c3674b77 Merge branch 'main' into cleanup-add-node-search-remove 2023-09-11 12:25:15 -04:00
dcc9627af7 Cleanup: Remove unnecessary "add node search"
Now that specific menus can be searched directly (see 7f9d51853c),
there is no need to maintain separate search functionality for adding
nodes. This PR removes the add node search. In a way this brings us
closer to the `NodeItem` situation before, but the setup is more
flexible since the menus are more standard and easier to customize.

In the few ways we customized the node search items before, this gives
us the same results as before. Overall the searching is less flexible,
but I think that is jsut a tradeoff we have to accept for the simplicity
of searching menus. In the future menus could be made more dynamic,
with each builtin node's menu path stored on the node type, similar to
assets. That might be a nice compromise. In the meantime this code
is just dead weight.
2023-09-06 16:15:55 -04:00