Bug 397248

Summary:	Tray icon not used form Plasma theme
Product:	[Applications] krfb	Reporter:	varlesh <varlesh>
Component:	general	Assignee:	George Goldberg <grundleborg>
Status:	CONFIRMED ---
Severity:	normal	CC:	kde, trmdi
Priority:	NOR
Version First Reported In:	unspecified
Target Milestone:	---
Platform:	Manjaro
OS:	Linux
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:
Attachments:	preview bug

Description varlesh 2018-08-07 19:21:05 UTC

All KDE applications used icons for tray from plasma theme. But KRFB not... it's spoil style with monochrome icons on tray.
After add file krfb.svg on plasma theme with ID's:
- krfb
- 22-22-krfb
KRFB used system icon from system icon theme, but not from plasma theme.

https://github.com/PapirusDevelopmentTeam/adapta-kde/issues/73

Comment 1 varlesh 2018-08-07 19:23:59 UTC

Created attachment 114362 [details]
preview bug

Comment 2 Kai Uwe Broulik 2018-08-08 13:15:45 UTC

The problem is that krfb sets a custom pixmap icon:

setIconByPixmap(QIcon::fromTheme("krfb").pixmap(22, 22, QIcon::Disabled));

So it can be "disabled" or "enabled" depending on whether there is a client connected or not.

Moreover, QIcon::fromTheme uses the application icon theme instead of Plasma's.