notifications icon indicating copy to clipboard operation
notifications copied to clipboard

Odin beta 2 notification bubble icon question

Open WebShapedBiz opened this issue 4 years ago • 8 comments

Describe the bug

I'm using Moderndeck as my Twitter client for some time now and it's hooked up with a notification server. In Hera, every time I get a new tweet the notification bubble pops up, showing at the left side a profile image of the user that just tweeted. In Odin beta 2 however, there is no profile image, just dialog-information status icon.

Platform Information

elementary OS 6 Early Access Linux 5.8.0-50-generic 8× AMD Ryzen 7 4800u with radeon graphics Advanced Micro Devices, Inc. [AMD®/ATI] Renoir

I'm using the latest released beta version with the latest updates at the time of writing this, vanilla installation with no compiling or custom icons or anything like that.

On both computers, Hera and Odin, the ModernDeck is installed as AppImage, and the app has both icons, in app menu and wingpanel, as you can see here: Slika zaslona s 2021-06-27 08-31-01

Don't know if it is a bug or a new feature, so if that's intentional change from Hera, sorry for this issue.

Cheers.

WebShapedBiz avatar Jun 27 '21 06:06 WebShapedBiz

@WebShapedBiz I presume it has something to do with appimages not integrating with OS apps - how were you able to get the app to show up in settings? did you create your own .desktop file?

hanaral avatar Jul 08 '21 22:07 hanaral

@hanaral I've actually copied it from my Hera computer, so it is the same .desktop file

WebShapedBiz avatar Jul 09 '21 05:07 WebShapedBiz

In that case it may be that the app running not being recognised as the source for that notification. Do you happen to know how the app is sending notifications? could it be just notify-send?

hanaral avatar Jul 09 '21 08:07 hanaral

Sorry, @hanaral, I'm just a user, I have no clue. The point is that the same app and same setup that works in Hera doesn't work in Odin, whether the app uses notify-send or not.

WebShapedBiz avatar Jul 09 '21 09:07 WebShapedBiz

I'm no developer either. I've never gotten notifications to properly work with electron apps anyway, they would pretty much always send it with the info icon. The reason it doesn't work in Odin may have something to do with the large notification server rewrite and its stricter rules for identifying apps. @jeremypw would you happen to know what might be causing this electron app to send notifs with the default icon?

hanaral avatar Jul 09 '21 09:07 hanaral

@hanaral Sorry, no. I do not know whether it is a feature of all electron apps or just this one. Probably need to look at the source code.

jeremypw avatar Jul 09 '21 12:07 jeremypw

there's two issues, one is that the server don't have support for the image-data hint and should be fixed by #123. the other is the fact that dialog-information is used instead of the application icon. and i believe is a problem with the application (probably the same issue from elementary/wingpanel-indicator-notifications#206).

electron uses the desktopName field of the application package.json as the desktop-entry hint. if that field isn't declared, productName or name is used instead.

looking the ModernDeck's package.json, it uses "ModernDeck" as productName and don't set a desktopName, so you need to make sure the .desktop file is named ModernDeck.desktop, otherwise, the notification server can't find the application .desktop.

Marukesu avatar Jul 09 '21 22:07 Marukesu

Thank you @Marukesu for your time and effort.

Cheers.

WebShapedBiz avatar Jul 10 '21 06:07 WebShapedBiz