@zander please have a look at: #5625. I think the problem addresses in the PR is win only.

I suggest using #ifdef Q_OS_WIN then for this code; so it won't make things worse on other OS's (esp linux) where this behavior change is not wanted.

@zander in general we like to keep the platform dependent stuff (#ifdef) on a minimum. And the problem with mixed colors could occur on other platforms.

Tested on Win,OSX,Ubuntu (change only visible on Windows, see below).

Tested ACK.
But it brought me to a new issue: #5653

And the problem with mixed colors could occur on other platforms.

The "problem" is debatable. Having the same icon in all places is quite important for a stable UI (consistency).
Anywhere I look having monochrome icons that follow the text color just isn't the default, and the menu in the latest screenshot looks broken. Its completely inconsistent with all the other apps because they all use color icons. Its also inconsistent with itself because the "Show / Hide" for instance is not actually the logo you see in the taskbar, its got a different color.

Bottom line, I have zero apps on either Windows (8.1) nor on KDE that have monochrome icons in their systray context menu. So this just doesn't look consistent in any way at all.

@zander The discussion over the monochrome icons has already been held. But feel free to contribute over issue and new pull requests. This pull requests focus on #5625.

ACK.