elementary / sideload

Sideload Flatpaks on elementary OS
https://elementary.io
GNU General Public License v3.0
46 stars 14 forks source link

MainWindow: remove set_icon from notification #123

Closed danirabbit closed 3 years ago

danirabbit commented 3 years ago

This is redundant since notifications already use the app icon from appinfo automatically

cassidyjames commented 3 years ago

I agree this is redundant, but… can we add the app's icon instead? :sweat_smile: That could be a follow-up but it felt weird to merge this and then immediately re-add a badge icon.

danirabbit commented 3 years ago

@cassidyjames that's probably a larger branch. It doesn't look like there's currently anything in place to get the app icon, just its name. It might be a bit more tricky to get