-
Notifications
You must be signed in to change notification settings - Fork 1k
fix(icon): use Qt tray icon in all cases, remove platform specific backends #5888
Conversation
4adc96f
to
ad11885
Compare
ad11885
to
b4bb10b
Compare
Testing completed, results listed at #5881 (comment). No regression found on all tested OSes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please also update INSTALL.md where it lists the dependencies and explains the now removed configuration switches
Reviewed 6 of 6 files at r1.
Reviewable status: 0 of 1 LGTMs obtained
…ckends qTox added specialized tray icon backends over time to work around bugs with the default implementation, since then our GTK+2 backend has stopped working, our appindicator backend was never selected by default by cmake, and statusnotifier was never selected at SystemTrayIcon construction, leaving us only ever using the broken GTK+2 backend, or theoretically the Unity backend, which I didn't see selected on Ubuntu 16.04 Unity. In all other cases we would fall back to the Qt backend. Qt icon has improved over time, and our platform specific icons have become stale, with GTK+2 becoming deprecated and GTK+3 not having a similar feature, QSystemTrayIcon has been tested on a variety of DEs and works as well or better in all cases, as shown in the table at qTox#5881 (comment) Fix qTox#5881 Fix qTox#5859
b4bb10b
to
e85d3f5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please also update INSTALL.md where it lists the dependencies and explains the now removed configuration switches
Done.
Reviewable status: 0 of 1 LGTMs obtained
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewed 1 of 1 files at r2.
Reviewable status: complete! 1 of 1 LGTMs obtained
Options were removed in qTox#5888
qTox added specialized tray icon backends over time to work around bugs with
the default implementation, since then our GTK+2 backend has stopped working,
our appindicator backend was never selected by default by cmake, and
statusnotifier was never selected at SystemTrayIcon construction, leaving us
only ever using the broken GTK+2 backend, or theoretically the Unity backend,
which I didn't see selected on Ubuntu 16.04 Unity. In all other cases we would
fall back to the Qt backend.
Qt icon has improved over time, and our platform specific icons have become
stale, with GTK+2 becoming deprecated and GTK+3 not having a similar feature,
QSystemTrayIcon has been tested on a variety of DEs and works as well or better
in all cases, as shown in the table at
#5881 (comment)
Fix #5881
Fix #5859
This change is