[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] Add notifications.el
From: |
Tassilo Horn |
Subject: |
Re: [PATCH] Add notifications.el |
Date: |
Wed, 9 Jun 2010 20:18:09 +0200 |
User-agent: |
KMail/1.13.3 (Linux/2.6.34-gentoo; KDE/4.4.4; x86_64; ; ) |
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
- Re: [PATCH] Add notifications.el, (continued)
- Re: [PATCH] Add notifications.el, Michael Albinus, 2010/06/08
- Re: [PATCH] Add notifications.el, Julien Danjou, 2010/06/08
- Re: [PATCH] Add notifications.el, Jan Moringen, 2010/06/10
- Re: [PATCH] Add notifications.el, Julien Danjou, 2010/06/11
- Message not available
- Re: [PATCH] Add notifications.el, Jan Moringen, 2010/06/11
- Re: [PATCH] Add notifications.el, Julien Danjou, 2010/06/11
- Re: [PATCH] Add notifications.el, Michael Albinus, 2010/06/11
- Re: [PATCH] Add notifications.el, Jan Moringen, 2010/06/11
- Re: [PATCH] Add notifications.el, Stefan Monnier, 2010/06/08
- Re: [PATCH] Add notifications.el, Michael Albinus, 2010/06/09
Re: [PATCH] Add notifications.el,
Tassilo Horn <=
- Re: [PATCH] Add notifications.el, Julien Danjou, 2010/06/09
- Re: [PATCH] Add notifications.el, Tassilo Horn, 2010/06/09
- Re: [PATCH] Add notifications.el, Julien Danjou, 2010/06/09
- Re: [PATCH] Add notifications.el, Tassilo Horn, 2010/06/09
- Re: [PATCH] Add notifications.el, Stefan Monnier, 2010/06/09
- Re: [PATCH] Add notifications.el, Julien Danjou, 2010/06/09
- Re: [PATCH] Add notifications.el, Stefan Monnier, 2010/06/09
- Re: [PATCH] Add notifications.el, Tassilo Horn, 2010/06/10
- Re: [PATCH] Add notifications.el, Julien Danjou, 2010/06/10
- Re: [PATCH] Add notifications.el, Michael Albinus, 2010/06/10