Sunday, January 30, 2022

Re: [net/tdesktop] enable dbus integration

Cc'ing all maintainers...

On Sun, Jan 30, 2022 at 05:21:45PM +0100, Omar Polo wrote:
> As discussed previously, here's a diff to enable the dbus integration
> for tdesktop. tested for days on amd64, works fine :)
>
> I'm particularly interested in this bit because it allows to "use native
> notifications" (settings -> notifications -> native notifications.)

Understanble, but upstream removed these bits:

commit b65d40a22b3dd3ed9cdb8937543f7b6ea53cec3e
Author: Ilya Fedin <fedin-ilja2010@ya.ru>
Date: Fri Jan 21 20:51:43 2022 +0400

Get rid of custom SNI implementation

XDG is inventing new tray specification, so SNI will be outdated soon and it's better to just use QSystemTrayIcon.
I believe all the major drawbacks of QSystemTrayIcon are solved and we can live with minor ones.
Given the planned MainWindow refactoring, it seems it's the best time to do that.

Not sure if enabling it now is the best idea.

Can we wait for the next update and go with QSystemTrayIcon or whatever
they've come up with?

No comments:

Post a Comment