mate-desktop / mate-notification-daemon

Daemon to display passive pop-up notifications
https://mate-desktop.org
GNU General Public License v2.0
30 stars 26 forks source link

Add support for GNotification #108

Closed kopr12 closed 8 years ago

raveit65 commented 8 years ago

Can you provide a test case please? Btw, which app use GNotification?

kopr12 commented 8 years ago

https://developer.gnome.org/GNotification/

Thing is, notification will show but without icon, with this commit icon will show. image_data and image_path are deprecated, see here : https://developer.gnome.org/notification-spec/ But we leave them because libnotify is still using those. Many apps are using GNotification, you can try to compile sample code from the first link or the code from here and see for yourself that it's not working : http://stackoverflow.com/questions/27341087/glib-gnotification-doesnt-show-icon

raveit65 commented 8 years ago

I only asked because with providing a quick test case in general for PRs we can test all faster. Don't forget, only 2 ppls do the reviews at the moment. Thx, i will look into it.

kopr12 commented 8 years ago

Sure, no problem, I understand. It was my mistake not to describe the problem in the first place.

kopr12 commented 8 years ago

To save you time and trouble here's my code I was using to test this : http://www.hastebin.com/cebalesafo.avrasm

or if you don't wanna mess with your custom images a simpler version using themed icon : http://www.hastebin.com/jaqusukazi.avrasm

In any case, you will see that icon doesn't show and that it shows with this commit.

raveit65 commented 8 years ago

Does what it promised, thank you.

monsta commented 8 years ago

It seems to me that the second part of commit adds already existing code...

kopr12 commented 8 years ago

I'm not sure what you mean by that, first part adds image-data hint and second part adds image-path hint, those were not in the code before. Can you point to that existing code ?

monsta commented 8 years ago

Ah... I mixed image_path and image-path, sorry. :smile:

kopr12 commented 8 years ago

np :)