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

Spawn test notifications is broken #2

Closed
bel57 opened this issue Apr 24, 2024 · 8 comments
Closed

Spawn test notifications is broken #2

bel57 opened this issue Apr 24, 2024 · 8 comments

Comments

@bel57
Copy link

bel57 commented Apr 24, 2024

Title.
Clicking on spawn notification does nothing..
I've got lucky once by restarting the app and spamming the button, but that's it. Can you look into it?
As a workaround, I have to send a message to self (IM, email, etc.)

Win11 23H2
v2.1.3
fr-FR

@SamarthMP
Copy link
Collaborator

The current implementation is using WinForms NotifyIcon, I will look into changing the method of spawning notifications to be more compatible.

@SamarthMP
Copy link
Collaborator

This is possibly an issue with your system's registry, try this.

From StackOverflow:

This was the fix in Windows 10:
Open regedit.exe
Navigate to HKCU\SOFTWARE\Microsoft\Windows\CurrentVersion\Explorer\Advanced
Set (or add) EnableBalloonTips (as REG_DWORD) and set value to 1
Reboot.

@bel57
Copy link
Author

bel57 commented Apr 24, 2024

Isn't that registry trick to bring back legacy balloon tooltips? I have no issue with notifications banners and tooltips, usually. Indeed I don't have this in the registry and don't think it's needed 🤔

@bel57
Copy link
Author

bel57 commented Apr 24, 2024

Looks like there is no need to mess with the registry, can you update your code instead?
Read this

@SamarthMP
Copy link
Collaborator

In Windows 10/11, balloon tips are notifications, so therefore creating a balloon tip will create a notification. I will check if there are other better methods to create a notification.

@bel57
Copy link
Author

bel57 commented Apr 24, 2024

I find it weird that it popped up, at least once. Again, as the app is working fine for me so that test button is not a priority. Thanks for your work!

@SamarthMP
Copy link
Collaborator

I have released 2.1.4, which uses Microsoft.Toolkit.Uwp.Notifications instead of NotifyIcon, please take a look and see if it's working for you.

@bel57
Copy link
Author

bel57 commented Apr 24, 2024

Confirmed, it's working perfectly now. Thank you for for the quick fix!

@bel57 bel57 closed this as completed Apr 24, 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

No branches or pull requests

2 participants