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

add systray.TrayOpenedCh which is triggered every time user clicks system tray #47

Merged
merged 2 commits into from
Jul 1, 2024

Conversation

pymq
Copy link

@pymq pymq commented May 8, 2023

My use case is to refresh some data each time user clicks icon and update submenus

Unfortunately I don't have mac to implement it on darwin

Probably related to #23

@andydotxyz
Copy link
Member

Won't the unix implementation trigger every time a submenu is opened as well? This doesn't sound like what you intended from the PR title.

@pymq
Copy link
Author

pymq commented May 9, 2023

oh, thanks, you are right! fixed that

@andydotxyz
Copy link
Member

Is there any way (or anyone listening) to get the macOS support added?

@pymq
Copy link
Author

pymq commented May 13, 2023

I found this example of similar feature, maybe it will help somebody. Without ways to test it I can't implement this, unfortunately

@davidnewhall
Copy link

🖖 I'd be happy to help test, but my usefulness at adapting Xcode/cocoa is pretty limited. The SO you linked looks promising, but all that "low-level" code is just not something I'm familiar with. That said, I maintain two heavily used apps that import the getlantern variant of this library. The fyne-io version seems like it should drop right in. I'm specifically looking for a way to open the apps' web ui when the menu is double-clicked. This contribution doesn't quite nail that, but it's pretty close, and moving it forward should make my goal easier to get to. If you'd like to stick some code into a branch and link me up, I'll tell you what happens on a Mac, at least. Might even be able to tinker on it. You can also drop my by discord and we can chat. https://golift.io/discord

@thallgren
Copy link

This PR would make our life easier. It also relates to this ticket that I just created. Anything I can do to help this getting merged? I can test on both Linux and macOS (Intel or Silicon).

@andydotxyz
Copy link
Member

I think we're looking for the Darwin code to be slotted in then tested

@andydotxyz andydotxyz merged commit 2272c23 into fyne-io:master Jul 1, 2024
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.

4 participants