qnapi icon indicating copy to clipboard operation
qnapi copied to clipboard

Load tray icon from theme if found

Open bilelmoussaoui opened this issue 6 years ago • 0 comments

Hey,

Can you please make the tray icon themeable? The tray icon, for now, is loaded from the qrc. Which makes it looks a bit strange with the rest of the monochrome tray icons on Linux. The best way to handle this is to make the tray icon themeable. Load it from the theme if found, if not fall back to qrc so everyone is happy. See here https://github.com/QNapi/qnapi/blob/cdc431b2214ae45e420286ff460bb0131d7a1360/gui/src/qnapiapp.cpp#L166

You can use QIcon::fromTheme("qnapi-tray") and install that icon on hicolor/status. Providing an SVG icon would be really nice so it can scale correctly. The tray icons should provide at least 16, 22, 24 sizes.

I can create a PR with the code patch, but my design skills are so bad to provide a tray icon :)

Thanks.

bilelmoussaoui avatar Aug 30 '17 12:08 bilelmoussaoui