welle.io icon indicating copy to clipboard operation
welle.io copied to clipboard

Append theme path from resources

Open dforsi opened this issue 2 years ago • 1 comments

Fixes issue #713.

Appending would allow local icons to take precedence over those bundled.

Calling QIcon::setThemeSearchPaths(themePaths); replaces the whole list of paths which results in missing icons on some systems. Adding qDebug() << QIcon::themeSearchPaths(); before and after the call to setThemeSearchPaths() before this patch:

2022-03-11T20:12:10.344 Debug: ("/home/user/.local/share/icons", "/usr/share/icons", "/var/lib/snapd/desktop/icons", ":/icons")
2022-03-11T20:12:10.344 Debug: (":/icons/welle_io_icons")

after this patch:

2022-03-11T20:14:59.384 Debug: ("/home/user/.local/share/icons", "/usr/share/icons", "/var/lib/snapd/desktop/icons", ":/icons")
2022-03-11T20:14:59.384 Debug: ("/home/user/.local/share/icons", "/usr/share/icons", "/var/lib/snapd/desktop/icons", ":/icons", ":/icons/welle_io_icons")

and all the icons are visible for me.

dforsi avatar Mar 11 '22 19:03 dforsi

any news on this please ?

tenzap avatar Jul 05 '22 12:07 tenzap

Thanks for this fix!

AlbrechtL avatar Aug 21 '22 11:08 AlbrechtL