search

jwiegley / alert

A Growl-like alerts notifier for Emacs
Other
441 stars 66 forks source link

notifications: Add a default action to open corresponding buffer #79

Closed DamienCassou closed 5 years ago

DamienCassou commented 5 years ago

The 'notifications' style supports attaching actions to a notification. This commit attaches a default action to every notification that opens the buffer associated with the alert.

jwiegley commented 5 years ago

Are we sure we want this to start happening?

DamienCassou commented 5 years ago

Are we sure we want this to start happening?

I don't understand your concern. An action has to be triggered manually for it to execute: i.e, the alert's buffer will only be switched to if the user clicks the alert or types a dedicated keystroke (depending on the notification daemon. In other words, this commits adds a new feature but no one is forced to use it.

DamienCassou commented 5 years ago

I updated my branch. Care to have a look?