-
lovetox
hannibal, thanks for the info, but this alone is not enough, GLib needs also to support FileIcon for the notification portal backend
-
lovetox
and then its not even clear this is the best solution
-
lovetox
currently we need to save avatar icons in a small format to a cache folder just so we can pass a file path, this means more writes to disk, more reads
-
lovetox
we do this only now for the freedesktop backend, because GLib simply does not support pixbuf or bytesicon
-
lovetox
as i understand it they will never support pixbuf, and bytesicon would need to be implemented
-
lovetox
hm seems its not necessary for the portal backend to add support because they serialize the whole notification
-
lovetox
so ignore that
-
bot
lovetox pushed 1 commits to branch gajim/master ci: Allow debian deploy to fail - https://dev.gajim.org/gajim/gajim/-/commit/3adad910cca245e10f7c2da1a9178a5aee45f6ba
-
rom1dep
> this means more writes to disk, more reads lovetox: Maybe something like this could be considered? https://unix.stackexchange.com/questions/66990/create-a-virtual-file-that-is-actually-a-command
-
hannibal
> i think we didnt have notifications with avatars on flatpak, but with this MR this should now work lovetox, avatars on flatpak work with current master. Do you still want me to test the MR with flatpak?
-
lovetox
User Avatar in a notification?
-
hannibal
yes
-
lovetox
.. this should not work
-
lovetox
are you using gnome?
-
hannibal
yes
-
lovetox
ok this shit is 20 layers deep
-
lovetox
we call Gio.Notification, which determines a backend, in flatpak case the portal backend, which passes the notification object further witout manipulating it
-
lovetox
then it depends what portal backend the xdg-desktop-portal uses
-
lovetox
in the case of gnome desktop env it uses probably xdg-desktop-portal-gtk
-
hannibal
There is also xdg-desktop-portal-gnome
-
lovetox
xdg-desktop-portal-gtk now checks if dbus name org.gtk.Notifications is available
-
lovetox
and if so, it just passes the notification object unseen to the dbus interface, which support pixbufs
-
lovetox
if the name is not available, it uses the fdo notification dbus name
-
lovetox
org.freedesktop.Notifications
-
lovetox
but this one does not support serialized notification objects, so xdg-desktop-portal-gtk needs to convert the notification object to whatever API this dbus interface wnats
-
lovetox
and there, it does not support conversion of all icons, and especially not pixbuf
-
lovetox
so yes hannibal please test the branch
-
lovetox
because it works only by accident for you because you use gnome
-
lovetox
and xdg-desktop-portal-gnome seems to have no notification code at all
-
lovetox
so dont know how that actually works if someone has installed that
-
lovetox
then of course there are even more backends, for example KDE does also only support a icon-name and bytes icon
-
Mike Yellow
## Versions: https://conference.gajim.org:5281/pastebin/e37db1d0-662a-438e-88a2-c1b1f75c929e
-
cal0pteryx
Mike Yellow, restart Gajim. it might be that there was a third party app blocking database access, or gajim wasn't shutdown properly
-
Mike Yellow
Thank you.
-
taba
Mike Yellow: yo
-
taba
can't message you
-
pcy012314jabbim.com
1
-
bot
lovetox pushed 1 commits to branch gajim/master cfix: OMEMO: Handle unknown fingerprint correctly - https://dev.gajim.org/gajim/gajim/-/commit/f3971067b7d11dbbe6a8e6c756a4635b2cd963f9
-
bot
lovetox pushed 2 commits to branch gajim/master imprv: Notifications: Make avatars available on more platforms - https://dev.gajim.org/gajim/gajim/-/commit/0215516bd40bbba6c6f8702ec3a801404efb7814 refactor: Notifications: Improve Linux backend - https://dev.gajim.org/gajim/gajim/-/commit/a46997d8e971b3ae06c7a1d4bbfa8de4006677f9