-
Notifications
You must be signed in to change notification settings - Fork 27
Notification feedback #14
Comments
|
For notifications on Windows maybe |
I think that's a regression, notifications did work with an older build on Windows.
|
I can't give an example, but I found a similar question (with screenshot): electron/electron#3148 |
Nevermind, it was a false positive, it never worked on Windows. See electron/electron#262 |
Just noticed that you linked to an issue I opened on the Electron repo. Here's a gist of the solution I ended up with to provide badges across OS X and Windows if you're curious: https://gist.github.com/johnthedebs/c22a04fcd08e598e69b8 |
Neat, I'll look into adding that into the code I have for the badges. |
That worked great, I've added it in PR #18. |
Thanks for the screenshot, this looks nice. |
PR #18 merged, we now have Windows and OS X covered. Linux remains. |
@lloeki have you tried to click on the notification when the app in minimised on OSX? I found out that the app changes the views inside mattermost but the app itself doesn't get restored as the browser does. Any thoughts? |
The text was updated successfully, but these errors were encountered: