Skip to content

Fix the tray icon menu not working on Linux - #107

Closed
robert-ancell wants to merge 1 commit into
leanflutter:mainfrom
robert-ancell:linux-menu-not-working
Closed

robert-ancell wants to merge 1 commit into
leanflutter:mainfrom
robert-ancell:linux-menu-not-working

Conversation

@robert-ancell

Copy link
Copy Markdown

The tray icon is not activatable until the context menu trigger is set.

The tray icon is not activatable until the context menu trigger is set.
@robert-ancell

Copy link
Copy Markdown
Author

This fix makes the example app work, but I'm not 100% sure if it's the correct solution. I don't know what it will do on other platforms, or if this behaviour should be set in the app and not the library. It does seem unexpected that menus don't work by default though.

@Tobiichi-Origuchi

Tobiichi-Origuchi commented Oct 3, 2026 •

Copy link
Copy Markdown

Thank you for your work!
I ran into the same issue (the tray icon menu wasn't shown), but I'm curious why modifying example/analysis_options.yaml fixed it.
(Maybe you forgot to push the code)

@lijy91

lijy91 commented Oct 30, 2026

Copy link
Copy Markdown
Member

@copilot resolve the merge conflicts in this pull request

@lijy91

lijy91 commented Nov 1, 2026

Copy link
Copy Markdown
Member

Thanks for the contribution. The actual diff only adds analyzer exclusions in example/analysis_options.yaml; those exclusions are already present on main (since 426f0d3), along with web/**. There are no context-menu runtime changes in this PR, so merging it would add no new fix. Closing as redundant.

The Linux menu behavior described here still needs a separate fix: the legacy wrapper does not set a context-menu trigger, while nativeapi defaults to None. We are addressing that path and the README example separately, with verification on a real Linux desktop.

@lijy91 lijy91 closed this Nov 1, 2026
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 this pull request may close these issues.

3 participants