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

menuhook doesn't restore menu items that the user has manually removed via Customize User Interface #14

Open
ForrestTrepte opened this issue Oct 15, 2020 · 1 comment · May be fixed by #15

Comments

@ForrestTrepte
Copy link

ForrestTrepte commented Oct 15, 2020

An artist installed a tool that was registered in the 3dsMax menus via menuhook. Subsequently the tool was uninstalled and the artist manually deleted the menu entries. Later, the artist was unable to use a new version of the tool because menuhook did not add the menu items back.

Repro steps:

  1. Install the HowTo samples by running install.sh
  2. Run 3dsMax, note the presence of menu items such as Scripting > Python3 Development > Browse Documentation > Getting Started With Python in 3ds Max
  3. Manually remove the Browse Documentation menu as follows
    • Open Customize > Customize User Interface
    • Select the Menus tab and on the right, under Main Menu Bar expand Scripting and expand Python3 Development.
    • Delete the item Browse Documentation
  4. Exit and restart 3dsMax

Result: The Browse Documentation menu is no longer present. Even if you were to reinstall the HowTos nothing will cause the menu to be recreated again.

Expected: On startup, the samples call menuhook.register and that should restore the menu that was deleted.

image

@ForrestTrepte ForrestTrepte linked a pull request Oct 15, 2020 that will close this issue
@hugow
Copy link
Collaborator

hugow commented Oct 21, 2020

Thanks a lot for this! I will take a deeper look.

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 a pull request may close this issue.

2 participants