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

Node context menu has missing label, shows "(label)" instead #195

Open
1 task done
ralphmeijerambassadors opened this issue Dec 13, 2024 · 2 comments
Open
1 task done
Assignees
Labels
community Issues and PRs coming from the community members

Comments

@ralphmeijerambassadors
Copy link

Is there an existing issue for this?

  • I have searched the existing issues and added correct labels.

Description

Current Behavior

There seems to be a proper definition of a label missing in when you right click on a node. Now it says "(label)":
Image

Expected Behavior

I'm assuming there should be a proper name for the label.

Steps To Reproduce:

Create a node and press right click.

Additional context:

Version

Version of the server, ayon-core and the addon that's affected

What platform were you running when you found the bug?

Windows
Linux - Rocky9
Linux - Ubuntu
MacOS

You can also provide any additional information relevant to the issue.

Relevant log output:

@BigRoy BigRoy changed the title missing label in rightclick menu? Node context menu has missing label, shows "(label)" instead Dec 15, 2024
@BigRoy BigRoy added the type: bug Something isn't working label Dec 15, 2024
@BigRoy
Copy link
Contributor

BigRoy commented Dec 15, 2024

Thanks for creating the issue. I had seen this too. I actually believe it's because we're attaching to an id in the menu that doesn't exist - I believe the id or menu structure changed somewhere in Houdini 20.0 or 20.5 and this worked before in older versions of Houdini.

We should definitely patch this up and create a dedicated AYON menu entry.

@MustafaJafar MustafaJafar added the community Issues and PRs coming from the community members label Dec 16, 2024
@MustafaJafar
Copy link
Contributor

Thank you for creating this issue. let me provide further info.
Menu hierarchy has changed.
Here's a comparison between 19.5 and 20.5

Image

  • 19.5:

    • Menu
      • opmenu.vhda_options_create
        • opmenu.vhda_create
        • opmenu.unsynchronize
  • 20.5:

    • Menu
      • opmenu.vhda_create
      • opmenu.unsynchronize

The missing label you are referring to is actually, the missing opmenu.vhda_options_create.
as the current OPMenu in AYON is as follows:

<menu>
<!-- Operator type and asset options. -->
<subMenu id="opmenu.vhda_options_create">
<insertBefore>opmenu.unsynchronize</insertBefore>
<scriptItem id="opmenu.vhda_create_ayon">
<insertAfter>opmenu.vhda_create</insertAfter>
<label>Create New (AYON)...</label>

@mkolar mkolar removed the type: bug Something isn't working label Feb 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community Issues and PRs coming from the community members
Projects
None yet
Development

No branches or pull requests

4 participants