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

No way to add an item to 'tools button' (EditorPlugin.add_tool_menu_item) #9969

Closed
Tracked by #6276
kubecz3k opened this issue Jul 29, 2017 · 8 comments
Closed
Tracked by #6276

Comments

@kubecz3k
Copy link
Contributor

kubecz3k commented Jul 29, 2017

Operating system or device - Godot version:
b276d92

Issue description:
It seems since 0aa7242 logic to do this task is non existent.

Steps to reproduce:

  1. Download Godot sources.
  2. Open EditorPlugin.cpp
  3. Check bindings near 400 line,
  4. See that add_tool_menu_item and remove_tool_menu_item bindings are commented, try to find their logic.
  5. See that add_tool_submenu_item has some commented logic, and don't seems to do anything.

Link to minimal example project:

@neikeq
Copy link
Contributor

neikeq commented Aug 10, 2017

Why was this removed?

@neikeq
Copy link
Contributor

neikeq commented Oct 6, 2017

@reduz I asked you about this the other day, and you said you thought this feature was removed. Is this feature meant to be gone and why? Was there any problem with it?

@reduz
Copy link
Member

reduz commented Oct 6, 2017 via email

@neikeq
Copy link
Contributor

neikeq commented Oct 6, 2017

But, as stated by OP, the logic for these methods was removed by 0aa7242. Why?

@mhilbrunner
Copy link
Member

@reduz Why was this commented out? Needed this & stumbled upon this issue

@vnen
Copy link
Member

vnen commented Jan 23, 2018

This was likely changed when the menus got rearranged. Tools used to be a top-level menu, now it's a submenu of Project. The code to add the menu may need some change in logic, I don't think it's just a matter of uncommenting it.

@mhilbrunner
Copy link
Member

Thanks for the explanation, makes sense :)

@neikeq
Copy link
Contributor

neikeq commented Jan 30, 2018

Would love to see this added back. I don't like to have the Mono menu button on a corner 😟

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants