You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The messenger.com use the HTML5 Notification API. I guess this differs from Electron Notification because the notification does not stay in the Notification Center.
You mentioned the Goofy App in #298 and I found this solution in it.
I think you missed by suggested solution there though:
We don't control the notifications, but we could monkey patch Notification in the renderer (browser) process and detect when new notifications are emitted, then send them to the main process, and use the Electron Notification API which supports "reply".
Ok. Thank you.
I am going to try monkey patch but I am not an advanced JS developer. Therefore I cannot say this will be fast.
Do you have any suggestions?
…into notification
* 'notification' of https://github.com/janosorcsik/caprine:
Catch notification in the renderer and send to the main process
Fix lint errors
Notifications don't show up in the Notification Center on macOS #317
sindresorhus
changed the title
Notifications stay in the Notification Center on macOS
Make notifications stay in the Notification Center on macOS
Oct 11, 2018
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
sindresorhus
force-pushed
the
master
branch
from
Fixes #317