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

Don't use notify-send on linux #215

Open
RedHatter opened this issue Dec 24, 2017 · 3 comments
Open

Don't use notify-send on linux #215

RedHatter opened this issue Dec 24, 2017 · 3 comments

Comments

@RedHatter
Copy link

notify-send on linux while convenient, is slow and doesn't support actions or notification timeout. A better option would be to use libnotify directly (with C bindings). Even better would be to use the DBus interface so as to not depend on the libnotify library. See below for some examples.

libnotify-ffil Uses libnotify directly.
freedesktop-notifications Uses DBus.

The best thing to do might be to depend on freedesktop-notifications and simply pass along the information to their api for linux. Why reinvent the wheel, right?

@mikaelbr
Copy link
Owner

Hi! If you would be interested I would happily accept a PR on this.

@albinscode
Copy link

Hello there, yes you're right @RedHatter, in linux node-notifier just wraps the notify-send command.

It is enough if we wan't only to push notifications without managing actions on notifications. But in my case I need it.
I can test it a provide a PR if you are interested @mikaelbr. I saw that lots of people have forked this project, don't know if it has already been done ;)

@Hill-98
Copy link

Hill-98 commented Feb 16, 2022

Hi all, this is my desktop notification library for Linux, it uses dbus to create notifications and has full functionality. i will consider merging it into this project, but try it first if needed.

node-dbus-notifier

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants