[Users] [Bug 3465] add libappindicator support to notification plugin

[noreply at thewildbeast.co.uk]

http://www.thewildbeast.co.uk/claws-mail/bugzilla/show_bug.cgi?id=3465

--- Comment #3 from Andrej Kacian <andrej at kacian.sk> ---
Good news at least for Debian users, the old-style tray icons will be supported
in KDE5 for the foreseeable future, see
https://lists.debian.org/msgid-search/1827691445107908@web19m.yandex.ru

In the meantime, I have played with libappindicator a bit, and found out that
working with it is rather trivial, after you get after one or two initial
gotchas. What stopped me is code structure of the notification plugin, where I
am not entirely sure what is the best way to add libappindicator support.

Logically, this would go right next to notification_trayicon.[ch] as
notification_appindicator.[ch], but in notification_trayicon.[ch], there is a
lot of code around handling of popup menu and preparing message counts. I do
not want to duplicate that code, since that is a surefire way to maintainer's
nightmare.

I was able to separate the popup menu creation into a notification_core.c (I
think), but I am unsure of what to do with the rest. It would help if Holger
could have a look at it, and suggest best solution - he knows the
notification_plugin code best. :)

-- 
You are receiving this mail because:
You are the assignee for the bug.