advanced-github-notifier icon indicating copy to clipboard operation
advanced-github-notifier copied to clipboard

[Win] Clicking notifications (in Action Center) does nothing

Open tooomm opened this issue 6 years ago • 7 comments

As in the title. Clicking a notification should open the relevant chat (and scroll to that message and highlight it) instead only hiding the notification. The X is for that. :)

In addition, when you miss a notification or not click it when it appears...the notifications are piled up in the Windows Notification/Action Center. When I read the new comments via Firefox afterwards, there is still a mess of already read notifications left because they get not cleared up after reading. The badge on the addon icon in Firefox updates correctly in that case (but it takes a while!).

Windows 10 1903 Firefox 69.0

tooomm avatar Sep 21 '19 08:09 tooomm

Yes, that's what notifications should do. However I think this is out of my control and more of an issue with the notification API Firefox provides and how that interacts with the action center.

The extension can not influence at all how the action center handles notifications, so they will just pile up there.

Yeah, the badge and panel can lag behind a bit.

freaktechnik avatar Sep 21 '19 12:09 freaktechnik

Hmm, there is no e.g. clear command to interact with already send notifications for exammple?

tooomm avatar Sep 21 '19 13:09 tooomm

There is a function to clear notifications, however in past tests it only worked while the actual notification was believed to be shown by Firefox (so when clicking it actually works). I think it may still make sense to start clearing notifications in case that's fixed (or works on Mac OS, for example).

freaktechnik avatar Sep 22 '19 01:09 freaktechnik

The latest version (1.6.x) will clear notifications when they are no longer active, however this will likely not fix the root issue here.

freaktechnik avatar Mar 03 '20 02:03 freaktechnik

The latest version (1.6.x) will clear notifications when they are no longer active, however this will likely not fix the root issue here.

@freaktechnik Now, with 1.6.3, it looks like the notifications get cleared immediately and always right after they popped up. The extension lost functionality and is less useful now. :/

For Example: New issues is opened. Notification gets triggered and the extension shows a pop up. The notification is recognized in the action center of windows while the notification stays alive (~5s?). Native windows notification disappears. Notification is cleared from action center immediately.

The problem here is that when you leave your pc and come back you have no clue if any notifications happened and if you missed something, because every notification stays only alive in the action center during the lifetime of the pop-up itself. Once the pop-up is gone, every hint that it ever happened is gone too. So there is no way windows can tell you that you even missed a notification and that you should check the action center for details. They should only get cleared at the time you read e.g. the new issue and your extension removes the red badge from the icon in the firefox bar as well!

tooomm avatar Mar 08 '20 19:03 tooomm

Hm, I did not change anything in that regard. That must be Firefox 74 or whatever's on stable behaving differently/more consistently with what it supports.

All I added was logic to clear a notification if it was no longer returned by GitHub's notification API: https://github.com/freaktechnik/advanced-github-notifier/commit/206717c85a9b89d443283e8e60a99e0f7968e8f5

Edit: it appears the native windows notifications are still unfinished (and currently unmaintained) and only enabled in nightly: https://bugzilla.mozilla.org/show_bug.cgi?id=1497425

freaktechnik avatar Mar 09 '20 10:03 freaktechnik

Looks like we have to wait for a better integration of native notifications in windows first...

tooomm avatar Mar 11 '20 15:03 tooomm