Comment 11 for bug 334344

Revision history for this message
In , Alexander Sack (asac) wrote :

IMO, this patch is not enough for thunderbird. The new notification system involves improvement to the notification bubbles, but since it removes the ability to use actions, it is supposed to go in together with a more stateful indicator for message notifications: libindicate.

Just this patch would be a regression, as it a) doesn't support accumulated notifications (e.g. notify more than one new message) and b) libnotify implementation on trunk doesn't provide actions - which is why we didn't pull this to 1.9.1 branch yet; and c) notification daemon in ubuntu will not have action support.

The complete approach would be:
 1. design a toolkit interface for abstract messaging notifications
 2. implement a default implementation that is xul based (in xulrunner)
 3. implement libindicate/libnotify support which is used if both are available on the system (in xulrunner)
 4. make thunderbird (and other messagin/chat apps based on xulrunner) use new messaging notification component.