-
-
Notifications
You must be signed in to change notification settings - Fork 18.8k
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
Add tab for custom nodes #92039
base: master
Are you sure you want to change the base?
Add tab for custom nodes #92039
Conversation
Hmm, this may add some UX friction. What happens when you search for a custom node and the builtin node tab is open? And does this window remember which tab was opened last and reopen on the same tab? Will this pr add another click every other time i want to add a node? Regardless, i always thought it was pretty clear which nodes were custom in the node list because they have the script name behind them. In terms of cleanliness, idk if this is worth the UX friction. |
I think this can be good and it can help resolve some problems, but it should be a dropdown with checkboxes for which nodes you want to see (built-in, custom, editor...). There's a proposal for showing editor nodes, but they'd need to be disabled at first. Nodes that are ancestors but don't fit into the checked category should be dimmed out and not selectable. |
I think this categorization will help users who manually select nodes. But it becomes more cumbersome for users who use the search function to create nodes. Maybe this can be not actual TabContainers, but two toggle buttons for filtering built-in and custom nodes. When searching anything, these two toggle buttons are disabled so it always search among all nodes. |
bb3d27b
to
0f1274a
Compare
With the toggle buttons (suggested by @timothyqiu): Example.mp4 |
By the way, I think that "Filter" term alone is confusing most of the time. "Filter out" would be more precise about effect of toggle button IMHO. |
That feels like the opposite of what it is, I'd just remove the line. |
I agree that 'filter' is confusing. I can see something like what @MewPurPur suggested: a dropdown named 'show' with checkboxes for which nodes you want to see (built-in, custom, editor). On top of that, seeing that last video, I would also suggest that maybe if only custom nodes are shown, that their parent classes are not shown as well, because technically those are built-in nodes? Not 100% on that, though. |
This may be a partial assistance in realization of godotengine/godot-proposals#8741, as the proposal mentioned the similar functionality of this one |
There are many good ideas. So far we have: a) dropdown with checkboxes Personally I think they're all great. Let me know which one I should choose or if it's better to push an update and video of each one first. 👍 |
In adddition, (this may be a bit off-topic but it's worth thinking of) if you don't mind, you can expand in the future the field from merely custom nodes to both custom nodes and custom resourses, because I found that when you arrange create_dialog.cpp, the same change may happen to the dialoge of creatibg new resourse as well. |
I think filtering OUT results options should be inside Matches window space in order to make it obvious that is is filtrring OUT results of search. |
I think that, for UX reasons too, a simpler quick way to filter is preferable than a drop down list. Check boxes of what we want or not is preferable IMHO. I would put inside Matches window a label "Show only : " with 2 check boxes "Built-in Nodes", "Custom Nodes", checked by default when popup window opens. I think this is good and complementary with godotengine/godot-proposals#8741 as Built-in and Custom are obvious default categories, not personnal ones. [Edit: typos] |
Hello! Just passing by to say that I'm going to implement what @Lazy-Rabbit-2001 proposed since his/her proposal is more detailed and no objections were raised since the last update. I think I can finish it by tomorrow and upload the code and video. If there are any other points that anyone would like to highlight (or change), please let me know. |
0f1274a
to
d235857
Compare
Implementation of godotengine/godot-proposals#8741 with a few changes:
Categories.mp4All changes are stored in |
I don't think copying child without distinguishable marks is a good idea, as children classes may be categorized in other categories as well. In this case, I think you can check how the superseded proposal of that one handled this situation, which may contribute to your progress a lot Edit: BTW, it's better to consider these:
|
Adds a new tab dedicated to displaying the nodes accessible by the editor through "class_name" so as not to mix with the built-in nodes.
To avoid affecting the logic of the dialog, I copied the custom nodes to the new tab and hid them from the main tab.