On Tue Feb 4 16:39:58 2025 +0000, Rémi Bernon wrote:
Not sure why is_wxrformat_compatible_with_visual has to be so
unobvious and depend on given visual id matching default for some reason, can't we just match argb visual with argb and non-argb with non-argb? We know that `argb_visual` has alpha because it's selected like that. However, the systray visual can be anything, and it depends on what the systray owner window has been created with. The specification says that it "must either be the default visual for the screen or it must be a TrueColor visual. If this property is set to a visual with an alpha channel, ..." https://specifications.freedesktop.org/systemtray-spec/0.3/manager-hints.html
I think we shouldn't think about systray gory details here, as soon as we match given visual and not assuming default_argb? What do you see wrong with checking for matching alpha presence as in my patch?