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

Linking a PopupMenu to a StatusIndicator forces the PopupMenu to use the Prefer Native Menu option #93623

Open
L-the-forgettable opened this issue Jun 26, 2024 · 3 comments

Comments

@L-the-forgettable
Copy link

Tested versions

  • Reproducible in v4.3.beta2.official [b75f048]
    StatusIndicator and the "Prefer Native Menu" option on PopupMenu are not available on the latest version of 4.2.2

System information

Windows 10 - v4.3.beta2.official [b75f048]

Issue description

When linking a PopupMenu to a StatusIndicator, the PopupMenu is forced to use the "Prefer Native Window" option regardless of how the PopupMenu was triggered

Steps to reproduce

  1. Create PopupMenu and StatusIndicator nodes
  2. Link PopupMenu to StatusIndicator node
  3. Add an item to the item list
  4. Trigger the PopupMenu outside of the StatusIndicator node (e.g. with .popup())

Minimal reproduction project (MRP)

bug-report.zip
I've included this project in two other bug reports I've posted recently - If you're coming from/going to one of the other two this is a part of, rest assured you don't need to redownload the project for each report. They are the exact same file.

@akien-mga
Copy link
Member

CC @bruvzg

@bruvzg bruvzg self-assigned this Jun 26, 2024
@bruvzg
Copy link
Member

bruvzg commented Jul 8, 2024

The same is true also for children of MenuBar, or nested PopupMenus. I'm not sure if anything should be done with it (except adding a note in the documentation and adding is_native_menu method to check actual status, like MenuBar has), these are cases when menu must be native regardless of settings, and it can't be both native and non-native at the same time.

@L-the-forgettable
Copy link
Author

Yeah, something in the documentation would be nice, for all 3 of these even as I had no idea this was intended behavior.

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

3 participants