Re: [PATCH] Add notifications.el

[Tassilo Horn]

Hi Julien,

I threw away my notification code and now use yours.  It's going pretty
well, except one thing I cannot figure out.  I want to use a :on-close
function, so that notifications I've explicitly clicked away don't
appear again. (They are triggered minutely by appt.el.)  But :on-close
functions only get the reason.  How do I get the id of the closed
notification?  Wouldn't it be better to pass the id to that function,
too?

And another thing: I think `notifications-notify' should error when
there is an unknown keyword.  I was nearly going mad when trying to
figure out why :replace-id didn't work.  It was, because the keyword is
:replaces-id. ;-)

Thanks for another brick in the desktop integration wall!
Tassilo